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

Add TLS to HTTP links #163

Merged
merged 2 commits into from Mar 4, 2018
Merged

Add TLS to HTTP links #163

merged 2 commits into from Mar 4, 2018

Conversation

sigv
Copy link
Contributor

@sigv sigv commented Feb 25, 2018

Please use HTTPS wherever it is supported. It is just good practise, you know.

Please use HTTPS wherever it is supported.
It is just good practise, you know.

This follows up commit 98924bc.
@Aktanusa
Copy link
Collaborator

I agree this is a good idea, but why the two pull requests? I see one is to master and other to dev, but it should be only to dev first, as that's the order to go to master.

@sigv
Copy link
Contributor Author

sigv commented Feb 25, 2018

Thought the other one goes to master as it is a README change. Can pick that into this PR in a moment.

@Aktanusa
Copy link
Collaborator

Hmm, it is true that it's just a README change, so nothing to test, but then the gh-pages branch needs it too, lol.

HTTPS is good to push for wherever possible,
even if the Cookie Clicker site itself does not support it now.

(cherry picked from commit 98924bc)
@sigv sigv mentioned this pull request Feb 25, 2018
@sigv
Copy link
Contributor Author

sigv commented Feb 25, 2018

Let's just go the regular route then.

@Aktanusa
Copy link
Collaborator

Well I am going to be doing an update to master and gh-pages soon anyway, due to the new version of CC, lol.

@Aktanusa Aktanusa merged commit 4ad6cc9 into CookieMonsterTeam:dev Mar 4, 2018
@sigv sigv deleted the https-load branch March 4, 2018 20:12
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.

None yet

2 participants