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.

Sign up for GitHub

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

Jump to bottom

Please add an option to disable RDNSS in radvd #4537

Closed
gk998 opened this issue Dec 20, 2020 · 4 comments
Closed

Please add an option to disable RDNSS in radvd #4537

gk998 opened this issue Dec 20, 2020 · 4 comments
Assignees
@marjohn56
Labels
feature Adding new functionality
Milestone
21.7

Comments

@gk998
Copy link

gk998 commented Dec 20, 2020

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md

[x] I have searched the existing issues and I'm convinced that mine is new.
I found #4087, but it has been closed without comment

Is your feature request related to a problem? Please describe.
There are some cases where one does not want to have DNS Servers assigned via SLAAC

Describe the solution you'd like
I know, it is against the RFCs, but radvd itself allows it not to advertise DNS Servers. There can be some conditions where such a behaviour is desired. I just found out, that pfsense already implemented this:
https://redmine.pfsense.org/issues/9302
pfsense/pfsense@e26ad76

It would be great to have this as well in opnsense!
@marjohn56
Copy link
Member

I've quickly done this in commit #4571 , Try it with opnsense-patch d692980 , let us know how you get on. The patch can be reversed by issuing the same command again.

@marjohn56
Copy link
Member

Due to a moment of idiocy I put the config in dhcpv6 and not Router Advertisements. I have not corrected that and the patch is now e3a9fce

@gk998
Copy link
Author

gk998 commented Jan 19, 2021

Sorry for the late reply. Thanks @marjohn56, your patch is looking good here :)

@fichtner fichtner added the feature Adding new functionality label Jan 21, 2021
@fichtner fichtner added this to the 21.7 milestone Jan 21, 2021
fichtner pushed a commit that referenced this issue Jan 21, 2021
@fichtner
Added toggle function and associated code for enable/disable RDNSS in…
8ded045 
… dhcpv6 Moved Configuration to RADVD interface

PR: #4537
@fichtner
Copy link
Member

There was an issue with the code (old var still used in dhcpd.inc) so I manually pushed 8ded045. Thanks ❤️

fichtner pushed a commit that referenced this issue Jan 22, 2021
@fichtner
Added toggle function and associated code for enable/disable RDNSS in…
a137820 
… dhcpv6 Moved Configuration to RADVD interface

PR: #4537
(cherry picked from commit 8ded045)
oshogbo pushed a commit to DynFi/opnsense-core that referenced this issue Mar 3, 2022
@fichtner
Added toggle function and associated code for enable/disable RDNSS in…
85026b1 
… dhcpv6 Moved Configuration to RADVD interface

PR: opnsense/core#4537
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marjohn56 marjohn56

Labels
feature Adding new functionality
Milestone
21.7
Development

No branches or pull requests
3 participants
@fichtner @marjohn56 @gk998