[PR #5371] [CLOSED] [managment] add flag to disable the old legacy grpc endpoint #22851

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/5371
Author: @pascal-fischer
Created: 2/17/2026
Status: Closed

Base: mainHead: feature/disable-legacy-port


📝 Commits (4)

  • 8eb4cd8 add sorting to access logs
  • 9425c7d extend fields to sort by
  • fb9dabd add url sorting option
  • e28e985 Merge branch 'main' into feature/disable-legacy-port

📊 Changes

4 files changed (+297 additions, -3 deletions)

View changed files

📝 management/internals/modules/reverseproxy/accesslogs/filter.go (+71 -2)
📝 management/internals/modules/reverseproxy/accesslogs/filter_test.go (+199 -0)
📝 management/server/store/sql_store.go (+13 -1)
📝 shared/management/http/api/openapi.yml (+14 -0)

📄 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

Release Notes

  • New Features

    • Access logs now support dynamic sorting by timestamp, URL, host, path, method, status code, duration, source IP, user ID, auth method, or reason with ascending/descending order options. Default sorting is by timestamp in descending order.
  • Documentation

    • Updated API specification to expose new sorting query parameters.
  • Tests

    • Added comprehensive test coverage for sorting functionality and parameter parsing.

🔄 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/5371 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 2/17/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `feature/disable-legacy-port` --- ### 📝 Commits (4) - [`8eb4cd8`](https://github.com/netbirdio/netbird/commit/8eb4cd888474342cddd90642c9fc58be6ee1e57f) add sorting to access logs - [`9425c7d`](https://github.com/netbirdio/netbird/commit/9425c7d77d40af252d3e144e6a89870e6ea8177b) extend fields to sort by - [`fb9dabd`](https://github.com/netbirdio/netbird/commit/fb9dabde8e0b4372694d2d1b0d922ba9f9f4a274) add url sorting option - [`e28e985`](https://github.com/netbirdio/netbird/commit/e28e9854fef34dc04189a53d52b9b58714bb937f) Merge branch 'main' into feature/disable-legacy-port ### 📊 Changes **4 files changed** (+297 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `management/internals/modules/reverseproxy/accesslogs/filter.go` (+71 -2) 📝 `management/internals/modules/reverseproxy/accesslogs/filter_test.go` (+199 -0) 📝 `management/server/store/sql_store.go` (+13 -1) 📝 `shared/management/http/api/openapi.yml` (+14 -0) </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 ## Release Notes * **New Features** * Access logs now support dynamic sorting by timestamp, URL, host, path, method, status code, duration, source IP, user ID, auth method, or reason with ascending/descending order options. Default sorting is by timestamp in descending order. * **Documentation** * Updated API specification to expose new sorting query parameters. * **Tests** * Added comprehensive test coverage for sorting functionality and parameter parsing. <!-- 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:06:28 -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#22851