Skip to content

Conversation

rajsinghtech
Copy link
Contributor

Adds support for the network flow logs API endpoint at /api/v2/tailnet/{tailnet}/logging/network. This endpoint returns network traffic flow data including virtual, subnet, exit, and physical traffic with packet/byte counts.

Updates #41

@rajsinghtech rajsinghtech force-pushed the list-network-logs branch 2 times, most recently from 928e93e to b1aad5d Compare August 14, 2025 23:13
Copy link
Member

@creachadair creachadair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this LGTM, I have a few further mainly documentation-related suggestions. Would love a second pair of eyeballs from @oxtoacart on the library API changes, too.

Copy link
Collaborator

@oxtoacart oxtoacart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally LGTM, just one question about validation.

Copy link
Collaborator

@oxtoacart oxtoacart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for tackling a streaming version of this, I think that'll be valuable.

Copy link
Member

@creachadair creachadair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few leftover suggestions at your option.

Adds support for the network flow logs API endpoint at
/api/v2/tailnet/{tailnet}/logging/network. This endpoint returns
network traffic flow data including virtual, subnet, exit, and
physical traffic with packet/byte counts.

Updates tailscale#41

Signed-off-by: rajsinghtech <[email protected]>
@oxtoacart oxtoacart merged commit 740bf17 into tailscale:main Aug 20, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants