- Merge request !2Issue #3291299: Log errors caught by HttpController β (Merged) created by bradjones1
- πΊπΈUnited States frob US
Drupal is already logging 404 errors and login request errors so why not log jsonrpc request errors?
- πΊπΈUnited States bradjones1 Digital Nomad Life
@frob - exactly! Any chance you could test & review this?
- π―π΅Japan ptmkenny
Thanks @bradjones1, and sorry for being late getting back to this issue.
I've been using the merge request since it was added in 2022 and it logs errors as I expect on a small site.
I hesitate to mark it as RTBC based on only my own experience, but I can say it is working great for me.
- πΊπΈUnited States bradjones1 Digital Nomad Life
One of the contribution principles is to "be bold" - you're welcome to RTBC.
- Status changed to RTBC
8 months ago 11:19am 14 April 2024 - π―π΅Japan ptmkenny
Ok, sure.
I read through the code and it all looks good to me, plus it is helping me catch lots of errors as I build a decoupled app with React, so I'll mark this RTBC.
-
ptmkenny β
committed 7356452c on 2.x authored by
bradjones1 β
git commit -m 'Issue #3291299 by ptmkenny, bradjones1: Log errors caught...
-
ptmkenny β
committed 7356452c on 2.x authored by
bradjones1 β
- Status changed to Fixed
3 months ago 4:01am 31 August 2024 - π―π΅Japan ptmkenny
Feels great to finally get this in! Thanks for this MR, which has saved me a lot of time by helping me quickly figure out what is going wrong from the logs.
Automatically closed - issue fixed for 2 weeks with no activity.