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

Onion location header redirects to homepage #724

Closed
deeplow opened this issue Aug 22, 2020 · 2 comments
Closed

Onion location header redirects to homepage #724

deeplow opened this issue Aug 22, 2020 · 2 comments
Assignees

Comments

@deeplow
Copy link

deeplow commented Aug 22, 2020

The "Onion location header" introduced in #695 redirects the user to the onion version of the securedrop website.

The user is being redirected to the homepage instead of to the page where they were, which is quite annoying.

A lot of websites had this problem. See how they fixed it on the Qubes website for inspiration.

@conorsch
Copy link
Contributor

@deeplow Thanks for pointing this out, you're right that it's a bit frustrating to navigate to a subpage and be redirected to the onion version of the homepage. Adding a bit more detail:

How it currently works

  1. Navigate to https://securedrop.org/news/piloting-securedrop-workstation-qubes-os/ in Tor Browser
  2. Click the Onion Available button
  3. Observe redirect to https://secrdrop5wyphb5x.onion/

How it Should™ work

  1. Navigate to https://securedrop.org/news/piloting-securedrop-workstation-qubes-os/ in Tor Browser
  2. Click the Onion Available button
  3. Observe redirect to https://secrdrop5wyphb5x.onion/news/piloting-securedrop-workstation-qubes-os/

The latter definitely feels much more intuitive to me, so let's go with that.

@chigby chigby self-assigned this Sep 9, 2020
@conorsch
Copy link
Contributor

This is now fixed. Note that we've updated to a v3 Onion URL as part of recent improvements, and retired the old v2 URL. SInce Onion-Location headers are served, Tor Browser will discover it automatically.

Just confirmed via interactive testing that https://securedrop.org/news/piloting-securedrop-workstation-qubes-os/ shows "onion available", and clicking on it takes me to https://securedrop.org/news/piloting-securedrop-workstation-qubes-os/

Thanks for your thoughtful report in the original post, @deeplow!

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 a pull request may close this issue.

3 participants