Problem/Motivation
QuizAccessDeniedSubscriber logs every 404 exception. This can quickly clutter the logs, especially for larger sites. Aside from that the resolution of https://www.drupal.org/node/3498572 is suboptimal as it bypasses the core CustomPageExceptionHtmlSubscriber in favour of an own implementation which would need to be kept updated.
Steps to reproduce
Install the quiz module & make 404 requests.
Proposed resolution
Instead of simply removing the logging line, I think it'd be better to take the approach of webform. By implementing the onException method the other 4xx implementations remain defaulted. That way the core CustomPageExceptionHtmlSubscriber::on404 will be executed again.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3565063-1.patch | 2.11 KB | duivw |
Issue fork quiz-3565063
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
duivw commentedAttached patch of the MR
Comment #8
smustgrave commentedThanks have included in relevant branches!