- 🇦🇹Austria mvonfrie
This also effects Webprofiler #64 where the webprofiler toolbar reports data to the backend via
POST /admin/reports/profiler/frontend/{profile}/navigation
. The route only allows HTTP POST but as the admin path check of the "admin language from user preferences" language negotiator can't pass the HTTP method to the AccessAwareRouter it fails with the MethodNotAllowedException, which happens at least once per main request.Patch #74 seems to fix this. I can't 100% confirm yet as I have another issue with webprofiler to solve first.
- last update
over 1 year ago Patch Failed to Apply - 🇳🇱Netherlands spokje
This seems to be the root cause for 💬 ckeditor5 upload image MethodNotAllowedException Closed: duplicate .
Let's first get a test-only should-fail patch on
11.x
based on #74 🐛 AccessAwareRouter does not respect HTTP method Fixed up. - last update
over 1 year ago 29,884 pass, 2 fail - 🇳🇱Netherlands spokje
And here's the full patch, based on #74 🐛 AccessAwareRouter does not respect HTTP method Fixed , taking #70 🐛 AccessAwareRouter does not respect HTTP method Fixed on board.
- last update
over 1 year ago 29,888 pass - Status changed to Needs review
over 1 year ago 11:46am 28 July 2023 - last update
over 1 year ago Patch Failed to Apply - 🇮🇳India sanket.addweb
I tested the patch 2706241-79.patch and identified a syntax error in the catch statement. I made a minor syntax adjustment to resolve the issue.
- 🇦🇺Australia dpi Perth, Australia
@sanket.addweb
There is no syntax problem, so long as you're on PHP8.
Make sure you upload an interdiff, you would have noticed you omitted a significant portion of the previous patch.
The current reviewable patch remains #79.
- Status changed to RTBC
over 1 year ago 4:44pm 9 August 2023 - 🇺🇸United States smustgrave
Updated the issue summary to include the solution from #79.
Majority of the patch is the added tests which #78 show are failing as expected.
Think this is good.
- last update
over 1 year ago 29,960 pass - last update
over 1 year ago 29,960 pass - last update
over 1 year ago 29,960 pass - last update
over 1 year ago 29,956 pass, 1 fail The last submitted patch, 79: 2706241-79.patch, failed testing. View results →
- 🇳🇱Netherlands spokje
Known random test failure ( 🐛 [random test failure] CronRunTest::testAutomatedCron Fixed ), ordered retest.
- last update
over 1 year ago 29,961 pass - last update
over 1 year ago 30,046 pass - last update
over 1 year ago 30,051 pass - last update
over 1 year ago 30,058 pass - last update
about 1 year ago 30,057 pass, 1 fail The last submitted patch, 79: 2706241-79.patch, failed testing. View results →
- last update
about 1 year ago 30,060 pass - last update
about 1 year ago 30,062 pass - last update
about 1 year ago 30,062 pass - last update
about 1 year ago 30,100 pass - last update
about 1 year ago 30,136 pass - last update
about 1 year ago 30,137 pass - last update
about 1 year ago 30,138 pass - last update
about 1 year ago 30,138 pass - last update
about 1 year ago 30,148 pass - last update
about 1 year ago 30,150 pass - last update
about 1 year ago 30,152 pass - last update
about 1 year ago 30,163 pass 9:37 4:45 Running- Status changed to Fixed
about 1 year ago 9:57am 18 September 2023 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked to 10.1.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.