-
-
Notifications
You must be signed in to change notification settings - Fork 52.8k
Description
Summary
After upgrading to 2026.2.25 (via openclaw doctor), all group messages are silently dropped
when groupPolicy: "allowlist" is configured without an explicit allowFrom field.
No INFO-level log is emitted — the only indication is a verbose-level log that most users
won't see.
Environment
- OpenClaw: 2026.2.25
- Trigger:
openclaw doctorswitched LaunchAgent fromentry.jstoindex.js
Steps to Reproduce
- Configure
groupPolicy: "allowlist"withoutallowFrom(valid config in pre-2026.2.25) - Run
openclaw doctor - Send a group message to the bot
Expected Behavior
Either:
- (a) The message is processed (backwards-compatible behavior), OR
- (b) An INFO-level log is emitted explaining why the message was dropped
Actual Behavior
Message is silently dropped. pending_update_count decreases (update consumed),
watermark advances, but no log appears at INFO level.
Root Cause
evaluateTelegramGroupPolicyAccess returns group-policy-allowlist-empty when
allowEmptyAllowlistEntries: false (hardcoded) and effectiveGroupAllow.hasEntries: false
(result of normalizeAllowFrom(undefined)).
shouldSkipGroupMessage handles this reason with logVerbose only.
Fix Applied
Added "allowFrom": ["*"] to group config. Works correctly after this change.
Suggested Improvements
- Migration warning: On startup, if
groupPolicy: "allowlist"is detected without
allowFrom, emit a WARN-level log explaining the new requirement - Better logging: Upgrade
group-policy-allowlist-emptyrejection fromlogVerbose
tologInfoorlogWarn - CHANGELOG clarity: The security(telegram): fail closed group allowlist against DM pairing store #25988 entry is accurate but doesn't mention that existing
configs withoutallowFromwill silently break