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.

Sign up for GitHub

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

Jump to bottom

Preserve protocol on .org -> .io domain redirect #2175

Closed
dcarley opened this issue Apr 28, 2016 · 3 comments
Closed

Preserve protocol on .org -> .io domain redirect #2175

dcarley opened this issue Apr 28, 2016 · 3 comments

Comments

@dcarley
Copy link

dcarley commented Apr 28, 2016

The redirect from the old readthedocs.org to the new readthedocs.io domain should preserve the protocol of the original request.

We always reference the HTTPS URL of our project, but the redirect currently sends people back to HTTP:

➜  paas-team-manual git:(master) ✗ curl -s -D- https://government-paas-team-manual.readthedocs.org/ | grep Location
Location: http://government-paas-team-manual.readthedocs.io/

For context, this is the email I received about the change of domains:

Starting today, Read the Docs will start hosting projects from subdomains on the domain readthedocs.io, instead of on readthedocs.org. This change addresses some security concerns around site cookies while hosting user generated data on the same domain as our dashboard.
…
Projects will automatically be redirected, and this redirect will remain in place for the foreseeable future. Still, you should plan on updating links to your documentation after the new domain goes live.
…
dcarley added a commit to alphagov/paas-team-manual that referenced this issue Apr 28, 2016
@dcarley
Update readthedocs domain
e4d16a0 
I received an email from Read The Docs that says:

> Starting today, Read the Docs will start hosting projects from subdomains
> on the domain readthedocs.io, instead of on readthedocs.org. This change
> addresses some security concerns around site cookies while hosting user
> generated data on the same domain as our dashboard.

They do serve a redirect from the old to new domain, however they will
deprecate it one day and per readthedocs/readthedocs.org#2175 the protocol isn't
preserved. So it's best that we update our links.

When merged I'll also update the URL in the repo description.
@dcarley dcarley mentioned this issue Apr 28, 2016
Merged
@ericholscher
Copy link
Member

Hmm -- I tested this and it was working previously. I'll look into it.

@ericholscher
Copy link
Member

Should be fixed now -- sorry about that!

@dcarley
Copy link
Author

dcarley commented Apr 29, 2016

Thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ericholscher @dcarley