Skip to content

Avoid including additionals when the answer is suppressed by known-answer supression#614

Merged
bdraco merged 1 commit into
python-zeroconf:masterfrom
bdraco:fix_known_answer_suppression
Jun 14, 2021
Merged

Avoid including additionals when the answer is suppressed by known-answer supression#614
bdraco merged 1 commit into
python-zeroconf:masterfrom
bdraco:fix_known_answer_suppression

Conversation

@bdraco

@bdraco bdraco commented Jun 14, 2021

Copy link
Copy Markdown
Member

Fixes #613

Broken out from #603

…swer supression

- If the PTR or SRV record was suppressed we should not
  include the additionals per
  https://tools.ietf.org/html/rfc6763#section-12.1
@codecov-commenter

codecov-commenter commented Jun 14, 2021

Copy link
Copy Markdown

Codecov Report

Merging #614 (575097e) into master (aea2c8a) will increase coverage by 0.16%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #614      +/-   ##
==========================================
+ Coverage   95.55%   95.71%   +0.16%     
==========================================
  Files          18       18              
  Lines        1956     1961       +5     
  Branches      345      347       +2     
==========================================
+ Hits         1869     1877       +8     
+ Misses         46       45       -1     
+ Partials       41       39       -2     
Impacted Files Coverage Δ
zeroconf/_handlers.py 100.00% <100.00%> (ø)
zeroconf/_services/__init__.py 93.22% <0.00%> (+0.26%) ⬆️
zeroconf/_core.py 89.86% <0.00%> (+0.67%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aea2c8a...575097e. Read the comment docs.

@bdraco bdraco merged commit 219aa3e into python-zeroconf:master Jun 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Additionals are included event when the answer is suppressed by known answer supression

2 participants