fix(poller): 401 auth failures loud + key hot-reload#20
Conversation
A rotated key returned a 401 error BODY with curl exit 0, which parsed as an empty room and silently muted the poller for 16h (Jul 3). The fetch now captures the HTTP status; on 401/403 it logs AUTH FAIL and re-reads poller.api_key from config so a rotation self-heals in one cycle. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0c52905f4b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| case "$(printf '%s' "$msg_from" | tr '[:upper:]' '[:lower:]')" in | ||
| @claudemb|claudemb) continue ;; |
There was a problem hiding this comment.
Preserve peer wakes for self-authored mentions
When this poller sees a room message authored by @claudeMB that mentions a configured peer such as @claudemm, this continue runs before the wake-on-mention.sh call below, so the peer no longer receives the immediate /wake nudge and must wait for its own polling cycle. If the intent is only to avoid local self-notifications, the mention hook should still run before skipping the local append/count/wake path.
Useful? React with 👍 / 👎.
A rotated key 401'd with curl exit 0 and parsed as an empty room — the poller went silently mute for 16h (Jul 3). Now the fetch reads the HTTP status, logs AUTH FAIL on 401/403, and hot-reloads the key from config so rotation self-heals.
🤖 Generated with Claude Code