Skip to content

Commit

Permalink
minor #5536 Removed reference to remove HTTPS off from nginx configur…
Browse files Browse the repository at this point in the history
…ation (wjzijderveld)

This PR was merged into the 2.3 branch.

Discussion
----------

Removed reference to remove HTTPS off from nginx configuration

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | #5517

As explained and confirmed in #5517 the `https off` flag could be causing issues, in my opinion it's better to rely on the defaults distributions give.

An alternative would be to replace the lines with `HTTPS $https;`, [nginx will fill $https depending on current request](http://nginx.org/en/docs/http/ngx_http_core_module.html#var_https). But I think it's less confusing if we just let the lines out of the example.

Commits
-------

3878e14 Removed reference to remove HTTPS off from nginx configuration
  • Loading branch information
xabbuh committed Jul 23, 2015
2 parents ebe8775 + 3878e14 commit 10cb3ed
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions cookbook/configuration/web_server_configuration.rst
Original file line number Diff line number Diff line change
Expand Up @@ -269,15 +269,13 @@ The **minimum configuration** to get your application running under Nginx is:
fastcgi_split_path_info ^(.+\.php)(/.*)$;
include fastcgi_params;
fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
fastcgi_param HTTPS off;
}
# PROD
location ~ ^/app\.php(/|$) {
fastcgi_pass unix:/var/run/php5-fpm.sock;
fastcgi_split_path_info ^(.+\.php)(/.*)$;
include fastcgi_params;
fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
fastcgi_param HTTPS off;
# Prevents URIs that include the front controller. This will 404:
# http://domain.tld/app.php/some-path
# Remove the internal directive to allow URIs like this
Expand Down

0 comments on commit 10cb3ed

Please sign in to comment.