UserAuth->authenticate() does not check for access when using getQuery()

Issue created by @z3cka
Merge request !41adds accessCheck(TRUE) to getQuery() call in UserAuth->authenticate() → (Closed) created by z3cka
Open in Jenkins → Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.2 & MySQL 8
last update 5 months ago
29 pass
Comment 5 months ago →
🇺🇸United States z3cka
provided a link to MR 41
Status changed to Needs review 5 months ago7:04pm 18 June 2024
Comment 5 months ago →
🇺🇸United States z3cka
MR !41 is ready for review.
Pipeline finished with Success
5 months ago
Total: 372s
#202282
Comment 5 months ago →
🇺🇸United States greggles Denver, Colorado, USA
Thanks for providing the MR and bug report.

Can you provide more info on the step 3 or what the expected vs. unexpected results are in the steps to reproduce?

Thanks!
Status changed to Closed: cannot reproduce 5 months ago10:48pm 18 June 2024
Comment 5 months ago →
🇺🇸United States z3cka
Thank for the speedy response! It turns out that when using this module alone, UserAuth->authenticate() does not seem to get called. So, there is no error.

I ran across this when doing testing with the simple_oauth module. This does get called via simple_oauth. So, I guess this is a nothing-burger and the overridden UserAuth->authenticate() that is in this branch should maybe be removed; but that is for another issue.
Comment 5 months ago →
System Message
z3cka → closed merge request !41

Merge Requests