6432 - add release note about basic OIDC support#6442
Conversation
| While with our OAuth 2.0 login options you had to implement support for every provider | ||
| by pull request, OpenID Connect provides a standardized way for authentication, user | ||
| details and more. You are able to use any provider just by loading a configuration file, | ||
| without touching the codebase. |
There was a problem hiding this comment.
Does this mean that #4334 could be addressed without a pull request? Previously pull request #4341 was made to add support for a country-specific OAuth provider but it was closed as "we want this to be a plugin outside the core code". My understanding is that @philippconzett and @oodu run a fork of Dataverse because they need support for their country-specific login. It would be great if they could stop running a fork and get the auth support they need with configuration!
There was a problem hiding this comment.
It looks like this will be the case.
https://docs.feide.no/service_providers/integration_guide/oauth_oidc/openidconnect.html
There was a problem hiding this comment.
Thanks, looks like Obi has commented on the issue and can repopen if needed.
djbrooke
left a comment
djbrooke
left a comment
There was a problem hiding this comment.
Thanks @poikilotherm for the addition here. I may reword when I work on the release notes for 4.19, but it's fine to merge this. Thanks again!
| While with our OAuth 2.0 login options you had to implement support for every provider | ||
| by pull request, OpenID Connect provides a standardized way for authentication, user | ||
| details and more. You are able to use any provider just by loading a configuration file, | ||
| without touching the codebase. |
There was a problem hiding this comment.
Thanks, looks like Obi has commented on the issue and can repopen if needed.
5 participants
What this PR does / why we need it:
This is an addendum to #6433 as requested by @pdurbin . Adding a release note as described at http://guides.dataverse.org/en/latest/developers/making-releases.html#write-release-notes
Which issue(s) this PR
closesrelates:Relates to #6432
Special notes for your reviewer:
Please change or use at your own discretion @djbrooke
Suggestions on how to test this:
Not applicable.
Does this PR introduce a user interface change?:
No it doesn't.
Is there a release notes update needed for this change?:
This is a release note update. 😉
Additional documentation:
n/a