fix(hooks): return 200 instead of 202 for webhook responses#28204
Merged
shakkernerd merged 3 commits intoopenclaw:mainfrom Mar 3, 2026
Merged
fix(hooks): return 200 instead of 202 for webhook responses#28204shakkernerd merged 3 commits intoopenclaw:mainfrom
shakkernerd merged 3 commits intoopenclaw:mainfrom
Conversation
Contributor
Greptile SummaryChanged webhook endpoint response status from
Confidence Score: 4/5
Last reviewed commit: 426b887 |
Contributor
Additional Comments (1)
Prompt To Fix With AIThis is a comment left during a code review.
Path: docs/automation/webhook.md
Line: 162
Comment:
Update to reflect new behavior - should be `200` instead of `202`
```suggestion
- `200` for `/hooks/agent` (async run started)
```
How can I resolve this? If you propose a fix, please make it concise. |
shakkernerd
added a commit
to Glucksberg/OpenClaw
that referenced
this pull request
Mar 3, 2026
3aaf531 to
c85b6c0
Compare
|
🤖 We're reviewing this PR with Aisle We're running a security check on the changes in this PR now. This usually takes a few minutes. ⌛ Progress:
Latest run failed. Keeping previous successful results. Trace ID: Last updated on: 2026-03-03T00:14:03Z Latest run failed. Keeping previous successful results. Trace ID: Last updated on: 2026-03-03T00:25:57Z |
shakkernerd
added a commit
to Glucksberg/OpenClaw
that referenced
this pull request
Mar 3, 2026
c85b6c0 to
b507a3c
Compare
b507a3c to
796d9ab
Compare
Member
|
Merged via squash. Thanks @Glucksberg! |
|
🤖 We're reviewing this PR with Aisle We're running a security check on the changes in this PR now. This usually takes a few minutes. ⌛ Progress:
Latest run failed. Keeping previous successful results. Trace ID: Last updated on: 2026-03-03T00:21:45Z |
dawi369
pushed a commit
to dawi369/davis
that referenced
this pull request
Mar 3, 2026
…#28204) * fix(hooks): return 200 instead of 202 for webhook responses (openclaw#22036) * docs(webhook): document 200 status for hooks agent * chore(changelog): add webhook ack note openclaw#28204 thanks @Glucksberg --------- Co-authored-by: Shakker <shakkerdroid@gmail.com>
OWALabuy
pushed a commit
to kcinzgg/openclaw
that referenced
this pull request
Mar 4, 2026
…#28204) * fix(hooks): return 200 instead of 202 for webhook responses (openclaw#22036) * docs(webhook): document 200 status for hooks agent * chore(changelog): add webhook ack note openclaw#28204 thanks @Glucksberg --------- Co-authored-by: Shakker <shakkerdroid@gmail.com>
AytuncYildizli
pushed a commit
to AytuncYildizli/openclaw
that referenced
this pull request
Mar 4, 2026
…#28204) * fix(hooks): return 200 instead of 202 for webhook responses (openclaw#22036) * docs(webhook): document 200 status for hooks agent * chore(changelog): add webhook ack note openclaw#28204 thanks @Glucksberg --------- Co-authored-by: Shakker <shakkerdroid@gmail.com>
zooqueen
pushed a commit
to hanzoai/bot
that referenced
this pull request
Mar 6, 2026
…#28204) * fix(hooks): return 200 instead of 202 for webhook responses (openclaw#22036) * docs(webhook): document 200 status for hooks agent * chore(changelog): add webhook ack note openclaw#28204 thanks @Glucksberg --------- Co-authored-by: Shakker <shakkerdroid@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
202 Acceptedto200 OKfor successfully dispatched agent hook requests200status and bounces with550on any other codeCloses #22036
Test plan
server.hooks.test.tstests updated and passing (7/7)🤖 Generated with Claude Code