coop-cloud/nextcloud
coop-cloud/nextcloud
6
2
Fork 6
Code Issues 10 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 1 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
add headers to embed nextcloud in frame on external site
Some checks failed
continuous-integration/drone/pr Build is failing
Details
e8a8f636d0 
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
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 Apfelwurm appletalk arjan basebuilder Brooke carla cas codegod100 coopcloud decentral1se fauno flancian Frando iexos javielico kawaiipunk knoflook lambdabundesverband mac-chaffee marlon mayel mirsal moritz nicksellen notplants p4u1 pau renovate-bot rix sef simon stevensting tobias trav val virtualboys wolcen wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
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?