Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revert "idp/proxy: Match users by ID instead of name by default" #6420

Merged
merged 1 commit into from
Jun 1, 2023
Merged

Revert "idp/proxy: Match users by ID instead of name by default" #6420

merged 1 commit into from
Jun 1, 2023

Commits on Jun 1, 2023

  1. Revert "idp/proxy: Match users by ID instead of name by default" 

    This reverts commit 52951b4.

The change broke authentication for at least the desktop client when
using the builtin idp. There seem to be issues in the IDP (lico) which
result in the implicit scoped not being added correctly in some case.
When that scope is missing the `lg.uuid` claim will not be present in
the userinfo and we can correctly match users by id.

This reverts back to the old behaviour of matching users by name. Which
also brings some aspects of owncloud#904

Fixes owncloud#6415
    @rhafer
    rhafer committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    6f90578 View commit details
    Browse the repository at this point in the history