Skip to content

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

Closed

Conversation

javiereguiluz
Copy link
Member

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.

@wouterj
Copy link
Member

wouterj commented Nov 19, 2014

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
Copy link
Member

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?

Copy link
Member Author

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.

Copy link
Member

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.

Copy link
Member

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
Copy link
Member

xabbuh commented Nov 19, 2014

@javiereguiluz Is it at least possible to provide a signed checksum of the file's contents?

@wouterj
Copy link
Member

wouterj commented Nov 19, 2014

Merged :)

wouterj added a commit that referenced this pull request Nov 19, 2014
…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
@wouterj wouterj closed this Nov 19, 2014
@xabbuh
Copy link
Member

xabbuh commented Nov 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).

@javiereguiluz javiereguiluz deleted the update_installer_instructions branch February 18, 2015 16:28
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.

4 participants