Skip to content

Use POST for Discord JWT request#14521

Merged
raymondjacobson merged 1 commit into
mainfrom
codex/discord-jwt-post-client
Jul 13, 2026
Merged

Use POST for Discord JWT request#14521
raymondjacobson merged 1 commit into
mainfrom
codex/discord-jwt-post-client

Conversation

@raymondjacobson

Copy link
Copy Markdown
Member

Summary

  • send the Discord JWT request as POST /request_discord_code with a JSON body instead of query params
  • encode the Discord OAuth state payload before appending it to the OAuth URL

Context

PR #14520 fixed the mixed-content issue by switching the production Discord bot URL to HTTPS. This follow-up moves signature/message transport out of the URL and into a JSON request body, avoiding URL length and query logging concerns.

Companion PRs

Rollout

Merge/deploy after the bot supports POST. Recommended order:

  1. Create/set discord-bot-JWT_SECRET in GCP Secret Manager.
  2. Merge/apply the k8s secret wiring PR.
  3. Merge/deploy the bot POST-support PR.
  4. Merge/deploy this apps PR.

Test plan

  • git diff --check

Could not run local Vitest because this worktree has no installed node_modules, so vitest is unavailable (sh: vitest: command not found).

@changeset-bot

changeset-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e636860

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@raymondjacobson raymondjacobson merged commit 280f000 into main Jul 13, 2026
13 checks passed
@raymondjacobson raymondjacobson deleted the codex/discord-jwt-post-client branch July 13, 2026 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant