Skip to content
Snippets Groups Projects

policy: log selected actions

Merged Tomas Krizek requested to merge log-policy-actions into master
All threads resolved!

Rebased on top of !1234 (merged) (but planned to merge to master after EDE gets there)


The following actions will now be logged in debug level (or request tracing): ANSWER, DENY, DENY_MSG, DROP, REFUSE, TC

This can be useful for RPZ and other policy debugging.

Purposefully ommitted actions:

  • PASS - since it's the same as normal processing
  • REROUTE - the action itself comes from renumber module
  • STUB,FORWARD,TLS_FORWARD - this could be more confusing than useful (e.g. when response comes from cache)

Fixes #689 (closed)

Edited by Tomas Krizek

Merge request reports

Pipeline #92602 failed

Pipeline: Knot Resolver

#92603

    Pipeline failed for 89aab472 on log-policy-actions

    Approval is optional

    Merged by Vladimír ČunátVladimír Čunát 3 years ago (Dec 22, 2021 10:04am UTC)

    Merge details

    Pipeline #92654 passed

    Pipeline passed for e862a78d on master

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • Tomas Krizek resolved all threads

    resolved all threads

  • Tomas Krizek deleted the extended-errors branch. This merge request now targets the master branch

    deleted the extended-errors branch. This merge request now targets the master branch

  • Tomas Krizek added 37 commits

    added 37 commits

    Compare with previous version

  • Tomas Krizek added 4 commits

    added 4 commits

    • 46158186 - policy: log selected actions
    • b4b918b9 - doc: fix link to log groups
    • f41ee8c3 - doc: add mention about policy debug logging near RPZ
    • 89aab472 - policy: add policy.IPTRACE logging action

    Compare with previous version

  • Tomas Krizek marked this merge request as ready

    marked this merge request as ready

  • mentioned in commit e862a78d

  • Please register or sign in to reply
    Loading