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

Parse original resolv.conf #1270

Merged
merged 17 commits into from
Nov 3, 2023
Merged

Parse original resolv.conf #1270

merged 17 commits into from
Nov 3, 2023

Conversation

pappz
Copy link
Contributor

@pappz pappz commented Oct 30, 2023

Handle original search domains in resolv.conf type implementations.

  • parse the original resolv.conf file
  • merge the search domains
  • ignore the domain keyword
  • append any other config lines (sortstlist, options)
  • fix read origin resolv.conf from bkp in resolvconf implementation
  • fix line length validation
  • fix number of search domains validation

Describe your changes

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

@pappz pappz changed the title Process original resolv.conf Parse original resolv.conf Oct 30, 2023
@pappz pappz marked this pull request as ready for review October 30, 2023 18:10
@pascal-fischer pascal-fischer self-requested a review October 31, 2023 15:51
pascal-fischer
pascal-fischer previously approved these changes Nov 1, 2023
@pappz pappz force-pushed the fix/search-domain-in-resolvconf branch from 0457a4b to 3c91812 Compare November 1, 2023 21:19
@pappz pappz force-pushed the fix/search-domain-in-resolvconf branch from 805cda2 to b7962c7 Compare November 1, 2023 23:58
@mlsmaycon mlsmaycon merged commit 9c4bf1e into main Nov 3, 2023
@mlsmaycon mlsmaycon deleted the fix/search-domain-in-resolvconf branch November 3, 2023 12:05
pulsastrix pushed a commit to pulsastrix/netbird that referenced this pull request Dec 24, 2023
Handle original search domains in resolv.conf type implementations.

- parse the original resolv.conf file
- merge the search domains
- ignore the domain keyword
- append any other config lines (sortstlist, options)
- fix read origin resolv.conf from bkp in resolvconf implementation
- fix line length validation
- fix number of search domains validation
Foosec pushed a commit to Foosec/netbird that referenced this pull request May 8, 2024
Handle original search domains in resolv.conf type implementations.

- parse the original resolv.conf file
- merge the search domains
- ignore the domain keyword
- append any other config lines (sortstlist, options)
- fix read origin resolv.conf from bkp in resolvconf implementation
- fix line length validation
- fix number of search domains validation
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