From ed21c7634ab5464f41d0ecdbbf3cc425d443a587 Mon Sep 17 00:00:00 2001 From: Markus Hilger Date: Fri, 14 Aug 2026 14:07:49 +0200 Subject: [PATCH] Drop a guard that never ran and would not have worked __init__ opened by checking for an initialized attribute, meaning it had been handed a session someone else was establishing. That attribute is only ever set further down in the same method, so the check cannot be true, and the port lost the return that made it work upstream. Waiting for someone else's login is done in __new__ now, so this is dead code claiming to protect something. --- confluent_server/aiohmi/ipmi/private/session.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/confluent_server/aiohmi/ipmi/private/session.py b/confluent_server/aiohmi/ipmi/private/session.py index 23c42003..5ce8e9e7 100644 --- a/confluent_server/aiohmi/ipmi/private/session.py +++ b/confluent_server/aiohmi/ipmi/private/session.py @@ -506,10 +506,6 @@ class Session(object): kg=None, privlevel=None, keepalive=True): - if hasattr(self, 'initialized'): - # new found an existing session, do not corrupt it - while self.logging and not self.broken: - await Session.wait_for_rsp() self.awaitingresponse = False self.lastresponse = None self.atomicop = asyncio.Lock()