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.

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

ApiTokenAuthentication against Tunnistamo fails with Invalid claim "amr" when drf-oidc-auth version is >= 1.0.0 #73

Open
hughman77 opened this issue Dec 12, 2021 · 2 comments

Comments

@hughman77
Copy link

Tunnistamo seems to provide the amr as string when it should be an array. This has worked with previous versions of drf-oidc-auth, but after updating the version to >= 1.0.0, authentication fails.

The issue should be fixed in Tunnistamo, but another option would be to replace the drf-oidc-auth like suggested in #68

@akikoskinen
Copy link
Contributor

To me getting rid of the drf-oidc-auth dependency sounds like a good option. Contributions welcome.

@nikomakela
Copy link
Contributor

nikomakela commented Jun 6, 2022

danipran added a commit to danipran/django-helusers that referenced this issue Mar 21, 2023
Apparently, this screws things up if drf-oidc-auth is below 1.0.0.
drf-oidc-auth >=1.0.0 is required for Django 4, however.
Bit of a workaround, but should work?
Source: City-of-Helsinki#73 (comment)
danipran added a commit to danipran/django-helusers that referenced this issue Mar 22, 2023
Apparently, this screws things up if drf-oidc-auth is below 1.0.0.
drf-oidc-auth >=1.0.0 is required for Django 4, however.
Bit of a workaround, but should work?
Source: City-of-Helsinki#73 (comment)
danipran added a commit to danipran/django-helusers that referenced this issue Mar 22, 2023
Apparently, this screws things up if drf-oidc-auth is below 1.0.0.
drf-oidc-auth >=1.0.0 is required for Django 4, however.
Bit of a workaround, but should work?
Source: City-of-Helsinki#73 (comment)
danipran added a commit to danipran/django-helusers that referenced this issue Mar 22, 2023
Apparently, this screws things up if drf-oidc-auth is below 1.0.0.
drf-oidc-auth >=1.0.0 is required for Django 4, however.
Bit of a workaround, but should work?
Source: City-of-Helsinki#73 (comment)
danipran added a commit to danipran/django-helusers that referenced this issue Mar 27, 2023
Apparently, this screws things up if drf-oidc-auth is below 1.0.0.
drf-oidc-auth >=1.0.0 is required for Django 4, however.
Bit of a workaround, but should work?
Source: City-of-Helsinki#73 (comment)
danipran added a commit to danipran/django-helusers that referenced this issue Mar 30, 2023
Apparently, this screws things up if drf-oidc-auth is below 1.0.0.
drf-oidc-auth >=1.0.0 is required for Django 4, however.
Bit of a workaround, but should work?
Source: City-of-Helsinki#73 (comment)
danipran added a commit that referenced this issue Mar 30, 2023
Apparently, this screws things up if drf-oidc-auth is below 1.0.0.
drf-oidc-auth >=1.0.0 is required for Django 4, however.
Bit of a workaround, but should work?
Source: #73 (comment)
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

No branches or pull requests

3 participants