zeroconf 0.147.2__tar.gz → 0.147.4__tar.gz

{zeroconf-0.147.2 → zeroconf-0.147.4}/CHANGELOG.md

@@ -1,5 +1,70 @@
 # CHANGELOG
 
+<!-- version list -->
+
+## v0.147.4 (2026-05-16)
+
+### Bug Fixes
+
+- **core**: Close owned event loop on Zeroconf.close() to stop FD leak
+  ([#1685](https://github.com/python-zeroconf/python-zeroconf/pull/1685),
+  [`2f78370`](https://github.com/python-zeroconf/python-zeroconf/commit/2f78370c75d1082afe3191b7447aebfff1206657))
+
+### Build System
+
+- Adjust actions checkout ref parameter on release
+  ([#1669](https://github.com/python-zeroconf/python-zeroconf/pull/1669),
+  [`bc8ec8d`](https://github.com/python-zeroconf/python-zeroconf/commit/bc8ec8d59d875522f75901644d423d30d803a030))
+
+### Documentation
+
+- Add CLAUDE.md orientation file and pr-workflow skill
+  ([#1672](https://github.com/python-zeroconf/python-zeroconf/pull/1672),
+  [`8f8b4d6`](https://github.com/python-zeroconf/python-zeroconf/commit/8f8b4d6526729906337f4562c7b391745bb878af))
+
+- Add Cython gotchas section to CLAUDE.md
+  ([#1679](https://github.com/python-zeroconf/python-zeroconf/pull/1679),
+  [`5cfb09d`](https://github.com/python-zeroconf/python-zeroconf/commit/5cfb09d89395cb507d436c45fc38edb9e44b94c8))
+
+- Add SECURITY.md with private vulnerability reporting policy
+  ([#1675](https://github.com/python-zeroconf/python-zeroconf/pull/1675),
+  [`13f9048`](https://github.com/python-zeroconf/python-zeroconf/commit/13f9048f0f9786ce18e89daef04073847735a006))
+
+### Testing
+
+- Add quick_timing fixture and apply to register-heavy tests
+  ([#1678](https://github.com/python-zeroconf/python-zeroconf/pull/1678),
+  [`d5e1f01`](https://github.com/python-zeroconf/python-zeroconf/commit/d5e1f01bb336ea19d982ce7d99f191723d3f18af))
+
+- Fix flaky test_run_coro_with_timeout
+  ([#1683](https://github.com/python-zeroconf/python-zeroconf/pull/1683),
+  [`277f80d`](https://github.com/python-zeroconf/python-zeroconf/commit/277f80da2c0fea5b256f981bd3f425906f6b7be6))
+
+- Pass timeout=0 explicitly in test_event_loop_blocked
+  ([#1676](https://github.com/python-zeroconf/python-zeroconf/pull/1676),
+  [`1b31ed5`](https://github.com/python-zeroconf/python-zeroconf/commit/1b31ed5fed9db1608255799701cd6f32b494952f))
+
+- Pass timeout=200 to ServiceInfo-request timeout tests
+  ([#1677](https://github.com/python-zeroconf/python-zeroconf/pull/1677),
+  [`01ef6ff`](https://github.com/python-zeroconf/python-zeroconf/commit/01ef6ffd9ff442b3cfb37d2793e0ca6ad5148832))
+
+
+## v0.148.0 (2025-10-05)
+
+### Features
+
+- Trigger semantic releases for 0.x branch
+  ([#1626](https://github.com/python-zeroconf/python-zeroconf/pull/1626),
+  [`812a2b3`](https://github.com/python-zeroconf/python-zeroconf/commit/812a2b3ff4370593a7a0c3ad67389c76c434aa9b))
+
+
+## v0.147.3 (2025-10-04)
+
+### Bug Fixes
+
+- Update poetry to v2 ([#1623](https://github.com/python-zeroconf/python-zeroconf/pull/1623),
+  [`2c3c296`](https://github.com/python-zeroconf/python-zeroconf/commit/2c3c29655bd365213a7e0a4360b8dd860d833470))
+
 
 ## v0.147.2 (2025-09-05)
 
@@ -1976,3 +2041,5 @@ Include documentation and test files in source distributions, in order to make t
 
 
 ## v0.15.1 (2014-07-10)
+
+- Initial Release

{zeroconf-0.147.2 → zeroconf-0.147.4}/PKG-INFO

@@ -1,14 +1,14 @@
-Metadata-Version: 2.3
+Metadata-Version: 2.4
 Name: zeroconf
-Version: 0.147.2
+Version: 0.147.4
 Summary: A pure python implementation of multicast DNS service discovery
-License: LGPL-2.1-or-later
+License-Expression: LGPL-2.1-or-later
+License-File: COPYING
 Author: Paul Scott-Murphy
-Requires-Python: >=3.9,<4.0
+Requires-Python: >=3.9
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Intended Audience :: Developers
 Classifier: Intended Audience :: System Administrators
-Classifier: License :: OSI Approved
 Classifier: Operating System :: MacOS :: MacOS X
 Classifier: Operating System :: POSIX
 Classifier: Operating System :: POSIX :: Linux
@@ -18,6 +18,7 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Programming Language :: Python :: Implementation :: CPython
 Classifier: Programming Language :: Python :: Implementation :: PyPy
 Classifier: Topic :: Software Development :: Libraries

{zeroconf-0.147.2 → zeroconf-0.147.4}/pyproject.toml

@@ -1,12 +1,28 @@
-[tool.poetry]
+[build-system]
+requires = ['setuptools>=77.0', 'Cython>=3.0.8', "poetry-core>=2.1.0"]
+build-backend = "poetry.core.masonry.api"
+
+[project]
 name = "zeroconf"
-version = "0.147.2"
-description = "A pure python implementation of multicast DNS service discovery"
-authors = ["Paul Scott-Murphy", "William McBrine", "Jakub Stasiak", "J. Nick Koston"]
+version = "0.147.4"
 license = "LGPL-2.1-or-later"
+description = "A pure python implementation of multicast DNS service discovery"
 readme = "README.rst"
-repository = "https://github.com/python-zeroconf/python-zeroconf"
-documentation = "https://python-zeroconf.readthedocs.io"
+authors = [
+    { name = "Paul Scott-Murphy" },
+    { name = "William McBrine" },
+    { name = "Jakub Stasiak" },
+    { name = "J. Nick Koston" },
+]
+requires-python = ">=3.9"
+
+[project.urls]
+"Repository" = "https://github.com/python-zeroconf/python-zeroconf"
+"Documentation" = "https://python-zeroconf.readthedocs.io"
+"Bug Tracker" = "https://github.com/python-zeroconf/python-zeroconf/issues"
+"Changelog" = "https://github.com/python-zeroconf/python-zeroconf/blob/master/CHANGELOG.md"
+
+[tool.poetry]
 classifiers=[
 	'Development Status :: 5 - Production/Stable',
 	'Intended Audience :: Developers',
@@ -15,11 +31,6 @@ classifiers=[
 	'Operating System :: POSIX :: Linux',
 	'Operating System :: MacOS :: MacOS X',
 	'Topic :: Software Development :: Libraries',
-	'Programming Language :: Python :: 3.9',
-	'Programming Language :: Python :: 3.10',
-	'Programming Language :: Python :: 3.11',
-	'Programming Language :: Python :: 3.12',
-	'Programming Language :: Python :: 3.13',
 	'Programming Language :: Python :: Implementation :: CPython',
 	'Programming Language :: Python :: Implementation :: PyPy',
 ]
@@ -35,22 +46,19 @@ include = [
 # Make sure we don't package temporary C files generated by the build process
 exclude = [ "**/*.c" ]
 
-[tool.poetry.urls]
-"Bug Tracker" = "https://github.com/python-zeroconf/python-zeroconf/issues"
-"Changelog" = "https://github.com/python-zeroconf/python-zeroconf/blob/master/CHANGELOG.md"
-
 [tool.poetry.build]
 generate-setup-file = true
 script = "build_ext.py"
 
 [tool.semantic_release]
 branch = "master"
-version_toml = ["pyproject.toml:tool.poetry.version"]
+version_toml = ["pyproject.toml:project.version"]
 version_variables = [
     "src/zeroconf/__init__.py:__version__"
 ]
 build_command = "pip install poetry && poetry build"
 tag_format = "{version}"
+allow_zero_version = true
 
 [tool.semantic_release.changelog]
 exclude_commit_patterns = [
@@ -64,8 +72,11 @@ keep_trailing_newline = true
 [tool.semantic_release.branches.master]
 match = "master"
 
+[tool.semantic_release.branches."release-0.x"]
+match = "release-0.x"
+
 [tool.semantic_release.branches.noop]
-match = "(?!master$)"
+match = "(?!(master|release-0.x)$)"
 prerelease = true
 
 [tool.poetry.dependencies]
@@ -74,16 +85,16 @@ ifaddr = ">=0.1.7"
 
 [tool.poetry.group.dev.dependencies]
 pytest = ">=7.2,<9.0"
-pytest-cov = ">=4,<7"
-pytest-asyncio = ">=0.20.3,<1.2.0"
-cython = "^3.0.5"
-setuptools = ">=65.6.3,<81.0.0"
+pytest-cov = ">=4,<8"
+pytest-asyncio = ">=0.20.3,<1.3.0"
+cython = "^3.2.4"
+setuptools = ">=65.6.3,<83.0.0"
 pytest-timeout = "^2.1.0"
-pytest-codspeed = ">=3.1,<5.0"
+pytest-codspeed = ">=4.5.0,<5.0"
 
 [tool.poetry.group.docs.dependencies]
 sphinx = "^7.4.7 || ^8.1.3"
-sphinx-rtd-theme = "^3.0.2"
+sphinx-rtd-theme = "^3.1.0"
 
 [tool.ruff]
 target-version = "py39"
@@ -159,12 +170,9 @@ select = [
     "FLY002", # too many to fix right now
     "PT018", # too many to fix right now
     "PLR0124", # too many to fix right now
-    "SIM202" , # too many to fix right now
     "PT012" , # too many to fix right now
     "TID252", # too many to fix right now
     "PLR0913", # skip this one
-    "SIM102" , # too many to fix right now
-    "SIM108", # too many to fix right now
     "T201", # too many to fix right now
     "PT004", # nice to have
 ]
@@ -266,11 +274,6 @@ allow_untyped_defs = true
 module = "bench.*"
 ignore_errors = true
 
-[build-system]
-# 1.5.2 required for https://github.com/python-poetry/poetry/issues/7505
-requires = ['setuptools>=65.4.1', 'wheel', 'Cython>=3.0.8', "poetry-core>=1.5.2"]
-build-backend = "poetry.core.masonry.api"
-
 [tool.codespell]
 ignore-words-list = ["additionals", "HASS"]
 

{zeroconf-0.147.2 → zeroconf-0.147.4}/setup.py

@@ -19,7 +19,7 @@ install_requires = \
 
 setup_kwargs = {
     'name': 'zeroconf',
-    'version': '0.147.2',
+    'version': '0.147.4',
     'description': 'A pure python implementation of multicast DNS service discovery',
     'long_description': 'python-zeroconf\n===============\n\n.. image:: https://github.com/python-zeroconf/python-zeroconf/workflows/CI/badge.svg\n   :target: https://github.com/python-zeroconf/python-zeroconf?query=workflow%3ACI+branch%3Amaster\n\n.. image:: https://img.shields.io/pypi/v/zeroconf.svg\n    :target: https://pypi.python.org/pypi/zeroconf\n\n.. image:: https://codecov.io/gh/python-zeroconf/python-zeroconf/branch/master/graph/badge.svg\n   :target: https://codecov.io/gh/python-zeroconf/python-zeroconf\n\n.. image:: https://img.shields.io/endpoint?url=https://codspeed.io/badge.json\n   :target: https://codspeed.io/python-zeroconf/python-zeroconf\n   :alt: Codspeed.io status for python-zeroconf\n\n.. image:: https://readthedocs.org/projects/python-zeroconf/badge/?version=latest\n    :target: https://python-zeroconf.readthedocs.io/en/latest/?badge=latest\n    :alt: Documentation Status\n\n`Documentation <https://python-zeroconf.readthedocs.io/en/latest/>`_.\n\nThis is fork of pyzeroconf, Multicast DNS Service Discovery for Python,\noriginally by Paul Scott-Murphy (https://github.com/paulsm/pyzeroconf),\nmodified by William McBrine (https://github.com/wmcbrine/pyzeroconf).\n\nThe original William McBrine\'s fork note::\n\n    This fork is used in all of my TiVo-related projects: HME for Python\n    (and therefore HME/VLC), Network Remote, Remote Proxy, and pyTivo.\n    Before this, I was tracking the changes for zeroconf.py in three\n    separate repos. I figured I should have an authoritative source.\n\n    Although I make changes based on my experience with TiVos, I expect that\n    they\'re generally applicable. This version also includes patches found\n    on the now-defunct (?) Launchpad repo of pyzeroconf, and elsewhere\n    around the net -- not always well-documented, sorry.\n\nCompatible with:\n\n* Bonjour\n* Avahi\n\nCompared to some other Zeroconf/Bonjour/Avahi Python packages, python-zeroconf:\n\n* isn\'t tied to Bonjour or Avahi\n* doesn\'t use D-Bus\n* doesn\'t force you to use particular event loop or Twisted (asyncio is used under the hood but not required)\n* is pip-installable\n* has PyPI distribution\n* has an optional cython extension for performance (pure python is supported as well)\n\nPython compatibility\n--------------------\n\n* CPython 3.9+\n* PyPy 3.9+\n\nVersioning\n----------\n\nThis project uses semantic versioning.\n\nStatus\n------\n\nThis project is actively maintained.\n\nTraffic Reduction\n-----------------\n\nBefore version 0.32, most traffic reduction techniques described in https://datatracker.ietf.org/doc/html/rfc6762#section-7\nwhere not implemented which could lead to excessive network traffic.  It is highly recommended that version 0.32 or later\nis used if this is a concern.\n\nIPv6 support\n------------\n\nIPv6 support is relatively new and currently limited, specifically:\n\n* `InterfaceChoice.All` is an alias for `InterfaceChoice.Default` on non-POSIX\n  systems.\n* Dual-stack IPv6 sockets are used, which may not be supported everywhere (some\n  BSD variants do not have them).\n* Listening on localhost (`::1`) does not work. Help with understanding why is\n  appreciated.\n\nHow to get python-zeroconf?\n===========================\n\n* PyPI page https://pypi.org/project/zeroconf/\n* GitHub project https://github.com/python-zeroconf/python-zeroconf\n\nThe easiest way to install python-zeroconf is using pip::\n\n    pip install zeroconf\n\n\n\nHow do I use it?\n================\n\nHere\'s an example of browsing for a service:\n\n.. code-block:: python\n\n    from zeroconf import ServiceBrowser, ServiceListener, Zeroconf\n\n\n    class MyListener(ServiceListener):\n\n        def update_service(self, zc: Zeroconf, type_: str, name: str) -> None:\n            print(f"Service {name} updated")\n\n        def remove_service(self, zc: Zeroconf, type_: str, name: str) -> None:\n            print(f"Service {name} removed")\n\n        def add_service(self, zc: Zeroconf, type_: str, name: str) -> None:\n            info = zc.get_service_info(type_, name)\n            print(f"Service {name} added, service info: {info}")\n\n\n    zeroconf = Zeroconf()\n    listener = MyListener()\n    browser = ServiceBrowser(zeroconf, "_http._tcp.local.", listener)\n    try:\n        input("Press enter to exit...\\n\\n")\n    finally:\n        zeroconf.close()\n\n.. note::\n\n    Discovery and service registration use *all* available network interfaces by default.\n    If you want to customize that you need to specify ``interfaces`` argument when\n    constructing ``Zeroconf`` object (see the code for details).\n\nIf you don\'t know the name of the service you need to browse for, try:\n\n.. code-block:: python\n\n    from zeroconf import ZeroconfServiceTypes\n    print(\'\\n\'.join(ZeroconfServiceTypes.find()))\n\nSee examples directory for more.\n\nChangelog\n=========\n\n`Changelog <CHANGELOG.md>`_\n\nLicense\n=======\n\nLGPL, see COPYING file for details.\n',
     'author': 'Paul Scott-Murphy',
@@ -31,7 +31,7 @@ setup_kwargs = {
     'packages': packages,
     'package_data': package_data,
     'install_requires': install_requires,
-    'python_requires': '>=3.9,<4.0',
+    'python_requires': '>=3.9',
 }
 from build_ext import *
 build(setup_kwargs)

{zeroconf-0.147.2 → zeroconf-0.147.4}/src/zeroconf/__init__.py

@@ -88,7 +88,7 @@ from ._utils.time import (  # noqa # import needed for backwards compat
 
 __author__ = "Paul Scott-Murphy, William McBrine"
 __maintainer__ = "Jakub Stasiak <jakub@stasiak.at>"
-__version__ = "0.147.2"
+__version__ = "0.147.4"
 __license__ = "LGPL"
 
 

{zeroconf-0.147.2 → zeroconf-0.147.4}/src/zeroconf/_core.py

@@ -676,13 +676,22 @@ class Zeroconf(QuietLogger):
 
     def _shutdown_threads(self) -> None:
         """Shutdown any threads."""
+        assert self.loop is not None
+        if self.loop.is_closed():
+            # close() is documented as idempotent — a second call after the
+            # loop has been torn down must be a no-op rather than raising.
+            return
         self.notify_all()
         if not self._loop_thread:
             return
-        assert self.loop is not None
         shutdown_loop(self.loop)
         self._loop_thread.join()
         self._loop_thread = None
+        # The loop's selector (epoll FD on Linux) and self-pipe sockets stay
+        # open until loop.close() is called. We own this loop because
+        # _start_thread() created it, so close it here to avoid leaking
+        # those file descriptors across Zeroconf() construct/close cycles.
+        self.loop.close()
 
     def close(self) -> None:
         """Ends the background threads, and prevent this instance from

{zeroconf-0.147.2 → zeroconf-0.147.4}/src/zeroconf/_services/browser.py

@@ -23,6 +23,7 @@ USA
 from __future__ import annotations
 
 import asyncio
+import contextlib
 import heapq
 import queue
 import random
@@ -793,7 +794,16 @@ class ServiceBrowser(_ServiceBrowserBase, threading.Thread):
         """Cancel the browser."""
         assert self.zc.loop is not None
         self.queue.put(None)
-        self.zc.loop.call_soon_threadsafe(self._async_cancel)
+        # While the loop is running, _async_cancel stops the query scheduler
+        # and cancels the query-sender task — that is the normal cleanup
+        # path. Skip scheduling solely because the loop is closed: a closed
+        # loop rejects call_soon_threadsafe with RuntimeError. The
+        # is_closed() check narrows the common case (loop already closed by
+        # Zeroconf.close()) without paying for raise/catch; suppress covers
+        # the residual is_closed() -> call_soon_threadsafe race window.
+        with contextlib.suppress(RuntimeError):
+            if not self.zc.loop.is_closed():
+                self.zc.loop.call_soon_threadsafe(self._async_cancel)
         self.join()
 
     def run(self) -> None:

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/__init__.py

@@ -95,10 +95,7 @@ def time_changed_millis(millis: float | None = None) -> None:
     """Call all scheduled events for a time."""
     loop = asyncio.get_running_loop()
     loop_time = loop.time()
-    if millis is not None:
-        mock_seconds_into_future = millis / 1000
-    else:
-        mock_seconds_into_future = loop_time
+    mock_seconds_into_future = millis / 1000 if millis is not None else loop_time
 
     with mock.patch("time.monotonic", return_value=mock_seconds_into_future):
         for task in list(loop._scheduled):  # type: ignore[attr-defined]

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/conftest.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import threading
+from collections.abc import Generator
 from unittest.mock import patch
 
 import pytest
@@ -40,3 +41,21 @@ def disable_duplicate_packet_suppression():
     """
     with patch.object(const, "_DUPLICATE_PACKET_SUPPRESSION_INTERVAL", 0):
         yield
+
+
+@pytest.fixture
+def quick_timing() -> Generator[None]:
+    """Shorten the probe/announce/goodbye intervals for tests on loopback.
+
+    The production values (_CHECK_TIME=500ms, _REGISTER_TIME=225ms,
+    _UNREGISTER_TIME=125ms) exist for RFC 6762 interop on real
+    networks. Tests on 127.0.0.1 do not need them and pay 1-2s per
+    register/unregister cycle without this fixture. Opt in by adding
+    `quick_timing` to a test's argument list.
+    """
+    with (
+        patch.object(_core, "_CHECK_TIME", 10),
+        patch.object(_core, "_REGISTER_TIME", 10),
+        patch.object(_core, "_UNREGISTER_TIME", 10),
+    ):
+        yield

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/services/test_browser.py

@@ -227,10 +227,7 @@ class TestServiceBrowser(unittest.TestCase):
             generated = r.DNSOutgoing(const._FLAGS_QR_RESPONSE)
             assert generated.is_response() is True
 
-            if service_state_change == r.ServiceStateChange.Removed:
-                ttl = 0
-            else:
-                ttl = 120
+            ttl = 0 if service_state_change == r.ServiceStateChange.Removed else 120
 
             generated.add_answer_at_time(
                 r.DNSText(
@@ -1575,9 +1572,8 @@ async def test_close_zeroconf_without_browser_before_start_up_queries():
     registration_name = f"xxxyyy.{type_}"
 
     def on_service_state_change(zeroconf, service_type, state_change, name):
-        if name == registration_name:
-            if state_change is ServiceStateChange.Added:
-                service_added.set()
+        if name == registration_name and state_change is ServiceStateChange.Added:
+            service_added.set()
 
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
     zeroconf_browser = aiozc.zeroconf
@@ -1644,9 +1640,8 @@ async def test_close_zeroconf_without_browser_after_start_up_queries():
     registration_name = f"xxxyyy.{type_}"
 
     def on_service_state_change(zeroconf, service_type, state_change, name):
-        if name == registration_name:
-            if state_change is ServiceStateChange.Added:
-                service_added.set()
+        if name == registration_name and state_change is ServiceStateChange.Added:
+            service_added.set()
 
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
     zeroconf_browser = aiozc.zeroconf

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/services/test_info.py

@@ -1050,12 +1050,12 @@ def test_request_timeout():
     """Test that the timeout does not throw an exception and finishes close to the actual timeout."""
     zeroconf = r.Zeroconf(interfaces=["127.0.0.1"])
     start_time = r.current_time_millis()
-    assert zeroconf.get_service_info("_notfound.local.", "notthere._notfound.local.") is None
+    assert zeroconf.get_service_info("_notfound.local.", "notthere._notfound.local.", timeout=200) is None
     end_time = r.current_time_millis()
     zeroconf.close()
-    # 3000ms for the default timeout
+    # 200ms for the timeout passed above
     # 1000ms for loaded systems + schedule overhead
-    assert (end_time - start_time) < 3000 + 1000
+    assert (end_time - start_time) < 200 + 1000
 
 
 @pytest.mark.asyncio
@@ -1888,7 +1888,7 @@ async def test_unicast_flag_if_requested() -> None:
     # patch the zeroconf send
     with patch.object(aiozc.zeroconf, "async_send", async_send):
         await aiozc.async_get_service_info(
-            f"willnotbefound.{type_}", type_, question_type=r.DNSQuestionType.QU
+            f"willnotbefound.{type_}", type_, timeout=200, question_type=r.DNSQuestionType.QU
         )
 
     await aiozc.async_close()

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/test_asyncio.py

@@ -260,7 +260,7 @@ async def test_async_service_registration_with_server_missing() -> None:
 
 
 @pytest.mark.asyncio
-async def test_async_service_registration_same_server_different_ports() -> None:
+async def test_async_service_registration_same_server_different_ports(quick_timing: None) -> None:
     """Test registering services with the same server with different srv records."""
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
     type_ = "_test1-srvc-type._tcp.local."
@@ -327,7 +327,7 @@ async def test_async_service_registration_same_server_different_ports() -> None:
 
 
 @pytest.mark.asyncio
-async def test_async_service_registration_same_server_same_ports() -> None:
+async def test_async_service_registration_same_server_same_ports(quick_timing: None) -> None:
     """Test registering services with the same server with the exact same srv record."""
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
     type_ = "_test1-srvc-type._tcp.local."
@@ -468,7 +468,7 @@ async def test_async_service_registration_name_does_not_match_type() -> None:
 
 
 @pytest.mark.asyncio
-async def test_async_service_registration_name_strict_check() -> None:
+async def test_async_service_registration_name_strict_check(quick_timing: None) -> None:
     """Test registering services throws when the name does not comply."""
     zc = Zeroconf(interfaces=["127.0.0.1"])
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
@@ -824,7 +824,7 @@ async def test_service_browser_cancel_async_context_manager():
 
 
 @pytest.mark.asyncio
-async def test_async_unregister_all_services() -> None:
+async def test_async_unregister_all_services(quick_timing: None) -> None:
     """Test unregistering all services."""
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
     type_ = "_test1-srvc-type._tcp.local."
@@ -870,8 +870,8 @@ async def test_async_unregister_all_services() -> None:
     _clear_cache(aiozc.zeroconf)
 
     tasks = []
-    tasks.append(aiozc.async_get_service_info(type_, registration_name))
-    tasks.append(aiozc.async_get_service_info(type_, registration_name2))
+    tasks.append(aiozc.async_get_service_info(type_, registration_name, timeout=200))
+    tasks.append(aiozc.async_get_service_info(type_, registration_name2, timeout=200))
     results = await asyncio.gather(*tasks)
     assert results[0] is None
     assert results[1] is None
@@ -883,7 +883,7 @@ async def test_async_unregister_all_services() -> None:
 
 
 @pytest.mark.asyncio
-async def test_async_zeroconf_service_types():
+async def test_async_zeroconf_service_types(quick_timing: None) -> None:
     type_ = "_test-srvc-type._tcp.local."
     name = "xxxyyy"
     registration_name = f"{name}.{type_}"
@@ -1276,12 +1276,15 @@ async def test_async_request_timeout():
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
     await aiozc.zeroconf.async_wait_for_start()
     start_time = current_time_millis()
-    assert await aiozc.async_get_service_info("_notfound.local.", "notthere._notfound.local.") is None
+    assert (
+        await aiozc.async_get_service_info("_notfound.local.", "notthere._notfound.local.", timeout=200)
+        is None
+    )
     end_time = current_time_millis()
     await aiozc.async_close()
-    # 3000ms for the default timeout
+    # 200ms for the timeout passed above
     # 1000ms for loaded systems + schedule overhead
-    assert (end_time - start_time) < 3000 + 1000
+    assert (end_time - start_time) < 200 + 1000
 
 
 @pytest.mark.asyncio

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/test_core.py

@@ -12,6 +12,7 @@ import time
 import unittest
 import unittest.mock
 import warnings
+from pathlib import Path
 from typing import cast
 from unittest.mock import AsyncMock, Mock, patch
 
@@ -83,6 +84,40 @@ class Framework(unittest.TestCase):
         rv.close()
         rv.close()
 
+    def test_close_releases_owned_event_loop(self):
+        """Closing a Zeroconf that started its own loop thread closes that loop.
+
+        Regression test for issue #1589 — without loop.close(), the selector
+        (epoll on Linux) and its self-pipe sockets stay open across each
+        Zeroconf construct/close cycle and the process eventually exhausts
+        its FD limit.
+        """
+        rv = r.Zeroconf(interfaces=["127.0.0.1"])
+        loop = rv.loop
+        assert loop is not None
+        assert loop.is_running()
+        rv.close()
+        assert loop.is_closed()
+
+    @unittest.skipUnless(sys.platform.startswith("linux"), "Requires /proc/<pid>/fd")
+    @unittest.skipUnless(Path(f"/proc/{os.getpid()}/fd").is_dir(), "/proc/<pid>/fd not available")
+    def test_close_does_not_leak_file_descriptors(self):
+        """Tight loops of Zeroconf()/close() do not leak FDs (issue #1589)."""
+        fd_dir = Path(f"/proc/{os.getpid()}/fd")
+
+        def _fd_count() -> int:
+            return sum(1 for _ in fd_dir.iterdir())
+
+        # Warm-up cycle so any one-shot import-time FDs land before measuring.
+        r.Zeroconf(interfaces=["127.0.0.1"]).close()
+        baseline = _fd_count()
+        for _ in range(10):
+            r.Zeroconf(interfaces=["127.0.0.1"]).close()
+        # Allow tiny slack for unrelated FDs the test harness may open
+        # (e.g. coverage), but reject the per-cycle linear growth pattern
+        # the bug produced (~3 FDs per cycle, so >=30 over 10 cycles).
+        assert _fd_count() - baseline < 10
+
     @unittest.skipIf(not has_working_ipv6(), "Requires IPv6")
     @unittest.skipIf(os.environ.get("SKIP_IPV6"), "IPv6 tests disabled")
     def test_launch_and_close_v4_v6(self):
@@ -721,7 +756,7 @@ async def test_multiple_sync_instances_stared_from_async_close():
     await asyncio.sleep(0)
 
 
-def test_shutdown_while_register_in_process():
+def test_shutdown_while_register_in_process(quick_timing: None) -> None:
     """Test we can shutdown while registering a service in another thread."""
 
     # instantiate a zeroconf instance
@@ -753,11 +788,10 @@ def test_shutdown_while_register_in_process():
 
 
 @pytest.mark.asyncio
-@patch("zeroconf._core._STARTUP_TIMEOUT", 0)
 @patch("zeroconf._core.AsyncEngine._async_setup", new_callable=AsyncMock)
 async def test_event_loop_blocked(mock_start):
     """Test we raise NotRunningException when waiting for startup that times out."""
     aiozc = AsyncZeroconf(interfaces=["127.0.0.1"])
     with pytest.raises(NotRunningException):
-        await aiozc.zeroconf.async_wait_for_start()
+        await aiozc.zeroconf.async_wait_for_start(timeout=0)
     assert aiozc.zeroconf.started is False

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/test_dns.py

@@ -76,7 +76,7 @@ class TestDunder(unittest.TestCase):
     def test_dns_question_repr(self):
         question = r.DNSQuestion("irrelevant", const._TYPE_SRV, const._CLASS_IN | const._CLASS_UNIQUE)
         repr(question)
-        assert not question != question
+        assert (question != question) is False
 
     def test_dns_service_repr(self):
         service = r.DNSService(
@@ -112,7 +112,7 @@ class TestDunder(unittest.TestCase):
             addresses=[socket.inet_aton("10.0.1.2")],
         )
 
-        assert not info != info
+        assert (info != info) is False
         repr(info)
 
     def test_service_info_text_properties_not_given(self):

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/test_exceptions.py

@@ -24,7 +24,7 @@ def teardown_module():
 
 
 class Exceptions(unittest.TestCase):
-    browser = None  # type: Zeroconf
+    browser: Zeroconf
 
     @classmethod
     def setUpClass(cls):

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/utils/test_asyncio.py

@@ -123,19 +123,25 @@ async def test_run_coro_with_timeout() -> None:
     """Test running a coroutine with a timeout raises EventLoopBlocked."""
     loop = asyncio.get_event_loop()
     task: asyncio.Task | None = None
+    task_created = asyncio.Event()
 
     async def _saved_sleep_task():
         nonlocal task
-        task = asyncio.create_task(asyncio.sleep(0.2))
-        assert task is not None
+        task = asyncio.create_task(asyncio.sleep(1))
+        task_created.set()
         await task
 
     def _run_in_loop():
-        aioutils.run_coro_with_timeout(_saved_sleep_task(), loop, 0.1)
+        aioutils.run_coro_with_timeout(_saved_sleep_task(), loop, 50)
 
     with pytest.raises(EventLoopBlocked), patch.object(aioutils, "_LOADED_SYSTEM_TIMEOUT", 0.0):
         await loop.run_in_executor(None, _run_in_loop)
 
+    # The outer .result() can raise EventLoopBlocked before the loop
+    # has scheduled the coroutine — wait until the inner task is
+    # created before asserting on it. Use an asyncio.Event so the
+    # wait yields back to the loop instead of blocking it.
+    await asyncio.wait_for(task_created.wait(), 1.0)
     assert task is not None
     # ensure the thread is shutdown
     task.cancel()

{zeroconf-0.147.2 → zeroconf-0.147.4}/tests/utils/test_net.py

@@ -357,7 +357,7 @@ def test_create_sockets_interfaces_all_unicast():
         mock_socket = Mock(spec=socket.socket)
         mock_new_socket.return_value = mock_socket
 
-        listen_socket, respond_sockets = r.create_sockets(
+        listen_socket, _respond_sockets = r.create_sockets(
             interfaces=r.InterfaceChoice.All, unicast=True, ip_version=r.IPVersion.All
         )