Skip to content

[codex] Improve REST 4xx error responses - #56

Merged
jillesvangurp merged 1 commit into
mainfrom
codex/improve-4xx-errors
Jun 11, 2026
Merged

[codex] Improve REST 4xx error responses#56
jillesvangurp merged 1 commit into
mainfrom
codex/improve-4xx-errors

Conversation

@jillesvangurp

Copy link
Copy Markdown
Contributor

Summary

  • add actionable details to the REST error envelope and generated OpenAPI type
  • return JSON 400 details for invalid bodies and path parameters
  • return JSON 403 details for permission and ownership rule failures
  • return JSON 404/405 details for missing paths and unsupported methods
  • document the REST error envelope and auth-related 4xx behavior

Validation

  • just bootstrap
  • just generate
  • go test ./internal/httpapi/handlers ./internal/auth ./cmd/hub
  • just check passed generation, module, vulnerability, build/static, and unit stages; Docker-backed integration tests failed locally because rootless Docker is unavailable

@jillesvangurp
jillesvangurp merged commit 7152d8f into main Jun 11, 2026
2 checks passed
@jillesvangurp
jillesvangurp deleted the codex/improve-4xx-errors branch June 11, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant