- Issue created by @z3cka
- Merge request !41adds accessCheck(TRUE) to getQuery() call in UserAuth->authenticate() β (Closed) created by z3cka
- last update
6 months ago 29 pass - Status changed to Needs review
6 months ago 7:04pm 18 June 2024 - πΊπΈ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
6 months ago 10:48pm 18 June 2024 - πΊπΈ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 viasimple_oauth
. So, I guess this is a nothing-burger and the overriddenUserAuth->authenticate()
that is in this branch should maybe be removed; but that is for another issue.