[PR #824] [MERGED] Enhance OIDC Support for Compatibility with Strict Providers #1114

Closed
opened 2026-02-04 21:51:09 +03:00 by OVERLORD · 0 comments
Owner

📋 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: masterHead: master


📝 Commits (3)

  • a6c8f1b ref: Parametrize OIDC authorization response mode
  • 9aaaca1 feat: Support OIDC signed UserInfo responses
  • 6b3f887 feat: 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:

  • Allow using the default value of response_mode or specifying a custom value without introducing breaking changes.
  • Ensure the client can handle signed user info responses from strict OIDC providers.
  • Expand Supported Signing Algorithms, by supporting more signing algorithms than just 'RS256' for the ID token endpoint.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/plankanban/planka/pull/824 **Author:** [@lebaudantoine](https://github.com/lebaudantoine) **Created:** 7/15/2024 **Status:** ✅ Merged **Merged:** 7/16/2024 **Merged by:** [@meltyshev](https://github.com/meltyshev) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (3) - [`a6c8f1b`](https://github.com/plankanban/planka/commit/a6c8f1bc23ecfff3fc728f515aa551d6f7fec651) ref: Parametrize OIDC authorization response mode - [`9aaaca1`](https://github.com/plankanban/planka/commit/9aaaca1b8d9c34f2587c361abeae7a13e3392331) feat: Support OIDC signed UserInfo responses - [`6b3f887`](https://github.com/plankanban/planka/commit/6b3f8876840988595807191a058ac7b0c431d066) feat: Parametrize OIDC ID token signing algorithm ### 📊 Changes **5 files changed** (+34 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `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) </details> ### 📄 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: - Allow using the default value of `response_mode` or specifying a custom value without introducing breaking changes. - Ensure the client can handle signed user info responses from strict OIDC providers. - Expand Supported Signing Algorithms, by supporting more signing algorithms than just 'RS256' for the ID token endpoint. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-04 21:51:09 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/planka#1114