Skip to content

Fix ServiceBrowsers not getting ServiceStateChange.Removed callbacks on PTR record expire#1064

Merged
bdraco merged 1 commit into
python-zeroconf:masterfrom
bdraco:missing_remove_callbacks
May 1, 2022
Merged

Fix ServiceBrowsers not getting ServiceStateChange.Removed callbacks on PTR record expire#1064
bdraco merged 1 commit into
python-zeroconf:masterfrom
bdraco:missing_remove_callbacks

Conversation

@bdraco
Copy link
Copy Markdown
Member

@bdraco bdraco commented May 1, 2022

ServiceBrowsers were only getting a ServiceStateChange.Removed callback when the record was sent with a TTL of 0. They now correctly get a ServiceStateChange.Removed callback when the record expires as well.

cc @AlexxIT

@bdraco bdraco changed the title Update example to inherit from ServiceListener and include required update_service Fix ServiceBrowsers not getting callbacks on PTR record expire May 1, 2022
@bdraco bdraco force-pushed the missing_remove_callbacks branch from f927628 to 53e5707 Compare May 1, 2022 14:05
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 1, 2022

Codecov Report

Merging #1064 (31662b7) into master (31662b7) will not change coverage.
The diff coverage is n/a.

❗ Current head 31662b7 differs from pull request most recent head 53e5707. Consider uploading reports for the commit 53e5707 to get more accurate results

@@           Coverage Diff           @@
##           master    #1064   +/-   ##
=======================================
  Coverage   99.92%   99.92%           
=======================================
  Files          23       23           
  Lines        2544     2544           
  Branches      473      473           
=======================================
  Hits         2542     2542           
  Misses          1        1           
  Partials        1        1           

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 31662b7...53e5707. Read the comment docs.

@bdraco bdraco changed the title Fix ServiceBrowsers not getting callbacks on PTR record expire Fix ServiceBrowsers not getting ServiceStateChange.Removed callbacks on PTR record expire May 1, 2022
@bdraco bdraco merged commit 10ee205 into python-zeroconf:master May 1, 2022
@bdraco bdraco deleted the missing_remove_callbacks branch May 1, 2022 16:43
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