[PR #5690] [MERGED] [misc] Add missing OpenAPI definitions #23654

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/5690
Author: @bcmmbaga
Created: 3/25/2026
Status: Merged
Merged: 3/30/2026
Merged by: @bcmmbaga

Base: mainHead: update-api-spec


📝 Commits (10+)

📊 Changes

9 files changed (+2564 additions, -124 deletions)

View changed files

shared/management/client/rest/azure_idp.go (+112 -0)
shared/management/client/rest/azure_idp_test.go (+252 -0)
📝 shared/management/client/rest/client.go (+12 -0)
shared/management/client/rest/google_idp.go (+112 -0)
shared/management/client/rest/google_idp_test.go (+248 -0)
shared/management/client/rest/okta_scim_idp.go (+112 -0)
shared/management/client/rest/okta_scim_idp_test.go (+246 -0)
📝 shared/management/http/api/openapi.yml (+1214 -119)
📝 shared/management/http/api/types.gen.go (+256 -5)

📄 Description

Describe your changes

Stack

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)

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__

Summary by CodeRabbit

  • New Features

    • Google Workspace, Azure AD, and Okta SCIM integrations: create, list, view, update, delete, manual sync, and view sync logs
    • Okta: regenerate SCIM token; Google/Azure: manual sync returns a sync result
  • Improvements

    • Unified enable/disable and sync-filter options (adds prefix/provider where required)
    • Clearer API grouping and consistent ID typing for integration routes
  • Tests

    • Added integration tests covering all new IDP client operations

🔄 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/5690 **Author:** [@bcmmbaga](https://github.com/bcmmbaga) **Created:** 3/25/2026 **Status:** ✅ Merged **Merged:** 3/30/2026 **Merged by:** [@bcmmbaga](https://github.com/bcmmbaga) **Base:** `main` ← **Head:** `update-api-spec` --- ### 📝 Commits (10+) - [`afe383f`](https://github.com/netbirdio/netbird/commit/afe383fa2df32169bf8af61493665022d4c959ce) add missing edr and idp sync endpoints - [`c935639`](https://github.com/netbirdio/netbird/commit/c935639a962ed6903d7dbe1ba038aeb6f114d9fb) Merge branch 'main' into update-api-spec - [`86f3e9e`](https://github.com/netbirdio/netbird/commit/86f3e9ec7887c0a264b274d36b2b125bc70e1d4b) Refactor - [`fa9c1c3`](https://github.com/netbirdio/netbird/commit/fa9c1c3b247e06af9c9f67d9e2cbfc81f9ab30e1) add rest client - [`b75274f`](https://github.com/netbirdio/netbird/commit/b75274f7e7cef397dc0ed9ca471d496fce1de71c) fix casing - [`50fc472`](https://github.com/netbirdio/netbird/commit/50fc47200a664523d85ea0cfb7fb4c75ccdecb0e) refactor - [`d8ca49b`](https://github.com/netbirdio/netbird/commit/d8ca49b6deb8e823ee3c591752565364aff6171d) fix tags - [`7512588`](https://github.com/netbirdio/netbird/commit/75125883df3ce878391de3beab3b7de915408fa5) add missing types - [`59541b5`](https://github.com/netbirdio/netbird/commit/59541b5c20598de3fe1a3c4614c9c723f3463c8c) Merge branch 'main' into update-api-spec - [`7d950ee`](https://github.com/netbirdio/netbird/commit/7d950eefc50c37858e26a458eeef1dd1d6d7bc59) Merge branch 'main' into update-api-spec ### 📊 Changes **9 files changed** (+2564 additions, -124 deletions) <details> <summary>View changed files</summary> ➕ `shared/management/client/rest/azure_idp.go` (+112 -0) ➕ `shared/management/client/rest/azure_idp_test.go` (+252 -0) 📝 `shared/management/client/rest/client.go` (+12 -0) ➕ `shared/management/client/rest/google_idp.go` (+112 -0) ➕ `shared/management/client/rest/google_idp_test.go` (+248 -0) ➕ `shared/management/client/rest/okta_scim_idp.go` (+112 -0) ➕ `shared/management/client/rest/okta_scim_idp_test.go` (+246 -0) 📝 `shared/management/http/api/openapi.yml` (+1214 -119) 📝 `shared/management/http/api/types.gen.go` (+256 -5) </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ## Stack <!-- branch-stack --> ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [x] It is a refactor - [ ] Created tests that fail without the change (if possible) > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Google Workspace, Azure AD, and Okta SCIM integrations: create, list, view, update, delete, manual sync, and view sync logs * Okta: regenerate SCIM token; Google/Azure: manual sync returns a sync result * **Improvements** * Unified enable/disable and sync-filter options (adds prefix/provider where required) * Clearer API grouping and consistent ID typing for integration routes * **Tests** * Added integration tests covering all new IDP client operations <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <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 06:07:33 -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#23654