- Merge request !74Issue #3082984: Reduce logging severity/don't log expired tokens/401s β (Open) created by bradjones1
- ππΊHungary segi
I re-rolled my patch if somebody use it. I have check the MR and it looks promising and it covers my little "fix" as well.
- πΊπΈUnited States bradjones1 Digital Nomad Life
Thanks, although to be clear the MR is the code that is under review, the patch is outdated and is not the subject of the issue any longer.
- π²πΊMauritius Arnaud Kali Roches-Brunes, Beau-Bassin
@bradjones1 I don't seem to be able to review/test that patch from #12. I cant apply the patch to 5.2.x either as the code seems to be outdated. I'm seeing the merge request and actually needing it on one of my projects, I'd guess it's about to be merged and released?
- πΊπΈUnited States bradjones1 Digital Nomad Life
I don't seem to be able to review/test that patch from #12. I cant apply the patch to 5.2.x either as the code seems to be outdated.
Yes, I'd consider that patch to be outdated so I can't help you here and not sure why you're trying to apply it.
I'm seeing the merge request and actually needing it on one of my projects, I'd guess it's about to be merged and released?
It will get released when it gets to RTBC, so I welcome the effort to review it. Feel free to apply the diff from the MR to your project and tell me how it goes. The fact it's an MR doesn't mean anything regarding its timeline to release. Patches are going away imminently on drupal.org, anyway.
- π²πΊMauritius Arnaud Kali Roches-Brunes, Beau-Bassin
Thanks for your comment @bradjones1 - Even if I kinda know after I've been around on d.o for such a long time, I'm just never sure about the way to proceed. I'm re-rolling the patch against 5.2.x as I need an online link to be able to test on my platform (which already has an online API from which I need to test that code).
Thanks! I'll keep you informed of my findings. - π±π»Latvia artis.bajars
artis.bajars β made their first commit to this issueβs fork.
- π¨π¦Canada TrevorBradley
Adding a request to also look at this issue for the new 6.x branch.
Would love to have a little more control over the >3,000 daily simple_oauth errors in the logs on our high volume site.
- First commit to issue fork.
- Merge request !153Add rebased version of 3082984-reduce-logging-severitydont β (Open) created by dieterholvoet