[PR #6730] [MERGED] Remote groups via HTTP Headers #13160

Closed
opened 2025-12-29 22:25:43 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/6730
Author: @MaxRink
Created: 7/8/2021
Status: Merged
Merged: 9/16/2021
Merged by: @jeremystretch

Base: developHead: remote_groups


📝 Commits (1)

  • d5e5cdd Add Remote Group Support to the RemoteUserAuth Backend and Middleware

📊 Changes

5 files changed (+382 additions, -45 deletions)

View changed files

📝 docs/configuration/optional-settings.md (+56 -0)
📝 netbox/netbox/authentication.py (+133 -23)
📝 netbox/netbox/middleware.py (+63 -3)
📝 netbox/netbox/settings.py (+7 -0)
📝 netbox/netbox/tests/test_authentication.py (+123 -19)

📄 Description

Fixes: #5775

Unlike the LDAP Backend, the RemoteUserBackend wasnt able to Sync Group memberships form the auth Provider.
This brings it up to parity, as the new Settings give you the Option to setup an automatic Group Sync (and revocation) on each User login.
This has been tested with oauth2-proxy
The old behaviopur of just Syncing Users and then manually managing Groups should be unaffected.
ToDo:

  • Tests for new Settings

🔄 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/netbox-community/netbox/pull/6730 **Author:** [@MaxRink](https://github.com/MaxRink) **Created:** 7/8/2021 **Status:** ✅ Merged **Merged:** 9/16/2021 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `remote_groups` --- ### 📝 Commits (1) - [`d5e5cdd`](https://github.com/netbox-community/netbox/commit/d5e5cdda23ca184c9e40cdaa0da611269dd72d65) Add Remote Group Support to the RemoteUserAuth Backend and Middleware ### 📊 Changes **5 files changed** (+382 additions, -45 deletions) <details> <summary>View changed files</summary> 📝 `docs/configuration/optional-settings.md` (+56 -0) 📝 `netbox/netbox/authentication.py` (+133 -23) 📝 `netbox/netbox/middleware.py` (+63 -3) 📝 `netbox/netbox/settings.py` (+7 -0) 📝 `netbox/netbox/tests/test_authentication.py` (+123 -19) </details> ### 📄 Description ### Fixes: #5775 Unlike the LDAP Backend, the RemoteUserBackend wasnt able to Sync Group memberships form the auth Provider. This brings it up to parity, as the new Settings give you the Option to setup an automatic Group Sync (and revocation) on each User login. This has been tested with [oauth2-proxy](https://github.com/oauth2-proxy/oauth2-proxy) The old behaviopur of just Syncing Users and then manually managing Groups should be unaffected. ToDo: - [ ] Tests for new Settings --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 22:25:43 +01:00
adam closed this issue 2025-12-29 22:25:44 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13160