-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Updated the Symfony Installer installation instructions #4494
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
Updated the Symfony Installer installation instructions #4494
Conversation
I'll merge this once my working directory is clean. I'm currently working on a PR that is comming soon today |
@@ -31,7 +31,7 @@ On **Linux** and **Mac OS X** systems, execute the following console commands: | |||
|
|||
.. code-block:: bash | |||
|
|||
$ curl -sS https://symfony.com/installer | php | |||
$ curl -LsS http://symfony.com/installer > symfony.phar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it really needed to switch back to plain HTTP?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately yes, because symfony.com doesn't support SSL yet. We're working on this but it will take some time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm okay, that should indeed be solved as fast as possible. Having to retrieve such an important tool from an HTTP connection is not really trustworthy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 I could see someone "burning" Symfony for this in the Twitter-world. And of course, we just shouldn't put people in this position. The faster the better on the SSL cert.
@javiereguiluz Is it at least possible to provide a signed checksum of the file's contents? |
Merged :) |
…viereguiluz) This PR was squashed before being merged into the 2.3 branch (closes #4494). Discussion ---------- Updated the Symfony Installer installation instructions | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | all | Fixed tickets | - @weaverryan @wouterj @xabbuh I'm truly sorry for *pushing* you, but this is a **ultra-high priority PR** because the current Symfony installation instructions displayed on http://symfony.com/doc/current/quick_tour/the_big_picture.html are broken. Commits ------- f374bf4 Updated the Symfony Installer installation instructions
Ah, I didn't see that there already is an issue for providing signatures. So we can continue there (symfony/symfony-installer#21). |
@weaverryan @wouterj @xabbuh I'm truly sorry for pushing you, but this is a ultra-high priority PR because the current Symfony installation instructions displayed on http://symfony.com/doc/current/quick_tour/the_big_picture.html are broken.