8000 Flex node for Apache · Pull Request #8806 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Flex node for Apache #8806

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
wants to merge 5 commits into from
Closed

Flex node for Apache #8806

wants to merge 5 commits into from

Conversation

ghost
Copy link
@ghost ghost commented Dec 1, 2017

This is related to #8724.

@ghost
Copy link
Author
ghost commented Dec 1, 2017

I'm against the following question, but it might be useful to add a tip adding the file yourself with the content of .htaccess without using a recipe?

Copy link
Member
@weaverryan weaverryan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Some ideas for changes:


$ cd my-project/
$ composer require symfony/apache-pack

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One way or another, the user will need the rewrite rules. They can either put them in public/.htaccess or put them in their virtual host. So:

  • Let's rename this section to something like "Adding Rewrite Rules"
  • Change the text from "recommended way" to "the easiest way".
  • We should also say (in the text) what this does: it will install a public/.htaccess file
  • No need to show the cd my-project

Then, after the command, mention that you can get a performance boost by putting the rewrite rules in your VirtualHost and changing AllowOverride All to AllowOverride None

@ghost
Copy link
Author
ghost commented Dec 2, 2017

Thanks @weaverryan for the feedback (I am sorry that sometimes I send bad English phrases but fortunately there are reviews! 🤓).

Changes are in d71d22a. Ready for re-review.


.. tip::

A performance improvement can be achieved by putting the rewrite rules in your
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion:
"by moving the rewrite rules from the .htaccess file to the VirtualHost block of your Apache configuration and disallowing .htaccess overrides by changing AllowOverride All to AllowOverride None (where though?).


$ composer require symfony/apache-pack

This recipe will install the ``.htaccess`` file in the ``public/`` directory.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"the .htaccess file" => "a .htaccess files containing the rewrite rules"?

@ghost
Copy link
Author
ghost commented Dec 2, 2017

Thank you @greg0ire for the suggestions. They look more natural now. Fixed in 07abdb8.

@weaverryan
Copy link
Member

Thank you Ricardo for taking this on!

weaverryan added a commit that referenced this pull request Dec 5, 2017
This PR was submitted for the master branch but it was merged into the 4.0 branch instead (closes #8806).

Discussion
----------

Flex node for Apache

This is related to #8724.

Commits
-------

23c49fa Minor changes
5f9b813 Update article block from pull request feedback.
0bee9a9 Forgot a apostrophe.
4f20edf Improvements on the apache section.
17748d3 Added the Flex node for Apache.
@weaverryan weaverryan closed this Dec 5, 2017
weaverryan added a commit that referenced this pull request Dec 5, 2017
weaverryan added a commit that referenced this pull request Dec 5, 2017
* 4.0:
  doc(http_foundation.rst): Fix code error which used create() instead of constructor method.
  [#8806] minor reword
  Minor changes
  Update article block from pull request feedback.
  Forgot a apostrophe.
  Improvements on the apache section.
  Added the Flex node for Apache.
  add back the trailing comma
  remove "choices_as_values" mention
  Fixed configuration block rendering in routing docs
  Update entity.rst
  Removing more AppBundle
  Follow the same code style in docblock annotations
@ghost ghost deleted the docchange/flex-apache branch December 5, 2017 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0