Skip to content

Handle malformed dashboard URL encodings#65

Open
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/handle-bad-dashboard-url-encoding
Open

Handle malformed dashboard URL encodings#65
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/handle-bad-dashboard-url-encoding

Conversation

@w287346141
Copy link
Copy Markdown

Summary

  • handle malformed percent-encoding in dashboard file-read paths with a 400 instead of an uncaught exception
  • handle malformed percent-encoding in dashboard session names with a 400 instead of an uncaught exception
  • add focused regression tests for both decode paths

Validation

  • npm.cmd test -- --run tests/dashboard-api-decode.test.ts tests/server-dashboard.test.ts
  • npm.cmd run lint
  • npm.cmd run typecheck
  • git diff --check
  • push hook ran npm.cmd run verify

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