Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle route expansion on client #2430

Merged

Conversation

lixmal
Copy link
Contributor

@lixmal lixmal commented Aug 14, 2024

Describe your changes

  • Handle mgmt server handing out sources instead of single source
  • Adds ipset to iptables and sets for nftables when len(sources) > 1
  • Make refcounter more generic for that purpose

Issue ticket number and link

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)
  • Extended the README / documentation, if necessary

@lixmal lixmal changed the title Handle route expansion on client client Handle route expansion on client Aug 14, 2024
@lixmal lixmal force-pushed the handle-route-expansion-client branch from 79d776a to b4eedd4 Compare August 14, 2024 13:42
Copy link

Copy link

@pappz
Copy link
Contributor

pappz commented Sep 26, 2024

After I stop the agent the table ip netbird still exists. I tested with nft.

@lixmal lixmal merged commit 0e1cd9b into feature/network-route-access-control Sep 27, 2024
21 checks passed
@lixmal lixmal deleted the handle-route-expansion-client branch September 27, 2024 13:26
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