brooktrails/pr-reviewer
Watch
1
0
Fork
You've already forked pr-reviewer
 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

feat(webhook): log signature rejections before returning 401 #76

Merged
rcsheets merged 1 commit from feat/webhook-signature-logging into main 2026-07-22 03:19:15 +00:00
Conversation 1 Commits 1 Files changed 1 +3
rcsheets commented 2026-07-22 03:18:08 +00:00
Owner
Copy link

Emit a Warn log at the VerifySignature call site so client 401s
(missing vs bad signature) are visible in the server log, with
remote address and whether a signature header was present.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

Emit a Warn log at the VerifySignature call site so client 401s (missing vs bad signature) are visible in the server log, with remote address and whether a signature header was present. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
feat(webhook): log signature rejections before returning 401
Some checks failed
ci / check (pull_request) Has been cancelled
Details
5545ca2bae 
Emit a Warn log at the VerifySignature call site so client 401s
(missing vs bad signature) are visible in the server log, with
remote address and whether a signature header was present.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
pr-reviewer-bot commented 2026-07-22 03:18:08 +00:00
Collaborator
Copy link

Automated review by pr-reviewer v0.35.0 | Safety Check | Mistral Small | tracking id r-603670-a0d796
This is an AI-generated review and may contain mistakes.

Status: ❌ Failed

Review failed. Tracking id r-603670-a0d796 — see logs for details.

Comment @pr-reviewer-bot retry to try again.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.35.0 | Safety Check | Mistral Small | tracking id `r-603670-a0d796`* *This is an AI-generated review and may contain mistakes.* **Status:** ❌ Failed --- Review failed. Tracking id `r-603670-a0d796` — see logs for details. Comment `@pr-reviewer-bot retry` to try again.
rcsheets force-pushed feat/webhook-signature-logging from 5545ca2bae
Some checks failed
ci / check (pull_request) Has been cancelled
Details
to a9258ccdfd
All checks were successful
ci / check (pull_request) Successful in 45s
Details
2026-07-22 03:18:14 +00:00 Compare
rcsheets deleted branch feat/webhook-signature-logging 2026-07-22 03:19:15 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
brooktrails/pr-reviewer!76
No description provided.