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

Jump to bottom

(1) A minor request to improve the log (2) A minor bug in one log message #384

Closed
ZjYwMj opened this issue Dec 27, 2020 · 1 comment
Closed

(1) A minor request to improve the log (2) A minor bug in one log message #384

ZjYwMj opened this issue Dec 27, 2020 · 1 comment
Assignees
@gthess

Comments

@ZjYwMj
Copy link

ZjYwMj commented Dec 27, 2020

unbound version is 1.12.0

  1. notice: init module 1: subnet
    Having a config-module: "subnet ..."
    results in an error. The correct configuration is, probably,
    config-module: "subnetcache ..."
    Is that correct?
    With config-module: "subnetcache ..."
    the log have:

    unbound[3228]: [3228:0] notice: init module 1: subnet

    I think the name in the log should match exactly the name in
    unbound -V, and in the configuration.

  2. See the list of linked modules with unbound -h
    Sometimes, when there is a reference to a module that is not present, the comment in the log is:

    ... This module is not present (not compiled in), See the list of linked modules with unbound -h

    Was that meant to be unbound -V?
@gthess gthess self-assigned this Feb 24, 2021
@gthess
Copy link
Member

gthess commented Feb 24, 2021

Thanks for reporting!

@gthess gthess closed this as completed in d69132b Feb 24, 2021
jedisct1 added a commit to jedisct1/unbound that referenced this issue Feb 26, 2021
@jedisct1
Merge remote-tracking branch 'nlnet/master'
5b3e626 
* nlnet/master: (103 commits)
  - Fix: Resolve interface names on control-interface too.
  - Fix for NLnetLabs#367: rc_ports don't have ub_sock; skip cleaning up.
  - Fix to allow rpz with wildcard that applies to all TLDs at once.
  Changelog note for NLnetLabs#365, NLnetLabs#367 and NLnetLabs#368. - Merge PR NLnetLabs#367 : DNSTAP log local address.  With code from PR NLnetLabs#365   and fixes NLnetLabs#368 : dnstap does not log the DNS message ID for   FORWARDER_QUERY.
  Fix comment item.
  Fix to use a simple pointer in the call of make_sock and make_sock_port.
  - spelling fix in header.
  - Fix unit test for added ulimit checks.
  - Fix function documentation.
  - On startup of unbound it checks if rlimits on memory size look   sufficient for the configured cache size, and logs warning if not.
  - ipsecmod: Better logging for detecting a cycle when attaching the   A/AAAA subquery.
  - Fix NLnetLabs#384: (1) A minor request to improve the log (2) A minor bug in   one log message.
  - Fix for zonemd, do not reject insecure result from trust anchor   validation step in dnssec chain of trust.
  - Fix for zonemd, that domain-insecure zones work without dnssec.
  Spelling fix.
  - Fix for zonemd, that nxdomain for the chain of trust is allowed   for island zones, it is treates as an insecure zone for verification.
  - Fix NLnetLabs#431: Squelch permission denied errors for tcp connect
  - rpz skip nsec3param records, and nicer log for unsupported actions.
  - Fix NLnetLabs#429: rpz: url: with https: broken (regression in 1.13.1).
  - Fix doxygen and pydoc warnings.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gthess gthess

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gthess @ZjYwMj