[PR #985] [MERGED] Refactor: Configurable supported scopes #14657

Closed
opened 2026-08-05 03:06:04 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/985
Author: @mlsmaycon
Created: 6/25/2023
Status: Merged
Merged: 6/25/2023
Merged by: @mlsmaycon

Base: mainHead: configurable-scopes


📝 Commits (4)

  • 1cda07c Refactor: Configurable supported scopes
  • 9117be6 correct var
  • 97dda51 correct var
  • 7d45e7d skip fetching scopes from openid-configuration

📊 Changes

4 files changed (+6 additions, -8 deletions)

View changed files

📝 .github/workflows/test-docker-compose-linux.yml (+1 -0)
📝 infrastructure_files/configure.sh (+0 -7)
📝 infrastructure_files/setup.env.example (+4 -1)
📝 infrastructure_files/tests/setup.env (+1 -0)

📄 Description

Previously, supported scopes were hardcoded and limited to Auth0 and Keycloak. This update removes the default set of values, providing flexibility. The value to be set for each Identity Provider (IDP) is specified in their respective documentation.

Describe your changes

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

🔄 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/netbirdio/netbird/pull/985 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 6/25/2023 **Status:** ✅ Merged **Merged:** 6/25/2023 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `configurable-scopes` --- ### 📝 Commits (4) - [`1cda07c`](https://github.com/netbirdio/netbird/commit/1cda07cc6917171995b19dde5cab364169b80541) Refactor: Configurable supported scopes - [`9117be6`](https://github.com/netbirdio/netbird/commit/9117be6ddb11520f5e3a26483f8f5493d5ec154e) correct var - [`97dda51`](https://github.com/netbirdio/netbird/commit/97dda51432ef8c95627db7ea15b0475385bb8fb4) correct var - [`7d45e7d`](https://github.com/netbirdio/netbird/commit/7d45e7d4ba2854fecfc121d23416f4125ab83fee) skip fetching scopes from openid-configuration ### 📊 Changes **4 files changed** (+6 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-docker-compose-linux.yml` (+1 -0) 📝 `infrastructure_files/configure.sh` (+0 -7) 📝 `infrastructure_files/setup.env.example` (+4 -1) 📝 `infrastructure_files/tests/setup.env` (+1 -0) </details> ### 📄 Description Previously, supported scopes were hardcoded and limited to Auth0 and Keycloak. This update removes the default set of values, providing flexibility. The value to be set for each Identity Provider (IDP) is specified in their respective documentation. ## Describe your changes ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [x] It is a refactor - [ ] Created tests that fail without the change (if possible) - [x] Extended the README / documentation, if necessary --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 03:06:04 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#14657