8000 dnsmasq: Conditionally remove values based on DHCPOption type by Monviech · Pull Request #8753 · opnsense/core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

dnsmasq: Conditionally remove values based on DHCPOption type #8753

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

Merged
merged 4 commits into from
Jun 4, 2025

Conversation

Monviech
Copy link
Member
@Monviech Monviech commented Jun 2, 2025

Before serialization, contracting data is removed in the controller, based on the styles defined here:

E.g., "interface" is only valid for type "set", it should not be serialized to the model if the type is "match".

Fixes: #8748

@Monviech Monviech requested a review from AdSchellevis June 2, 2025 08:00
@Monviech Monviech self-assigned this Jun 2, 2025
@Monviech Monviech added the cleanup Low impact changes label Jun 2, 2025
@Monviech Monviech force-pushed the dnsmasq-tag-formatter branch from d491107 to 149e222 Compare June 2, 2025 08:02
@Monviech Monviech force-pushed the dnsmasq-tag-formatter branch from 149e222 to 7bfb6b8 Compare June 3, 2025 14:48
@Monviech Monviech changed the title dnsmasq: Add tag formatters to dhcp options dnsmasq: Conditionally remove values based on DHCPOption type Jun 3, 2025
@Monviech Monviech merged commit 4419b0f into master Jun 4, 2025
@Monviech Monviech deleted the dnsmasq-tag-formatter branch June 4, 2025 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Low impact changes
Development

Successfully merging this pull request may close these issues.

Tag column repeated in Dnsmasq options panel
1 participant
0