Catch exceptions during authentication

Created on 26 March 2020, over 4 years ago
Updated 29 January 2024, 10 months ago

When using a filter on domain League\OAuth2\Client\Exception\HostedDomainException::notMatchingDomain() is raised without being catched .
I've added a little patch as workaround just in case someone is having the same issue.

πŸ’¬ Support request
Status

Needs review

Version

4.0

Component

Code

Created by

πŸ‡²πŸ‡¦Morocco hamzahasbi

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡ΈUnited States bob.hinrichs

    We are experiencing quite a few errors where google comes back with this to our callback:
    /user/login/google/callback?error=access_denied&state=54e334cd95aa9e74c...

    This creates an error in dblog like:
    BadMethodCallException: Required parameter not passed: "code" in League\OAuth2\Client\Grant\AbstractGrant->checkRequiredParameter() (line 35 of /code/vendor/league/oauth2-client/src/Tool/RequiredParameterTrait.php).

    Is this the same issue? It appears perhaps so, and the callback ought to catch this.

    There is an additional question here about whether this constitutes an actual error to be logged. It could be argued, when users are generally able to log in, but some do not have access via google, that is normal functioning.

Production build 0.71.5 2024