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

Increase level when AAAA queries are mistreated #778

Closed
matsduf opened this issue Jun 20, 2019 · 2 comments
Closed

Increase level when AAAA queries are mistreated #778

matsduf opened this issue Jun 20, 2019 · 2 comments
Assignees
@matsduf
Milestone
v2019.2

Comments

@matsduf
Copy link
Contributor

matsduf commented Jun 20, 2019

Test case Nameserver05 tests if the nameserver has bad behavior when receiving AAAA query. If the query is dropped (no response) the message QUERY_DROPPED is outputted with the default level of NOTICE in the implementation (the test case has no specification).

The level should be at least WARNING and should be specified in the test case. The test case specification should be updated.

If an unexpected RCODE is returned the default level is also NOTICE and should also be increased.

This is an partial answer to zonemaster/zonemaster-engine#563.
@matsduf matsduf added this to the v2019.2 milestone Jun 20, 2019
@matsduf matsduf self-assigned this Jun 20, 2019
@matsduf matsduf mentioned this issue Jun 28, 2019
Closed
@matsduf matsduf mentioned this issue Jul 15, 2019
Merged
@matsduf
Copy link
Contributor Author

matsduf commented Jul 15, 2019

PR #787 has been created to solve this issue.

@matsduf
Copy link
Contributor Author

matsduf commented Oct 10, 2019

Resolved by PR #787.

@matsduf matsduf closed this as completed Oct 10, 2019
tgreenx pushed a commit to tgreenx/zonemaster that referenced this issue Jan 3, 2022
@mattias-p
Merge pull request zonemaster#778 from mattias-p/324-attr-type-constr…
bff5efd 
…aint

Add type constraint to Z::E::Nameserver::Cache->data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matsduf matsduf

Labels
None yet
Projects
None yet
Milestone
v2019.2
Development

No branches or pull requests
1 participant
@matsduf