openclaw - 💡(How to fix) Fix [BUG] LINE channel: dmPolicy:"open" silently drops all incoming messages [1 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
openclaw/openclaw#78446Fetched 2026-05-07 03:36:50
View on GitHub
Comments
1
Participants
2
Timeline
2
Reactions
2
Timeline (top)
closed ×1commented ×1

Error Message

  • Bug is invisible: webhook returns 200 OK, no error logs
  1. Message never reaches agent; no error logged

Root Cause

mergeDmAllowFromSources() in allow-from-L4GtxkFC.js does not add a wildcard "*" entry when dmPolicy is "open". This results in an empty allowlist, causing isSenderIdAllowed() to return false silently. The message is dropped by shouldProcessLineEvent().

Code Example

function mergeDmAllowFromSources(params) {
    const storeEntries = params.dmPolicy === "allowlist" || params.dmPolicy === "open" ? [] : params.storeAllowFrom ?? [];
    return normalizeStringEntries([...params.allowFrom ?? [], ...storeEntries]);
}

---

function mergeDmAllowFromSources(params) {
    const storeEntries = params.dmPolicy === "allowlist" || params.dmPolicy === "open" ? [] : params.storeAllowFrom ?? [];
    const entries = [...params.allowFrom ?? [], ...storeEntries];
    if (params.dmPolicy === "open") entries.push("*");
    return normalizeStringEntries(entries);
}
RAW_BUFFERClick to expand / collapse

Describe the bug

When dmPolicy: "open" is configured for the LINE channel, all incoming messages are silently dropped. The webhook returns 200 OK, but messages never reach the agent session. No errors are logged.

Root Cause

mergeDmAllowFromSources() in allow-from-L4GtxkFC.js does not add a wildcard "*" entry when dmPolicy is "open". This results in an empty allowlist, causing isSenderIdAllowed() to return false silently. The message is dropped by shouldProcessLineEvent().

Code

File: allow-from-L4GtxkFC.jsmergeDmAllowFromSources

Before (broken):

function mergeDmAllowFromSources(params) {
    const storeEntries = params.dmPolicy === "allowlist" || params.dmPolicy === "open" ? [] : params.storeAllowFrom ?? [];
    return normalizeStringEntries([...params.allowFrom ?? [], ...storeEntries]);
}

After (fixed):

function mergeDmAllowFromSources(params) {
    const storeEntries = params.dmPolicy === "allowlist" || params.dmPolicy === "open" ? [] : params.storeAllowFrom ?? [];
    const entries = [...params.allowFrom ?? [], ...storeEntries];
    if (params.dmPolicy === "open") entries.push("*");
    return normalizeStringEntries(entries);
}

Impact

  • LINE users with dmPolicy: "open" cannot receive any messages
  • Bug is invisible: webhook returns 200 OK, no error logs

To Reproduce

  1. Configure LINE channel with dmPolicy: "open"
  2. Send a message from LINE to the bot
  3. Webhook receives event (200 OK)
  4. Message never reaches agent; no error logged

Expected behavior

dmPolicy: "open" should allow all incoming messages from LINE.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

FAQ

Expected behavior

dmPolicy: "open" should allow all incoming messages from LINE.

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

openclaw - 💡(How to fix) Fix [BUG] LINE channel: dmPolicy:"open" silently drops all incoming messages [1 comments, 2 participants]