mirror/nanobot
1
0
Fork 0
mirror of https://github.com/HKUDS/nanobot.git synced 2026-05-20 00:22:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

simplify(pairing): unify allow_list lookup in BaseChannel.is_allowed()

Browse Source 
Merge the three-branch dict lookup (allow_from key check, allowFrom
fallback, getattr) into a single `or` chain. Same semantics, less
branching.
This commit is contained in:
chengyongru 2026-05-15 14:42:31 +08:00 committed by Xubin Ren
parent ada11b38c4
commit cab4bdbf33
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
nanobot/channels/base.py
View File

@ -184,12 +184,9 @@ class BaseChannel(ABC):
def is_allowed(self, sender_id: str) -> bool: def is_allowed(self, sender_id: str) -> bool:
"""Check sender permission: star > allowlist > pairing store > deny.""" """Check sender permission: star > allowlist > pairing store > deny."""
if isinstance(self.config, dict): if isinstance(self.config, dict):
if "allow_from" in self.config: allow_list = self.config.get("allow_from") or self.config.get("allowFrom") or []
allow_list = self.config.get("allow_from") or []
else: else:
allow_list = self.config.get("allowFrom", []) or [] allow_list = getattr(self.config, "allow_from", None) or []
else:
allow_list = getattr(self.config, "allow_from", []) or []
if "*" in allow_list: if "*" in allow_list:
return True return True
# allowFrom entries are opaque tokens — must match exactly. # allowFrom entries are opaque tokens — must match exactly.