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

upgrade to HTTPS #165

Closed
garrett opened this issue May 1, 2018 · 12 comments
Closed

upgrade to HTTPS #165

garrett opened this issue May 1, 2018 · 12 comments
Assignees

Comments

@garrett
Copy link
Member

garrett commented May 1, 2018

GitHub pages now supports HTTPS for custom domains.

We should upgrade to HTTPS and make it the default.

More information at https://blog.github.com/2018-05-01-github-pages-custom-domains-https/

@garrett
Copy link
Member Author

garrett commented May 2, 2018

It looks like we're using A, which requires DNS modification:
https://help.github.com/articles/setting-up-an-apex-domain/#configuring-a-records-with-your-dns-provider

TODO:

  • update DNS with the new IPs (HTTPS should automatically work)
  • remove & re-add domain to trigger new cert
  • "Enforce HTTPS" option in the repo settings (after confirming it works)

@martinpitt
Copy link
Member

martinpitt commented May 9, 2018

I locally verified that the IPs in https://help.github.com/articles/setting-up-an-apex-domain/#configuring-a-records-with-your-dns-provider work in general. Of course there is still no valid LE certificate on it, as creating that depends on the official DNS servers pointing to these IPs. But it confirms that changing DNS now won't break anything.

Right now, the IPs are still old indeed:

❱❱❱ dig A cockpit-project.org
cockpit-project.org.	55	IN	A	192.30.252.154
cockpit-project.org.	55	IN	A	192.30.252.153

@sgallagher : Can you please switch them to the four IPs that are mentioned on the above gh page?

 185.199.108.153
 185.199.109.153
 185.199.110.153
 185.199.111.153

Thanks!

@sgallagher
Copy link

I just pushed out this change. It may take up to a day to propagate.

@garrett
Copy link
Member Author

garrett commented May 9, 2018

I've checked step 1. Let's see how well this works and flip the switch to enforcing in a day (or most likely: next week, due to the holiday weekend).

@sgallagher
Copy link

So, after this propagated, I get a certificate validation failure, which is expected because the SSL cert is issued with only the following SAN values:

X509v3 Subject Alternative Name: 
                DNS:www.github.com, DNS:*.github.com, DNS:github.com, DNS:*.github.io, DNS:github.io, DNS:*.githubusercontent.com, DNS:githubusercontent.com

Connecting via https://cockpit-project.github.io works successfully for that connection, however it automatically redirects the browser to http://cockpit-project.org (non-SSL).

@larskarlitski
Copy link
Contributor

Connecting via https://cockpit-project.github.io works successfully for that connection, however it automatically redirects the browser to http://cockpit-project.org (non-SSL).

We need to remove and re-add the custom domain to trigger a new cert. I'll do that later today to give DNS a bit more time.

@martinpitt
Copy link
Member

We need to remove and re-add the custom domain to trigger a new cert.

I just did that FYI, DNS looks good from here. No immediate effect yet, it might take a while?

@larskarlitski
Copy link
Contributor

It still didn't work even though DNS returns the right IPs now for me as well. I removed and added the custom domain again.

@martinpitt
Copy link
Member

Still no valid certificate, so just temporarily removing and adding the custom domain is clearly not working.

@larskarlitski
Copy link
Contributor

Apparently not. GitHub explicitly mentions this in their docs, though 😞

@garrett
Copy link
Member Author

garrett commented May 14, 2018

I contacted GitHub support a little earlier this morning, and they just fixed it & responded back. So now we have a proper certificate, and can (probably) flip HTTPS on by default!

@garrett
Copy link
Member Author

garrett commented May 14, 2018

I just flipped the check on, so it should be enforcing HTTPS now.

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

No branches or pull requests

4 participants