Skip to content

Commit 36aee31

Browse files
Turns out _send_buffered_messages was only used in one place
1 parent 8d9161e commit 36aee31

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

src/replit_river/session.py

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -152,24 +152,14 @@ async def replace_with_new_websocket(
152152
if new_ws.id != old_ws_id:
153153
await old_wrapper.close()
154154
self._ws_wrapper = WebsocketWrapper(new_ws)
155-
await self._send_buffered_messages(new_ws)
156155

157-
async def _get_current_time(self) -> float:
158-
return asyncio.get_event_loop().time()
159-
160-
def _reset_session_close_countdown(self) -> None:
161-
self._heartbeat_misses = 0
162-
self._close_session_after_time_secs = None
163-
164-
async def _send_buffered_messages(
165-
self, websocket: websockets.WebSocketCommonProtocol
166-
) -> None:
156+
# Send buffered messages to the new ws
167157
buffered_messages = list(self._buffer.buffer)
168158
for msg in buffered_messages:
169159
try:
170160
await self._send_transport_message(
171161
msg,
172-
websocket,
162+
new_ws,
173163
)
174164
except WebsocketClosedException:
175165
logger.info(
@@ -180,6 +170,13 @@ async def _send_buffered_messages(
180170
logger.exception("Error while sending buffered messages")
181171
break
182172

173+
async def _get_current_time(self) -> float:
174+
return asyncio.get_event_loop().time()
175+
176+
def _reset_session_close_countdown(self) -> None:
177+
self._heartbeat_misses = 0
178+
self._close_session_after_time_secs = None
179+
183180
async def _send_transport_message(
184181
self,
185182
msg: TransportMessage,

0 commit comments

Comments
 (0)