Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upReplace http:// to https:// #287
Conversation
|
Thanks for this patch. Luckily, I think the PHPeople are now finally ready for such changes because I too think that they are really important for 2019: Not using (and promoting) https is like living in the past... Thumbs up @dusta |
|
With this links should not be problems. I have checked and they have forcing redirect to https. There is more in the code http but on php.net is not supported like for example http://docs.php.net/manual/ so I haven't change that. |
|
Thanks for this @dusta! There are a bunch of other php.net sites that look to support HTTPS, would you mind updating references to these as well?
|
|
svn.php.net is not supported |
What do you mean by this? |
Interesting... Yesterday https did not work. Can you confirm if everything is alright? |
I confirm that everything is alright. |
|
Is there anything left to do for this PR (besides resolving the conflicts) before it can be merged? |
Not that I'm aware of. @dusta Can you resolve the conflicts on top of current master? It would also be nice if you could squash the 17 commits into 1 big commit at the same time. |
|
@salathe I will try it |
# Conflicts: #README.md #bin/createNewsEntry #conferences/index.php #index.php #releases/index.php
|
@salathe Wouldn't it be easier to just |
|
The merge conflicts have been resolved long ago. :) |
|
@cmb69 What exactly is blocking the merge then? :) |
Change of important links