[PR #253] [MERGED] Fix IDP Manager config structs with correct tags #12372

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/253
Author: @mlsmaycon
Created: 3/9/2022
Status: Merged
Merged: 3/9/2022
Merged by: @mlsmaycon

Base: mainHead: fix-idpmanager-config


📝 Commits (5)

  • 449f2e3 Fix IDP Manager config structs with correct tags
  • c783181 Added debug logs for updating meta and jwt token
  • 214b0cf Add configuration checks for auth0 manager
  • 786e6dd Update test data example
  • c40fcb7 Rename Client ID field name following standards

📊 Changes

4 files changed (+96 additions, -10 deletions)

View changed files

📝 management/server/idp/auth0.go (+30 -6)
📝 management/server/idp/auth0_test.go (+62 -0)
📝 management/server/idp/idp.go (+1 -1)
📝 management/server/testdata/management.json (+3 -3)

📄 Description

When loading the configuration from file we will use the Auth0ClientConfig and when sending the post to retrieve a token
we use the auth0JWTRequest with proper tags.

Also, removed the idle timeout as it was closing all idle connections


🔄 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/253 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 3/9/2022 **Status:** ✅ Merged **Merged:** 3/9/2022 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `fix-idpmanager-config` --- ### 📝 Commits (5) - [`449f2e3`](https://github.com/netbirdio/netbird/commit/449f2e38f131b197eedab6c47b9c687b981cf075) Fix IDP Manager config structs with correct tags - [`c783181`](https://github.com/netbirdio/netbird/commit/c7831815b42a81ae2f5e607ba066b5392e4f0567) Added debug logs for updating meta and jwt token - [`214b0cf`](https://github.com/netbirdio/netbird/commit/214b0cf42d4b56c1c459b7151957a85bd23a3857) Add configuration checks for auth0 manager - [`786e6dd`](https://github.com/netbirdio/netbird/commit/786e6dd44e7da4ee467e05a45867a651015620d2) Update test data example - [`c40fcb7`](https://github.com/netbirdio/netbird/commit/c40fcb705b6745524525ac3338aad31a36e512ea) Rename Client ID field name following standards ### 📊 Changes **4 files changed** (+96 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `management/server/idp/auth0.go` (+30 -6) 📝 `management/server/idp/auth0_test.go` (+62 -0) 📝 `management/server/idp/idp.go` (+1 -1) 📝 `management/server/testdata/management.json` (+3 -3) </details> ### 📄 Description When loading the configuration from file we will use the Auth0ClientConfig and when sending the post to retrieve a token we use the auth0JWTRequest with proper tags. Also, removed the idle timeout as it was closing all idle connections --- <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 02:05:46 -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#12372