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

[stable25] Change X-Robots-Tag header from "none" to "noindex, nofollow" #36749

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #36689

@backportbot-nextcloud backportbot-nextcloud bot added bug enhancement pending documentation This pull request needs an associated documentation update privacy labels Feb 16, 2023
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 25.0.4 milestone Feb 16, 2023
@szaimen
Copy link
Contributor

szaimen commented Feb 16, 2023

Not sure about this backport as this woild be a breaking change on some instances?

@MichaIng
Copy link
Member

It does not "break" anything but may show a new admin panel warning until admin takes action.

I still wonder whether we should explain the change a little, when the value "none" is found, like

The X-Robots-Tag HTTP header is set to "none", which prevents Google from indexing it, but Bing and other search machines have been found to not support this value. To prevent your Nextcloud from appearing in all common search machines, we recommend to switch the value to "noindex, nofollow".

So it is hopefully made clear that this is not just for bothering people but has some direct impact on privacy and indirectly security of Nextcloud instances which are not meant for public access.

@blizzz blizzz added the 3. to review Waiting for reviews label Mar 15, 2023
@blizzz blizzz merged commit 892a804 into stable25 Mar 15, 2023
@blizzz blizzz deleted the backport/36689/stable25 branch March 15, 2023 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug enhancement pending documentation This pull request needs an associated documentation update privacy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants