[PR #1290] [MERGED] feat(oauth): 1289 - Adding Oauth Callback support for mattermost #858

New Issue

OVERLORD · 2025-10-09T19:04:38+03:00

OVERLORD commented

2025-10-09 19:04:38 +03:00

📋 Pull Request Information

Original PR: https://github.com/plankanban/planka/pull/1290
Author: @luisgreen
Created: 8/7/2025
Status: ✅ Merged
Merged: 8/9/2025
Merged by: @meltyshev

Base: master ← Head: FEAT-1289

📝 Commits (5)

f345dd7 feat(oauth): 1289 - Adding Oauth Callback support
c1f1c7b fix(dockerfile): Fixing a permission issue when building
87ce9f5 chore: Add environment variable to compose files
0254e62 chore: Fix letter casing
43885d6 chore: Revert dockerfile changes

📊 Changes

6 files changed (+31 additions, -15 deletions)

View changed files

📝 docker-compose-dev.yml (+1 -0)
📝 docker-compose.yml (+1 -0)
📝 server/.env.sample (+1 -0)
📝 server/api/helpers/users/get-or-create-one-with-oidc.js (+24 -12)
📝 server/config/custom.js (+1 -0)
📝 server/package-lock.json (+3 -3)

📄 Description

This solves https://github.com/plankanban/planka/issues/1289

This is still not a complete solution but enables the authentication.

You need to set a custom location in a reverse proxy like this so the discovery can work avoiding adding more environment variables.

location = /.well-known/openid-configuration {
    default_type application/json;
    return 200 '{
  "issuer": "https://<YOUR_URL>",
  "authorization_endpoint": "https://<YOUR_URL>/oauth/authorize",
  "token_endpoint": "https://<YOUR_URL>/oauth/access_token",
  "revocation_endpoint": "https://<YOUR_URL>/oauth/revoke",
  "userinfo_endpoint": "https://<YOUR_URL>/api/v4/users/me",
  "scopes_supported": ["openid", "profile", "email"],
  "response_types_supported": ["code"],
  "grant_types_supported": ["authorization_code"],
  "token_endpoint_auth_methods_supported": ["client_secret_post"],
  "claims_supported": ["name", "preferred_username", "email", "profile"]
}';
}

_{🔄 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/1290 **Author:** [@luisgreen](https://github.com/luisgreen) **Created:** 8/7/2025 **Status:** ✅ Merged **Merged:** 8/9/2025 **Merged by:** [@meltyshev](https://github.com/meltyshev) **Base:** `master` ← **Head:** `FEAT-1289` --- ### 📝 Commits (5) - [`f345dd7`](https://github.com/plankanban/planka/commit/f345dd7926e55ecc0ae35c30ec861066fdb1d89a) feat(oauth): 1289 - Adding Oauth Callback support - [`c1f1c7b`](https://github.com/plankanban/planka/commit/c1f1c7b8589308c01f2ac3d31ac579e527a8a2d6) fix(dockerfile): Fixing a permission issue when building - [`87ce9f5`](https://github.com/plankanban/planka/commit/87ce9f5c70024ff1ec86722f5cc205616dc179c7) chore: Add environment variable to compose files - [`0254e62`](https://github.com/plankanban/planka/commit/0254e6236005730416d33d6c50c282471c78cd19) chore: Fix letter casing - [`43885d6`](https://github.com/plankanban/planka/commit/43885d61921922cd0640b052f7c2a4d98864015c) chore: Revert dockerfile changes ### 📊 Changes **6 files changed** (+31 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `docker-compose-dev.yml` (+1 -0) 📝 `docker-compose.yml` (+1 -0) 📝 `server/.env.sample` (+1 -0) 📝 `server/api/helpers/users/get-or-create-one-with-oidc.js` (+24 -12) 📝 `server/config/custom.js` (+1 -0) 📝 `server/package-lock.json` (+3 -3) </details> ### 📄 Description This solves https://github.com/plankanban/planka/issues/1289 This is still not a complete solution but enables the authentication. You need to set a custom location in a reverse proxy like this so the discovery can work avoiding adding more environment variables. ``` location = /.well-known/openid-configuration { default_type application/json; return 200 '{ "issuer": "https://<YOUR_URL>", "authorization_endpoint": "https://<YOUR_URL>/oauth/authorize", "token_endpoint": "https://<YOUR_URL>/oauth/access_token", "revocation_endpoint": "https://<YOUR_URL>/oauth/revoke", "userinfo_endpoint": "https://<YOUR_URL>/api/v4/users/me", "scopes_supported": ["openid", "profile", "email"], "response_types_supported": ["code"], "grant_types_supported": ["authorization_code"], "token_endpoint_auth_methods_supported": ["client_secret_post"], "claims_supported": ["name", "preferred_username", "email", "profile"] }'; } ``` --- <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 2025-10-09 19:04:38 +03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/planka#858