coop-cloud/nextcloud
coop-cloud
/
nextcloud
5
1
Fork 3
Code Issues 6 Pull Requests Releases Wiki Activity

add headers to embed nextcloud in frame on external site #28

Merged
yksflip merged 1 commits from embed_nextcloud_in_iframe into main 2022-09-02 14:32:05 +00:00
Conversation 0 Commits 1 Files Changed 5 +36 -3
yksflip commented 2022-08-31 13:51:35 +00:00
Owner
Copy Link

This introduces new env variables to configure nextloud to be embedded via
iframe on an external site.
Setting X_FRAME_OPTIONS_ENABLED=1 will configure nginx and nextcloud to
set X-Frame-Options and CSP headers to allow the domain configured in
X_FRAME_OPTIONS_ALLOW_FROM.

I created a PR because I'm not sure if this is helpful for other people or just a custom hack that bloats the recipe :D

This introduces new env variables to configure nextloud to be embedded via iframe on an external site. Setting X_FRAME_OPTIONS_ENABLED=1 will configure nginx and nextcloud to set X-Frame-Options and CSP headers to allow the domain configured in X_FRAME_OPTIONS_ALLOW_FROM. I created a PR because I'm not sure if this is helpful for other people or just a custom hack that bloats the recipe :D
yksflip added 1 commit 2022-08-31 13:51:36 +00:00
continuous-integration/drone/pr Build is failing Details
e8a8f636d0 add headers to embed nextcloud in frame on external site 
This introduces new env variables to configure nextloud to be embedded via
iframe on an external site.
Setting X_FRAME_OPTIONS_ENABLED=1 will configure nginx and nextcloud to
set X-Frame-Options and CSP headers to allow the domain configured in
X_FRAME_OPTIONS_ALLOW_FROM.
decentral1se approved these changes 2022-08-31 14:02:20 +00:00
decentral1se left a comment
Owner
Copy Link

LGTM! I guess if someone needs it (you :)) then another might! It follows the usual dance of overriding things (i.e. entrypoint.sh) so it fits for me. I won't use this though so maybe someone else can weigh in. Feel free to merge imho.

LGTM! I guess if someone needs it (you :)) then another might! It follows the usual dance of overriding things (i.e. `entrypoint.sh`) so it fits for me. I won't use this though so maybe someone else can weigh in. Feel free to merge imho.
yksflip merged commit 16ad6c22ea into main 2022-09-02 14:32:05 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
decentral1se
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/nextcloud#28
No description provided.
Delete Branch "embed_nextcloud_in_iframe"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?