[PR #1230] [MERGED] Make possible set IdpSignKeyRefreshEnabled from setup.env #16408

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1230
Author: @Fantu
Created: 10/18/2023
Status: Merged
Merged: 10/18/2023
Merged by: @mlsmaycon

Base: mainHead: add-IdpSignKeyRefresh-cfg


📝 Commits (2)

  • dfdfaf2 Make possible set IdpSignKeyRefreshEnabled from setup.env
  • 00f9d40 add template IdpSignKeyRefreshEnabled value test

📊 Changes

5 files changed (+10 additions, -1 deletions)

View changed files

📝 .github/workflows/test-infrastructure-files.yml (+3 -0)
📝 infrastructure_files/base.setup.env (+2 -0)
📝 infrastructure_files/management.json.tmpl (+1 -0)
📝 infrastructure_files/setup.env.example (+2 -0)
📝 infrastructure_files/tests/setup.env (+2 -1)

📄 Description

Describe your changes

IdpSignKeyRefreshEnabled is default to false but with some idps on token expire of logged users netbird always give error and return usable only on server restart so I think is useful make easier/faster set it on server configuration

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/1230 **Author:** [@Fantu](https://github.com/Fantu) **Created:** 10/18/2023 **Status:** ✅ Merged **Merged:** 10/18/2023 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `add-IdpSignKeyRefresh-cfg` --- ### 📝 Commits (2) - [`dfdfaf2`](https://github.com/netbirdio/netbird/commit/dfdfaf2990accfe439b55814eac31d9a04e29488) Make possible set IdpSignKeyRefreshEnabled from setup.env - [`00f9d40`](https://github.com/netbirdio/netbird/commit/00f9d4090436472799a8ab75d7a96e49e6c3186c) add template IdpSignKeyRefreshEnabled value test ### 📊 Changes **5 files changed** (+10 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-infrastructure-files.yml` (+3 -0) 📝 `infrastructure_files/base.setup.env` (+2 -0) 📝 `infrastructure_files/management.json.tmpl` (+1 -0) 📝 `infrastructure_files/setup.env.example` (+2 -0) 📝 `infrastructure_files/tests/setup.env` (+2 -1) </details> ### 📄 Description ## Describe your changes IdpSignKeyRefreshEnabled is default to false but with some idps on token expire of logged users netbird always give error and return usable only on server restart so I think is useful make easier/faster set it on server configuration ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [X] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) - [ ] 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 04:06:06 -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#16408