mirror of
https://github.com/plankanban/planka.git
synced 2026-02-25 03:14:50 +03:00
[PR #824] [MERGED] Enhance OIDC Support for Compatibility with Strict Providers #1114
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/plankanban/planka/pull/824
Author: @lebaudantoine
Created: 7/15/2024
Status: ✅ Merged
Merged: 7/16/2024
Merged by: @meltyshev
Base:
master← Head:master📝 Commits (3)
a6c8f1bref: Parametrize OIDC authorization response mode9aaaca1feat: Support OIDC signed UserInfo responses6b3f887feat: Parametrize OIDC ID token signing algorithm📊 Changes
5 files changed (+34 additions, -6 deletions)
View changed files
📝
server/api/controllers/access-tokens/exchange-using-oidc.js(+7 -0)📝
server/api/controllers/show-config.js(+9 -4)📝
server/api/helpers/users/get-or-create-one-using-oidc.js(+5 -0)📝
server/api/hooks/oidc/index.js(+9 -2)📝
server/config/custom.js(+4 -0)📄 Description
I am working for the French government, which employs a very strict OIDC provider. Government applications often adhere to more stringent implementations of the OIDC specification to enhance security.
To properly connect Planka with their OIDC provider, I needed to add greater flexibility to the current openid-client integration. The initial implementation was missing some important features, such as support for signed user info.
Given the nature of this work, it's crucial to clearly explain every change, I tried to be as explicit as possible in my commits.
Here's a summary:
response_modeor specifying a custom value without introducing breaking changes.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.