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

More https changes #3297

Merged
merged 4 commits into from
Jun 6, 2019
Merged

More https changes #3297

merged 4 commits into from
Jun 6, 2019

Conversation

FichteFoll
Copy link
Contributor

Continuation of #3296. See commit messages for details.

This time I also changed/added a couple references in code.

Regarding beets.io, you'll probably need to update your gh-pages settings to remove the custom domain name and then set it again so that Github will create a proper certificate for your domain. They support this for about a year now.

I'm not sure where these are used, but the website supports https and
the API url already uses https, so this should be a safe call and not
require a util.SNI_SUPPORTED check.
Should really be all now (pending the next commit).
I'm unsure regarding the pygst tutorial, so I just added another URL
of the best resource I could find with a quick web search.
@FichteFoll FichteFoll changed the title Pr/more https More https changes Jun 5, 2019
@sampsyo
Copy link
Member

sampsyo commented Jun 6, 2019

Cool; thanks! Thanks especially much for tracking down the new URLs for source repositories that have moved around.

I've done the thing where I remove the custom domain from the Pages settings and re-add it. (I've also updated the DNS entries for beets.io to point to the new CDN servers.) The certificate still doesn't seem to be valid, but I'd be willing to believe this could take a while to set up, so let's see what happens.

@sampsyo sampsyo merged commit c04462d into beetbox:master Jun 6, 2019
@FichteFoll
Copy link
Contributor Author

Yes, it may take little while to be updated. I suppose if it doesn't work within a day or so, you could contact support.

@FichteFoll
Copy link
Contributor Author

Once that works, I can update all the references to http://beets.io as well.

@FichteFoll FichteFoll deleted the pr/more-https branch June 6, 2019 13:26
@FichteFoll
Copy link
Contributor Author

Actually, it works already for me, so 🎉

@sampsyo
Copy link
Member

sampsyo commented Jun 6, 2019

Oh yeah! It just started working for me too.

FichteFoll added a commit to FichteForks/beets that referenced this pull request Jun 6, 2019
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.

2 participants