Skip to content

Handle if user attributes are a property of returned user info#56

Open
mwithheld wants to merge 1 commit intodominik-th:4.x-devfrom
mwithheld:patch-2
Open

Handle if user attributes are a property of returned user info#56
mwithheld wants to merge 1 commit intodominik-th:4.x-devfrom
mwithheld:patch-2

Conversation

@mwithheld
Copy link
Copy Markdown

@mwithheld mwithheld commented Jul 8, 2021

Do not fail on implementations that return attributes as a property of the user info. Our CAS-based OIDC service returns the result like this:

result=stdClass Object
(
    [service] => https://mymatomosite.somedomain.com/index.php?module=LoginOIDC&action=callback&provider=oidc
    [attributes] => stdClass Object
        (
            [mail] => [email protected]
...
        )
     [id] => myusername
     [client_id] => myclientid

Do not fail on implementations that return attributes as a property of the user info.
result=stdClass Object
(
    [service] => https://mymatomosite.somedomain.com/index.php?module=LoginOIDC&action=callback&provider=oidc
    [attributes] => stdClass Object
        (
            [mail] => [email protected]
...
@mwithheld mwithheld changed the title Handle if user attributes are a property the returned user info Handle if user attributes are a property of returned user info Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant