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

Nextcloud 14 Beta 3 The "Referrer-Policy" HTTP header is not set to "no-referrer" #10624

Closed
ghost opened this issue Aug 9, 2018 · 12 comments
Closed
Labels
bug design Design, UI, UX, etc.

Comments

@ghost
Copy link

ghost commented Aug 9, 2018

Hello getting this error in NC 14 Beta 3

image

@nextcloud-bot nextcloud-bot added the design Design, UI, UX, etc. label Aug 9, 2018
@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #10502 (Nextcloud 14 Beta 3), #8550 (The "X-Frame-Options" HTTP header is not set to "SAMEORIGIN".), #10605 (Nextcloud 14 Beta 4), #10410 (Nextcloud 14 Beta 2), and #9122 (Add setupcheck for Referrer-Policy header).

@rullzer
Copy link
Member

rullzer commented Aug 9, 2018

So it is not and error it is a warning as you can see.
It is warning you a header is not set which makes your browser forward any referrer information which is not recommended. So the system is actually doing what it should do. Warning you.

@rullzer rullzer closed this as completed Aug 9, 2018
@ghost
Copy link
Author

ghost commented Aug 9, 2018 via email

@rullzer
Copy link
Member

rullzer commented Aug 9, 2018

Ah no. It is a new security header. We only added a check for it in NC14. That si why it shows up in the beta ;)

@ghost
Copy link
Author

ghost commented Aug 16, 2018

@rullzer is this warrning always going to show?

@rullzer
Copy link
Member

rullzer commented Aug 16, 2018

Yes. Unless you set the header. Same as for the other security header we recommend

@ghost
Copy link
Author

ghost commented Aug 16, 2018

@rullzer Thank you! how do i set the header and where ?

@ghost
Copy link
Author

ghost commented Aug 20, 2018

@rullzer any update on this?

@ShipNecro
Copy link

@andyxh This header needs to be added into your vhost conf.

Ex: <VirtualHost *:443>
Header add Strict-Transport-Security: "max-age=15768000;includeSubdomains"
Header always set Referrer-Policy "strict-origin"

@mathiasconradt
Copy link

Yes. Unless you set the header. Same as for the other security header we recommend

It should be added to the NC docs as well, not? The location where and what do add, like with this other header that's documented. Currently there is a W3C link, but I think it's not very comprehensive for everyone.

@penCsharpener
Copy link

I agree with @mathiasconradt. Only figured out what to do with @ShipNecro comment.

@chriscroome
Copy link

Why not add these headers to the .htaccess file that Nextcloud generates (on Apache servers)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug design Design, UI, UX, etc.
Projects
None yet
Development

No branches or pull requests

6 participants