The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇸🇮Slovenia KlemenDEV
Not sure if any 4xx logs should log as error? If not on4xx would work better. HttpExceptionSubscriberBase may be inherited to make it simpler?
I agree with that
- Assigned to bhanu951
- @bhanu951 opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 8:51am 1 February 2023 - Status changed to Needs work
almost 2 years ago 5:12pm 16 February 2023 - 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
As a bug this will need test case
- 🇺🇸United States bob.hinrichs
We are going to use this patch in order to squelch the flood of spurious errors in the logs. Because thsi issue creates noisy log files, it is a pretty important core bug, IMO. Thank you for this nice little patch.
- last update
over 1 year ago 30,416 pass, 1 fail - 🇧🇬Bulgaria pfrenssen Sofia
In addition to needing tests, comment #13 (and #18) still need to be addressed.