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

Linux legacy routing #1774

Merged
merged 7 commits into from
Apr 3, 2024
Merged

Linux legacy routing #1774

merged 7 commits into from
Apr 3, 2024

Conversation

lixmal
Copy link
Contributor

@lixmal lixmal commented Mar 30, 2024

This PR adds classic route handling on Linux if policy based routing is not available

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 force-pushed the linux-legacy-routing branch 2 times, most recently from 191d4ea to 455e6ca Compare April 1, 2024 18:41
@lixmal lixmal marked this pull request as ready for review April 1, 2024 19:27
@lixmal lixmal force-pushed the linux-legacy-routing branch 3 times, most recently from 50172fc to 455e6ca Compare April 1, 2024 20:47
Base automatically changed from feature/exit-nodes-windows to main April 3, 2024 09:11
@lixmal lixmal force-pushed the linux-legacy-routing branch from 312ecfd to 1e32dc1 Compare April 3, 2024 09:17
@lixmal lixmal merged commit bb0d5c5 into main Apr 3, 2024
16 checks passed
@lixmal lixmal deleted the linux-legacy-routing branch April 3, 2024 16:04
Foosec pushed a commit to Foosec/netbird that referenced this pull request May 8, 2024
* Add Linux legacy routing if ip rule functionality is not available

* Ignore exclusion route errors if host has no route

* Exclude iOS from route manager

* Also retrieve IPv6 routes

* Ignore loopback addresses not being in the main table

* Ignore "not supported" errors on cleanup

* Fix regression in ListenUDP not using fwmarks
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