Skip to content

Update hyperlinks used in phpinfo function - #3206

Closed
EatonZ wants to merge 2 commits into
php:masterfrom
EatonZ:patch-1
Closed

EatonZ wants to merge 2 commits into
php:masterfrom
EatonZ:patch-1

Conversation

@EatonZ

@EatonZ EatonZ commented Apr 1, 2018

Copy link
Copy Markdown
  1. The www.php.net site is extremely slow on the first request, probably because it's rarely used and needs to "wake up" to serve the first request in a while, compared to just php.net which is always fast. Also changing it to HTTPS.

  2. Now referencing HTTPS Zend website.

1. The www.php.net site is extremely slow on the first request, probably because it's rarely used and needs to "wake up" to serve the first request in a while, compared to just php.net which is always fast. Also changing it to HTTPS.

2. Now referencing HTTPS Zend website.
@EatonZ EatonZ changed the title Update Hyperlinks Update Hyperlinks printed by phpinfo function Apr 1, 2018
@KalleZ

KalleZ commented Apr 1, 2018

Copy link
Copy Markdown
Member

We should use secure.php.net if we are going to use https for php.net because of the certificates

@EatonZ EatonZ changed the title Update Hyperlinks printed by phpinfo function Update hyperlinks used in phpinfo function Apr 1, 2018
@EatonZ

EatonZ commented Apr 1, 2018

Copy link
Copy Markdown
Author

@KalleZ I saw that it redirects to secure.php.net, but I imagine that will change at some point? Just having https://php.net/ will account for the event you fully migrate everything to HTTPS without needing secure dot.

@KalleZ

KalleZ commented Apr 1, 2018

Copy link
Copy Markdown
Member

Hi, I doubt that we will do that in any reasonable future because of the mirror certificates, so far no one have stepped up to solve it once and for all so until then I'm afraid we are stuck with just secure.php.net

1 similar comment
@KalleZ

KalleZ commented Apr 1, 2018

Copy link
Copy Markdown
Member

Hi, I doubt that we will do that in any reasonable future because of the mirror certificates, so far no one have stepped up to solve it once and for all so until then I'm afraid we are stuck with just secure.php.net

@EatonZ

EatonZ commented Apr 1, 2018

Copy link
Copy Markdown
Author

@KalleZ Ok then. I have changed it.

@carusogabriel

carusogabriel commented Apr 1, 2018 •

Copy link
Copy Markdown
Contributor

I have an open PR for this already #3069

@EatonZ

EatonZ commented Apr 1, 2018

Copy link
Copy Markdown
Author

@carusogabriel I just checked yours and it does not include changes this one does. Do you want to add them and then I will just close this?

@carusogabriel

Copy link
Copy Markdown
Contributor

Please close it, I'll update that one 👍

@EatonZ EatonZ closed this Apr 1, 2018
@EatonZ
EatonZ deleted the patch-1 branch April 1, 2018 22:42
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.

3 participants