Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Nginx subdir headers to match webroot config #1597

Merged
merged 1 commit into from
Sep 23, 2019

Conversation

MichaIng
Copy link
Member

  • Ref: Add X-Frame-Options header to nginx  #1520
  • The master merge to resolve conflicts has reverted the headers changes for the subdir config. This commit redoes the intended changes, to match Nginx webroot config and .htaccess.

Signed-off-by: Micha Felle micha@dietpi.com

+ Ref: #1520
+ The master merge to resolve conflicts has reverted the headers changes for the subdir config. This commit redoes the intended changes, to match Nginx webroot config and .htaccess.

Signed-off-by: Micha Felle <micha@dietpi.com>
@skjnldsv
Copy link
Member

Bump!

@skjnldsv
Copy link
Member

/backport to stable17

@J0WI
Copy link
Contributor

J0WI commented Sep 23, 2019

Introduced with d2963c0

@skjnldsv skjnldsv merged commit 9958118 into master Sep 23, 2019
@welcome
Copy link

welcome bot commented Sep 23, 2019

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/documentation/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22
Most developers hang out on IRC. So join #nextcloud-dev on Freenode for a chat!

@backportbot-nextcloud
Copy link

backport to stable17 in #1631

backportbot-nextcloud bot pushed a commit that referenced this pull request Nov 20, 2019
Signed-off-by: Bernhard Ostertag <bernieo.github@gmx.de>

Nextcloud 17 adds the Same-Origin header which was removed with Nextcloud 12.
ref #1597 (which should IMHO be merged before release of Nextcloud 17)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants