Skip to content

Limit duplicate packet suppression to 1s intervals - #841

Merged
bdraco merged 1 commit into
python-zeroconf:masterfrom
bdraco:limit_suppress_duplicate_packet
Jun 26, 2021
Merged

bdraco merged 1 commit into
python-zeroconf:masterfrom
bdraco:limit_suppress_duplicate_packet

Conversation

@bdraco

@bdraco bdraco commented Jun 26, 2021

Copy link
Copy Markdown
Member
  • Only suppress duplicate packets that happen within the same
    second. Legitimate queriers will retry the question if they
    are suppressed. The limit was reduced to one second to be
    in line with rfc6762:

    To protect the network against excessive packet flooding due to
    software bugs or malicious attack, a Multicast DNS responder MUST NOT
    (except in the one special case of answering probe queries) multicast
    a record on a given interface until at least one second has elapsed
    since the last time that record was multicast on that particular

- Only suppress duplicate packets that happen within the same
  second. Legitimate queriers will retry the question if they
  are suppressed. The limit was reduced to one second to be
  in line with rfc6762:

   To protect the network against excessive packet flooding due to
   software bugs or malicious attack, a Multicast DNS responder MUST NOT
   (except in the one special case of answering probe queries) multicast
   a record on a given interface until at least one second has elapsed
   since the last time that record was multicast on that particular
@codecov-commenter

codecov-commenter commented Jun 26, 2021 •

Copy link
Copy Markdown

Codecov Report

Merging #841 (ef34152) into master (937be52) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #841   +/-   ##
=======================================
  Coverage   98.63%   98.63%           
=======================================
  Files          22       22           
  Lines        2342     2349    +7     
  Branches      397      398    +1     
=======================================
+ Hits         2310     2317    +7     
  Misses         21       21           
  Partials       11       11           
Impacted Files Coverage Δ
zeroconf/_core.py 98.02% <100.00%> (+0.03%) ⬆️
zeroconf/_protocol.py 100.00% <100.00%> (ø)

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 937be52...ef34152. Read the comment docs.

@bdraco
bdraco merged commit 7fb11bf into python-zeroconf:master Jun 26, 2021
@bdraco
bdraco deleted the limit_suppress_duplicate_packet branch June 26, 2021 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants