Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Updated the Symfony Installer installation instructions#4494
Uh oh!
There was an error while loading.Please reload this page.
Conversation
wouterj commentedNov 19, 2014
I'll merge this once my working directory is clean. I'm currently working on a PR that is comming soon today |
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.
xabbuh commentedNov 19, 2014
@javiereguiluz Is it at least possible to provide a signed checksum of the file's contents? |
wouterj commentedNov 19, 2014
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 onhttp://symfony.com/doc/current/quick_tour/the_big_picture.html are broken.Commits-------f374bf4 Updated the Symfony Installer installation instructions
xabbuh commentedNov 20, 2014
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 forpushing you, but this is aultra-high priority PR because the current Symfony installation instructions displayed onhttp://symfony.com/doc/current/quick_tour/the_big_picture.html are broken.