Skip to content

feat(route): list with kernel filtering#262

Merged
jsimonetti merged 1 commit intojsimonetti:masterfrom
terinjokes:routeservice-get
May 11, 2025
Merged

feat(route): list with kernel filtering#262
jsimonetti merged 1 commit intojsimonetti:masterfrom
terinjokes:routeservice-get

Conversation

@terinjokes
Copy link
Copy Markdown
Contributor

@terinjokes terinjokes commented May 1, 2025

A new "ListMatch" function has been added to the RouteService to allow for kernel-based filtering of the routing tables. The netlink flags required for getting a route for an IP (eg, rtnl.RouteGetAll) is different than those required while filtering the table.

This changeset removes the superfluous "netlink.DumpFiltered" flag from the Get function of RouteService, as this flag is for kernel responses.

A new "ListMatch" function has been added to the RouteService to allow
for kernel-based filtering of the routing tables. The netlink flags
required for getting a route for an IP (eg, rtnl.RouteGetAll) is
different than those required while filtering the table.

This changeset removes the superfluous "netlink.DumpFiltered" flag from
the Get function of RouteService, as this flag is for kernel responses.

Signed-off-by: Terin Stock <[email protected]>
@terinjokes terinjokes changed the title fix(route): use Dump flag to filter feat(route): list with kernel filtering May 1, 2025
@jsimonetti
Copy link
Copy Markdown
Owner

Sorry for the long delay in review. I was away on holiday.

This lgtm. Nice addition!

@jsimonetti jsimonetti merged commit 5d4f0c4 into jsimonetti:master May 11, 2025
8 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.

2 participants