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

Match cnames in ipset #632

Merged
merged 1 commit into from
Mar 2, 2022
Merged

Match cnames in ipset #632

merged 1 commit into from
Mar 2, 2022

Conversation

scottrw93
Copy link
Contributor

ipset module does not add ip to list when the specified domain is a CNAME

@gthess gthess self-assigned this Mar 1, 2022
@gthess gthess merged commit a688064 into NLnetLabs:master Mar 2, 2022
gthess added a commit that referenced this pull request Mar 2, 2022
- Merge PR #632 from scottrw93: Match cnames in ipset.
gthess added a commit that referenced this pull request Mar 2, 2022
  to str once, accept trailing dot in the local-zone ipset option.
@gthess
Copy link
Member

gthess commented Mar 2, 2022

Thanks for this! I went ahead and made some further fixes and introduced an ipset test.

@scottrw93 scottrw93 deleted the cnames-ipset branch March 3, 2022 09:31
jedisct1 added a commit to jedisct1/unbound that referenced this pull request Mar 14, 2022
* nlnet/master: (27 commits)
  Changelog note for NLnetLabs#644, move commands together for library binary.
  Make `install-lib` make target install the pkg-config file.
  - Fix configure for python to use sysutils, because distutils is   deprecated. It uses sysutils when available, distutils otherwise.
  - Fix for NLnetLabs#637: fix integer overflow checks in sldns_str2period.
  - Fix NLnetLabs#637: Integer Overflow in sldns_str2period function.
  - Fix compile warnings for printf ll format on mingw compile.
  - Various fixes for NLnetLabs#632: variable initialisation, convert the qinfo   to str once, accept trailing dot in the local-zone ipset option.
  Changelog entry for NLnetLabs#632 - Merge PR NLnetLabs#632 from scottrw93: Match cnames in ipset.
  - Added tests for ipset.
  - Fix pythonmod for change in iter_dp_is_useless function prototype.
  - Fix for edns client subnet option add fix in removal code, from review.
  - Fix edns client subnet to add the option based on the option list,   so that it is not state dependent, after the state fix of NLnetLabs#605 for   double EDNS options.
  Changelog entry for NLnetLabs#623: - Merge NLnetLabs#623 from rex4539: Fix typos.
  - Fix NLnetLabs#630: Unify the RPZ log messages.
  - Fix for NLnetLabs#633: updated fix with new text.
  - Fix NLnetLabs#633: Document unix domain socket support for unbound-control.
  - Fix check interface existence for support detection in remote lookup.
  - update Makefile dependencies.
  - Fix to detect that no IPv6 support means that IPv6 addresses are   useless for delegation point lookups.
  Match cnames in ipset
  ...
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.

None yet

2 participants