Skip to content

Commit c99acc8

Browse files
committed
Merge branch '2.1'
* 2.1: added a warning about a security issue when using the _internal route updated VENDORS for 2.1.5 updated vendors to dev branches updated VENDORS for 2.0.20 added default configuration for trusted proxies Add a note to the 2.0 -> 2.1 upgrade guide about completing all steps before running composer install updated VENDORS for 2.1.4 updated vendors to dev branches updated VENDORS for 2.0.19 Conflicts: app/config/config.yml composer.lock
2 parents 99f74db + 1a13f1a commit c99acc8

File tree

4 files changed

+19
-5
lines changed

4 files changed

+19
-5
lines changed

UPGRADE.md

+2
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,8 @@ Finally, run Composer:
3232

3333
$ composer.phar install
3434

35+
Note: You must complete the upgrade steps below so composer can successfully generate the autoload files.
36+
3537
### `app/autoload.php`
3638

3739
The default `autoload.php` reads as follows (it has been simplified a lot as

app/SymfonyRequirements.php

+11-3
Original file line numberDiff line numberDiff line change
@@ -500,9 +500,9 @@ function_exists('simplexml_import_dom'),
500500
$pcreVersion = defined('PCRE_VERSION') ? (float) PCRE_VERSION : null;
501501

502502
$this->addRequirement(
503-
null !== $pcreVersion && $pcreVersion > 8.0,
504-
sprintf('PCRE extension must be available and at least 8.0 (%s installed)', $pcreVersion ? $pcreVersion : 'not'),
505-
'Upgrade your <strong>PCRE</strong> extension (8.0+).'
503+
null !== $pcreVersion,
504+
'PCRE extension must be available',
505+
'Install the <strong>PCRE</strong> extension (version 8.0+).'
506506
);
507507

508508
/* optional recommendations follow */
@@ -531,6 +531,14 @@ function_exists('simplexml_import_dom'),
531531
'Your project might not work properly due to the PHP bug #61453 ("Cannot dump definitions which have method calls"). Install PHP 5.4.1 or newer.'
532532
);
533533

534+
if (null !== $pcreVersion) {
535+
$this->addRecommendation(
536+
$pcreVersion >= 8.0,
537+
sprintf('PCRE extension should be at least version 8.0 (%s installed)', $pcreVersion),
538+
'<strong>PCRE 8.0+</strong> is preconfigured in PHP since 5.3.2 but you are using an outdated version of it. Symfony probably works anyway but it is recommended to upgrade your PCRE extension.'
539+
);
540+
}
541+
534542
$this->addRecommendation(
535543
class_exists('DomDocument'),
536544
'PHP-XML module should be installed',

app/config/config.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ framework:
1313
csrf_protection: true
1414
validation: { enable_annotations: true }
1515
templating: { engines: ['twig'] } #assets_version: SomeVersionScheme
16-
default_locale: %locale%
17-
trust_proxy_headers: false # Whether or not the Request object should trust proxy headers (X_FORWARDED_FOR/HTTP_CLIENT_IP)
16+
default_locale: "%locale%"
17+
trusted_proxies: ~
1818
session: ~
1919

2020
# Twig Configuration

app/config/routing.yml

+4
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
11
# Internal routing configuration to handle ESI
2+
# ****** WARNING ******
3+
# This route should be enabled with caution as it can open a security vulnerability
4+
# see http://symfony.com/blog/security-release-symfony-2-0-20-and-2-1-5-released for more information
5+
# ****** WARNING ******
26
#_internal:
37
# resource: "@FrameworkBundle/Resources/config/routing/internal.xml"
48
# prefix: /_internal

0 commit comments

Comments
 (0)