[PR #5293] [CLOSED] Reverse proxy logs pagination #25508

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

📋 Pull Request Information

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

Base: prototype/reverse-proxyHead: prototype/reverse-proxy-logs-pagination


📝 Commits (7)

  • fb4cc37 add pagination for access logs
  • bf48044 push filter files
  • d069145 add more filters
  • 1ffe8de add general search filter
  • 5bcdf36 fix source_ip
  • 9dba262 add index to access log entries
  • e020950 concat host and path for search and add a status filter

📊 Changes

10 files changed (+575 additions, -29 deletions)

View changed files

📝 management/internals/modules/reverseproxy/accesslogs/accesslogentry.go (+7 -7)
management/internals/modules/reverseproxy/accesslogs/filter.go (+124 -0)
management/internals/modules/reverseproxy/accesslogs/filter_test.go (+161 -0)
📝 management/internals/modules/reverseproxy/accesslogs/interface.go (+1 -1)
📝 management/internals/modules/reverseproxy/accesslogs/manager/api.go (+21 -2)
📝 management/internals/modules/reverseproxy/accesslogs/manager/manager.go (+41 -7)
📝 management/server/store/sql_store.go (+77 -6)
📝 management/server/store/store.go (+1 -1)
📝 shared/management/http/api/openapi.yml (+115 -5)
📝 shared/management/http/api/types.gen.go (+27 -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/__


🔄 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/5293 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 2/11/2026 **Status:** ❌ Closed **Base:** `prototype/reverse-proxy` ← **Head:** `prototype/reverse-proxy-logs-pagination` --- ### 📝 Commits (7) - [`fb4cc37`](https://github.com/netbirdio/netbird/commit/fb4cc37a4a462b39cebdc0949059c86bc66d6b5f) add pagination for access logs - [`bf48044`](https://github.com/netbirdio/netbird/commit/bf48044e5c233b197e30c9c72d20fd33b5a1b84a) push filter files - [`d069145`](https://github.com/netbirdio/netbird/commit/d069145bd1c57df70a8e80123a6af28c637e4dbe) add more filters - [`1ffe8de`](https://github.com/netbirdio/netbird/commit/1ffe8deb104cfc3a734d3e1c33eda32dc0822753) add general search filter - [`5bcdf36`](https://github.com/netbirdio/netbird/commit/5bcdf36377971635c84889a0bcaaf8f5235ad100) fix source_ip - [`9dba262`](https://github.com/netbirdio/netbird/commit/9dba262a20e1cd51c1fa88b4e1da347704151a94) add index to access log entries - [`e020950`](https://github.com/netbirdio/netbird/commit/e020950cfd0f28dd2c30640ad165008e79039c3c) concat host and path for search and add a status filter ### 📊 Changes **10 files changed** (+575 additions, -29 deletions) <details> <summary>View changed files</summary> 📝 `management/internals/modules/reverseproxy/accesslogs/accesslogentry.go` (+7 -7) ➕ `management/internals/modules/reverseproxy/accesslogs/filter.go` (+124 -0) ➕ `management/internals/modules/reverseproxy/accesslogs/filter_test.go` (+161 -0) 📝 `management/internals/modules/reverseproxy/accesslogs/interface.go` (+1 -1) 📝 `management/internals/modules/reverseproxy/accesslogs/manager/api.go` (+21 -2) 📝 `management/internals/modules/reverseproxy/accesslogs/manager/manager.go` (+41 -7) 📝 `management/server/store/sql_store.go` (+77 -6) 📝 `management/server/store/store.go` (+1 -1) 📝 `shared/management/http/api/openapi.yml` (+115 -5) 📝 `shared/management/http/api/types.gen.go` (+27 -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 - [ ] 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/__ --- <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 07:06:05 -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#25508