gpp-client 26.7.3.dev7__tar.gz → 26.7.3.dev8__tar.gz

{gpp_client-26.7.3.dev7 → gpp_client-26.7.3.dev8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: gpp-client
-Version: 26.7.3.dev7
+Version: 26.7.3.dev8
 Summary: Gemini Program Platform client.
 Project-URL: Homepage, https://github.com/gemini-hlsw/gpp-client
 Project-URL: Source, https://github.com/gemini-hlsw/gpp-client

{gpp_client-26.7.3.dev7 → gpp_client-26.7.3.dev8}/src/gpp_client/domains/scheduler.py

@@ -269,10 +269,9 @@ class SchedulerDomain(BaseDomain):
 
         # Get sequence
         if observations:
-            async with self._rest as client:
-                atom_digest_response = (
-                    await client.get_atom_digests(observations)
-                ).split("\n")
+            atom_digest_response = (
+                await self._rest.get_atom_digests(observations)
+            ).split("\n")
             obs_atoms_mapping = self._parse_atom_digest(atom_digest_response)
         else:
             obs_atoms_mapping = {}
@@ -332,9 +331,8 @@ class SchedulerDomain(BaseDomain):
         aiohttp.ClientError
             For HTTP errors, connection failures, or timeouts.
         """
-        async with self._rest as client:
-            body = await client.get_visibility_changes(since)
-            return parse_visibility_changes(body)
+        body = await self._rest.get_visibility_changes(since)
+        return parse_visibility_changes(body)
 
     async def subscribe_to_calculation_updates(
         self,

{gpp_client-26.7.3.dev7 → gpp_client-26.7.3.dev8}/src/gpp_client/rest/client.py

@@ -21,6 +21,13 @@ class RESTClient:
     REST API client to non-GraphQL requests that help with the function of managers and
     coordinators.
 
+    The ``aiohttp`` session is created on first use and shared by every caller that
+    goes through :meth:`get_session`, so closing it is the job of whoever built the
+    client -- normally ``GPPClient.close()``. Code that was handed a client (the
+    domain classes, for instance) must use :meth:`get_session` and leave the session
+    open: closing it mid-flight aborts other callers' in-progress requests with
+    ``ClientConnectionError("Connector is closed.")``.
+
     Parameters
     ----------
     base_url : str
@@ -93,9 +100,15 @@ class RESTClient:
     async def close(self) -> None:
         """
         Close the session if it exists and is not already closed.
+
+        Call this only on a client you own; see the class docstring. The lock is
+        held so a concurrent :meth:`get_session` cannot hand out the session being
+        closed, and the reference is dropped so the next call builds a fresh one.
         """
-        if self._session and not self._session.closed:
-            await self._session.close()
+        async with self._lock:
+            if self._session and not self._session.closed:
+                await self._session.close()
+            self._session = None
 
     async def __aenter__(self):
         return self