set_error_handler works separate from error_reporting. So when wfSuppressWarnings changes the error_reporting, MWExceptionHandler::handleError still gets called all the same.
I'm not sure whether they should completely be suppressed or not in this area. Perhaps have it be configurable, or add a flag of some kind (so that we can e.g. hide them by default in logstash, but still have a query for trending warnings that were originally suppressed)
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T157850 Interacting with Wikimedia logs should be a pleasant experience | |||
| Resolved | Krinkle | T45086 Capture PHP warnings with stacktraces in MediaWiki and save to logstash | |||
| Resolved | Krinkle | T75619 Debug: Log group 'error' for PHP errors should abide wfSuppressWarnings |
Change 177095 had a related patch set uploaded (by Krinkle):
exception: Implement error-json log
Change 177096 had a related patch set uploaded (by Krinkle):
exception: Hide suppressed errors in 'error' log, flag in 'error-json'
Change 177096 merged by jenkins-bot:
exception: Hide suppressed errors in 'error' log, flag in 'error-json'