fix(http): test acquire_scoped_lock's bool, not the always-truthy tuple
acquire_scoped_lock returns (acquired, existing_record); the old 'if not acquire_scoped_lock(...)' tested the tuple, which is always truthy, so the 'port in use by another profile' pre-check never fired and a conflict surfaced as a generic bind failure. Unpack and test the first element, matching gateway/platforms/base.py's canonical usage. Bump VERSION / plugin.yaml to 0.1.3.
This commit is contained in:
1 parent
5b78e1566f
commit
8657e6afc6
3 files changed
+7
-3
No files matched your search
@@ -199,7 +199,11 @@ class HttpServer:
|
||||
from gateway.status import acquire_scoped_lock
|
||||
|
||||
lock_key = f"http:{host}:{port}"
|
||||
if not acquire_scoped_lock("iris", lock_key):
|
||||
# acquire_scoped_lock returns (acquired, existing_record); the
|
||||
# tuple is always truthy, so test the first element (matching
|
||||
# gateway/platforms/base.py's canonical usage).
|
||||
acquired, _ = acquire_scoped_lock("iris", lock_key)
|
||||
if not acquired:
|
||||
logger.warning(
|
||||
"iris: HTTP port %s:%s in use by another profile; server disabled",
|
||||
host,
|
||||
|
||||
Reference in new issue
Block a user