coop-cloud/nextcloud
9
6
Fork 12
Code Issues 23 Pull Requests 8 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
Copy Source

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
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
Copy Source

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
bug
Something is not working
 documentation
For issues that are calling out documentation gap
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) BornDeleuze Brooke cas (Cassowary) coopcloud dannygroenewegen (Danny Groenewegen) decentral1se (d1) fauno (fauno) ineiti (Linus Gasser) javielico (Javielico) jjsfunhouse kawaiipunk (KawaiiPunk) knoflook moosemower notplants oxaliq (sorrel) p4u1 renovate-bot (Comrade Renovate Bot) stevensting trav (Trav Fryer) yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/nextcloud#28