{"push_id":852840996,"size":334,"distinct_size":16,"ref":"refs/heads/master-3","head":"b4e83a7d7d3989a77fe4b4d804f406ac30799624","before":"1319fdf8454ceda845053e3a29ec528ff0e424fc","commits":[{"sha":"de41002f28436dcc8b555e9ae48032bf848d19c8","author":{"email":"0712a68aba493ca0413b8a0d0238658b52132f56@seld.be","name":"Jordi Boggiano"},"message":"Fix potential access to undefined index","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/de41002f28436dcc8b555e9ae48032bf848d19c8"},{"sha":"378db759e0bf2fcbcfe93b139641ae17da444799","author":{"email":"ff019a5748a52b5641624af88a54a2f0e46a9fb5@pcservice.co.za","name":"Pierre du Plessis"},"message":"Added more tests for PropertyAccess","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/378db759e0bf2fcbcfe93b139641ae17da444799"},{"sha":"d040be773c236f650fe43539f80b753a53ccde74","author":{"email":"aa743a0aaec8f7d7a1f01442503957f4d7a2d634@nnucomputerwhiz.com","name":"David Stone"},"message":"[Yaml] Allow tabs before comments at the end of a line\n\nIn Yaml 1.2 spec white space is space or tab","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/d040be773c236f650fe43539f80b753a53ccde74"},{"sha":"bd9997e5e7423ebfeb447fd3634cc4b908d57f77","author":{"email":"d4f8dcd89cfe40276c0fbf336903f4a05ab082c9@gmail.com","name":"Fabien Potencier"},"message":"bug #15793 [Yaml] Allow tabs before comments at the end of a line (superdav42)\n\nThis PR was merged into the 2.3 branch.\n\nDiscussion\n----------\n\n[Yaml] Allow tabs before comments at the end of a line\n\n| Q             | A\n| ------------- | ---\n| Bug fix?      | yes\n| New feature?  | no\n| BC breaks?    | no\n| Deprecations? | no\n| Tests pass?   | yes\n| Fixed tickets |\n| License       | MIT\n\nIf a yml file has a tab character before a line ending comment the comment will be included in the parsed value. Yaml spec allows tab or space as whitespace characters so we need to check for tab as well. See included test.\nRecently caused an odd and hard to find bug in our project.\n\nSee spec:\nhttp://www.yaml.org/spec/1.2/spec.html#s-b-comment\nhttp://www.yaml.org/spec/1.2/spec.html#s-separate-in-line\nhttp://www.yaml.org/spec/1.2/spec.html#s-white\n\nThis is a new PR replacing https://github.com/symfony/symfony/pull/15747\n\n@fabpot\n\nCommits\n-------\n\nd040be7 [Yaml] Allow tabs before comments at the end of a line","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/bd9997e5e7423ebfeb447fd3634cc4b908d57f77"},{"sha":"2da72cbbe814d2634bb938339f03167c57eb6de3","author":{"email":"8d3fadd4fad95f65a95a07af54a8b1c515d4d5e3@gmail.com","name":"Nicolas Grekas"},"message":"[Translation] Fix tests compat with 2.8","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/2da72cbbe814d2634bb938339f03167c57eb6de3"},{"sha":"dc4a7dfc686e507ed7821d72e64f0692c4998d58","author":{"email":"8d3fadd4fad95f65a95a07af54a8b1c515d4d5e3@gmail.com","name":"Nicolas Grekas"},"message":"minor #16162 [Translation] Fix tests compat with 2.8 (nicolas-grekas)\n\nThis PR was merged into the 3.0-dev branch.\n\nDiscussion\n----------\n\n[Translation] Fix tests compat with 2.8\n\n| Q             | A\n| ------------- | ---\n| Bug fix?      | no\n| New feature?  | no\n| BC breaks?    | no\n| Deprecations? | no\n| Tests pass?   | yes\n| Fixed tickets | -\n| License       | MIT\n| Doc PR        | -\n\nCommits\n-------\n\n2da72cb [Translation] Fix tests compat with 2.8","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/dc4a7dfc686e507ed7821d72e64f0692c4998d58"},{"sha":"a954fea00efdd18dcec543a92c9ca2c351832ca3","author":{"email":"d4f8dcd89cfe40276c0fbf336903f4a05ab082c9@gmail.com","name":"Fabien Potencier"},"message":"minor #16155 Added more tests for PropertyAccess (pierredup)\n\nThis PR was merged into the 2.7 branch.\n\nDiscussion\n----------\n\nAdded more tests for PropertyAccess\n\n| Q             | A\n| ------------- | ---\n| Bug fix?      | no\n| New feature?  | no\n| BC breaks?    | no\n| Deprecations? | no\n| Tests pass?   | yes\n| Fixed tickets |\n| License       | MIT\n| Doc PR        |\n\nThis is a follow up for [16090#issuecomment-145183635](https://github.com/symfony/symfony/pull/16090#issuecomment-145183635)\n\nCommits\n-------\n\n378db75 Added more tests for PropertyAccess","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/a954fea00efdd18dcec543a92c9ca2c351832ca3"},{"sha":"96a4071891d5fef632f90fb35304b062b8469edf","author":{"email":"a51dda7c7ff50b61eaea0444371f4a6a9301e501@johnkary.net","name":"John Kary"},"message":"[Console] Add additional ways to detect OS400 platform","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/96a4071891d5fef632f90fb35304b062b8469edf"},{"sha":"2406cc7b4337d2810ee653b862ff0a0b4493e5de","author":{"email":"d4f8dcd89cfe40276c0fbf336903f4a05ab082c9@gmail.com","name":"Fabien Potencier"},"message":"bug #16095 [Console] Add additional ways to detect OS400 platform (johnkary)\n\nThis PR was squashed before being merged into the 2.3 branch (closes #16095).\n\nDiscussion\n----------\n\n[Console] Add additional ways to detect OS400 platform\n\n| Q             | A\n| ------------- | ---\n| Bug fix?      | yes\n| New feature?  | no\n| BC breaks?    | no\n| Deprecations? | no\n| Tests pass?   | yes\n| Fixed tickets | #16053\n| License       | MIT\n| Doc PR        | None\n\nThis PR adds support for detecting the OS400 platform when the PHP function `php_uname()` is disabled. OS400 platform detection was added in #15058 to fix character encoding issues present on OS400. See that PR for more info.\n\nThis PR fixes regression introduced in #16053, which did not work on the IBM OS400 server I have access to. The constant `PHP_OS` being checked outputs \"AIX\" on my IBM OS400 server. I can't say for sure if it works on other IBM platforms... but I preserved this check just in case.\n\nUser @eloigranado [commented here](https://github.com/symfony/symfony/pull/15058#issuecomment-130743928) asking if we could switch to using `PHP_OS` constant instead of `php_uname()` because he claims some admins might \"[hide] the exact kernel build from any attacker who discovers a remote PHP code execution vulnerability\". I personally don't think we should accommodate this use case, but I was able to find alternate approaches.\n\n### Why use case insensitive string matching stristr() instead of in_array()?\n\nHere are the various outputs on my OS400 server:\n\n    echo PHP_OS; // \"AIX\"\n    echo getenv('OSTYPE'); // \"os400\"\n    echo php_uname('s'); // \"OS400\"\n\nSo we have various case issues here, and possible blank values on platforms where OSTYPE var doesn't exist or php_uname() is disabled. Concatenating these optional values together delimited by ; then case-insensitive searching the string for \"OS400\" seemed like a fair compromise. I would've probably done `in_array()` if case wasn't an issue.\n\nCommits\n-------\n\n96a4071 [Console] Add additional ways to detect OS400 platform","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/2406cc7b4337d2810ee653b862ff0a0b4493e5de"},{"sha":"97868686a8d31a17e1c01fc006cba75574905f2d","author":{"email":"22b016e7ae7c39dda82e1247c62c83dd74fbada6@gmail.com","name":"Michiel Boeckaert"},"message":"[FrameworkBundle] composer suggest fix\n\n| Q             | A\r\n| ------------- | ---\r\n| Bug fix?      | [yes]\r\n| New feature?  | [yes|no]\r\n| BC breaks?    | [yes|no]\r\n\r\nSince cache is now required, it makes no sense to suggest it","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/97868686a8d31a17e1c01fc006cba75574905f2d"},{"sha":"491c12c510291e7017b1a50d579244eaf10d268b","author":{"email":"d4f8dcd89cfe40276c0fbf336903f4a05ab082c9@gmail.com","name":"Fabien Potencier"},"message":"minor #16165 [FrameworkBundle] composer suggest fix (Miliooo)\n\nThis PR was merged into the 2.8 branch.\n\nDiscussion\n----------\n\n[FrameworkBundle] composer suggest fix\n\n| Q             | A\n| ------------- | ---\n| Bug fix?      | no\n| New feature?  | no\n| BC breaks?    | no\n| Deprecations? | no\n| Tests pass?   | yes\n| Fixed tickets | none\n| License       | MIT\n| Doc PR        | no\nSince cache is now required (See https://github.com/symfony/symfony/pull/16159), it makes no sense to suggest it\n\nCommits\n-------\n\n9786868 [FrameworkBundle] composer suggest fix","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/491c12c510291e7017b1a50d579244eaf10d268b"},{"sha":"a6cb1c0d97b3a6e391377703c5f5438d66b27f20","author":{"email":"8d3fadd4fad95f65a95a07af54a8b1c515d4d5e3@gmail.com","name":"Nicolas Grekas"},"message":"[FrameworkBundle] Replace PhpFileCache by FilesystemCache","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/a6cb1c0d97b3a6e391377703c5f5438d66b27f20"},{"sha":"f52b3a0a836880247863b084f83290f4fe0096e0","author":{"email":"bdfc83b49179f4905b3a5b06f3b74bb5172a9a16@notk.org","name":"Christophe Coevoet"},"message":"Fix the BC layer for the key->secret renaming for remember_me","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/f52b3a0a836880247863b084f83290f4fe0096e0"},{"sha":"864a58faa3f0cacc90ed1d1cd6f96c2af7bc39cf","author":{"email":"d4f8dcd89cfe40276c0fbf336903f4a05ab082c9@gmail.com","name":"Fabien Potencier"},"message":"bug #16173 [FrameworkBundle] Replace PhpFileCache by FilesystemCache (nicolas-grekas)\n\nThis PR was merged into the 2.8 branch.\n\nDiscussion\n----------\n\n[FrameworkBundle] Replace PhpFileCache by FilesystemCache\n\n| Q             | A\n| ------------- | ---\n| Bug fix?      | no\n| New feature?  | no\n| BC breaks?    | no\n| Deprecations? | no\n| Tests pass?   | yes\n| Fixed tickets | #16168\n| License       | MIT\n| Doc PR        | -\n\nCommits\n-------\n\na6cb1c0 [FrameworkBundle] Replace PhpFileCache by FilesystemCache","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/864a58faa3f0cacc90ed1d1cd6f96c2af7bc39cf"},{"sha":"b1bd093f8abc12795fc2798a42679aa2d914a526","author":{"email":"73675debcd8a436be48ec22211dcf44fe0df0a64@indietorrent.org","name":"Ben"},"message":"[Process] Workaround buggy PHP warning","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/b1bd093f8abc12795fc2798a42679aa2d914a526"},{"sha":"520608626618044570e4868430147cf85b01c61a","author":{"email":"50f3f01caa053693ce619d596e14b0ff3901ab49@tubo-world.de","name":"Tobias Schultze"},"message":"bug #16182 [Process] Workaround buggy PHP warning (cbj4074)\n\nThis PR was submitted for the 2.8 branch but it was merged into the 2.3 branch instead (closes #16182).\n\nDiscussion\n----------\n\n[Process] Workaround buggy PHP warning\n\n| Q             | A\n| ------------- | ---\n| Bug fix?      | yes\n| New feature?  | no\n| BC breaks?    | no\n| Deprecations? | no\n| Tests pass?   | yes\n| Fixed tickets | -\n| License       | MIT\n| Doc PR        | -\n\nThis temporary workaround is intended to address an uncaught exception that may occur, until https://bugs.php.net/bug.php?id=69240 is fixed.\n\nObviously the `@` operator is another option, but I'm erring towards the notion that the Symfony authors/maintainers may wish to implement more elegant exception-handling logic than simply using `@`.\n\nCommits\n-------\n\nb1bd093 [Process] Workaround buggy PHP warning","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/520608626618044570e4868430147cf85b01c61a"},{"sha":"814d96165da4fc267a349fdec9ce83149c5c83c7","author":{"email":"8d3fadd4fad95f65a95a07af54a8b1c515d4d5e3@gmail.com","name":"Nicolas Grekas"},"message":"[tests] Use @requires annotation when possible","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/814d96165da4fc267a349fdec9ce83149c5c83c7"},{"sha":"4032c88a216e45913287af3ea59e7aed2ea935b2","author":{"email":"8d3fadd4fad95f65a95a07af54a8b1c515d4d5e3@gmail.com","name":"Nicolas Grekas"},"message":"[ci] Enable collecting and replaying skipped tests","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/4032c88a216e45913287af3ea59e7aed2ea935b2"},{"sha":"dd640f52f76401ef30fd238ace6387b97052ef66","author":{"email":"8d3fadd4fad95f65a95a07af54a8b1c515d4d5e3@gmail.com","name":"Nicolas Grekas"},"message":"[PhpUnitBridge] Add SkippedTestsListener to collect and replay skipped tests","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/dd640f52f76401ef30fd238ace6387b97052ef66"},{"sha":"4061d37e87e1ec63ac34d20a80850a1b7221a691","author":{"email":"8d3fadd4fad95f65a95a07af54a8b1c515d4d5e3@gmail.com","name":"Nicolas Grekas"},"message":"[tests] Use @requires annotation when possible","distinct":false,"url":"https://api.github.com/repos/nicolas-grekas/symfony/commits/4061d37e87e1ec63ac34d20a80850a1b7221a691"}]}