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

Remove anonymizer support #1358

Merged
merged 1 commit into from
Sep 29, 2022
Merged

Remove anonymizer support #1358

merged 1 commit into from
Sep 29, 2022

Conversation

jtojnar
Copy link
Member

@jtojnar jtojnar commented Sep 29, 2022

Initially implemented in #104, these were mostly meant to strip the Referer header. But we have long been specifying rel="noreferer" on links, which achieved the same effect without going through a questionable third-party website that usually messed up the viewed web page.

For better privacy, web browser extensions like NoScript or uBlock Origin and Tor or a VPN provider are probably much better anyway.

Closes: #612

Initially implemented in #104,
these were mostly meant to strip the `Referer` header.
But we have long been specifying `rel="noreferer"` on links,
which achieved the same effect without going through a questionable
third-party website that usually messed up the viewed web page.

For better privacy, web browser extensions like NoScript or uBlock Origin
and Tor or a VPN provider are probably much better anyway.

Closes: #612
@jtojnar jtojnar merged commit 1b25717 into master Sep 29, 2022
@jtojnar jtojnar deleted the drop-anonymizer branch September 29, 2022 01:40
@jtojnar jtojnar added this to the 2.19 milestone Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Anonymizer breaks fancy-box
1 participant