- last update
about 2 years ago Patch Failed to Apply - last update
about 2 years ago 29,378 pass - Status changed to RTBC
about 2 years ago 10:41pm 5 May 2023 - 🇺🇸United States smustgrave
Removing credit for myself as the reroll was minimal.
- last update
about 2 years ago 29,379 pass - 🇬🇧United Kingdom mcdruid 🇬🇧🇪🇺
Going to call this RTBC on the basis that it's a very simple addition to an existing test that's already been reviewed by a number of contributors in the private security issue.
- 🇬🇧United Kingdom mcdruid 🇬🇧🇪🇺
Sorry - didn't notice that @smustgrave had already marked it RTBC.
- last update
almost 2 years ago 29,380 pass - last update
almost 2 years ago 29,383 pass 3:26 59:45 Running- last update
almost 2 years ago 29,388 pass - last update
almost 2 years ago 29,364 pass, 4 fail The last submitted patch, 5: 3358533-5.patch, failed testing. View results →
- last update
almost 2 years ago 29,388 pass - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Failing test is irrelevant. RTBC+1
Depending on who get's merged first, there will probably be a conflict between this issue and 📌 Make phpinfo on the admin status report configurable Fixed as they seem to add the same assertions.
- last update
almost 2 years ago 29,388 pass 48:27 46:24 Running- last update
almost 2 years ago 29,395 pass - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - last update
almost 2 years ago Patch Failed to Apply - Status changed to Closed: duplicate
almost 2 years ago 1:52pm 16 June 2023 - 🇬🇧United Kingdom longwave UK
Duplicate, this was already fixed in 📌 Make phpinfo on the admin status report configurable Fixed
- Status changed to RTBC
almost 2 years ago 2:09pm 16 June 2023 - 🇬🇧United Kingdom mcdruid 🇬🇧🇪🇺
The tests added in the other issue were specifically for when the
sa_core_2023_004_phpinfo_flags
setting is used to override the (new) default.The lines added in this issue test that default and are very slightly different; we don't want the phpinfo page to contain
$_COOKIE
by default. - Status changed to Needs work
almost 2 years ago 3:31pm 16 June 2023 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Needs work then since because of the other issue the patch no longer applies