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

Updates Syntax06 specification to lower level of no response #954

Merged
merged 1 commit into from
May 26, 2021

Conversation

matsduf
Copy link
Contributor

@matsduf matsduf commented May 25, 2021

BASIC04 has been implemented to report on problem with responses from nameservers. The Syntax06 specification and its implementation, repeats reporting the same thing with no benefit. With this PR the Syntax06 test case will refer that to BASIC04.

A PR to lower the level in the profile is created in Zonemaster-Engine (zonemaster/zonemaster-engine#923).

  • Adds reference to BASIC04, which tests all name servers for non-response.
  • Lower default level to DEBUG for messages that report on non-response.

This PR relates to #950, #951, #952, #953

* Adds reference to BASIC04, which tests all name servers for non-response.
* Lower default level to DEBUG for messages that report on non-response.
@matsduf matsduf added A-TestCase Area: Test case specification or implementation of test case P-High Priority: Issue to be solved before other labels May 25, 2021
@matsduf matsduf added this to the v2021.1 milestone May 25, 2021
@matsduf matsduf requested review from mattias-p and a user May 25, 2021 12:28
@matsduf matsduf merged commit e8ad300 into zonemaster:develop May 26, 2021
@matsduf matsduf deleted the syntax-lower-no-response branch May 26, 2021 17:57
tgreenx pushed a commit to tgreenx/zonemaster that referenced this pull request Jan 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-TestCase Area: Test case specification or implementation of test case P-High Priority: Issue to be solved before other
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants