Skip to content

[codex] Remove legacy device authentication#1156

Draft
gemcoder21 wants to merge 1 commit into
mainfrom
codex/remove-legacy-device-auth
Draft

[codex] Remove legacy device authentication#1156
gemcoder21 wants to merge 1 commit into
mainfrom
codex/remove-legacy-device-auth

Conversation

@gemcoder21
Copy link
Copy Markdown
Contributor

Summary

  • remove support for legacy x-device-* and x-wallet-id request authentication headers
  • require wallet-scoped device auth to use the Authorization: Gem ... payload wallet id
  • simplify Gem auth parsing and update device/wallet authentication docs

Validation

  • just format
  • cargo clippy -p gem_auth -- -D warnings
  • cargo clippy -p storage -- -D warnings
  • cargo clippy -p api -- -D warnings
  • just test storage
  • just test gem_auth
  • just test api

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes legacy device authentication mechanisms, including individual x-device-* and x-wallet-id headers, in favor of a single unified Authorization: Gem ... header. It cleans up the authentication logic across the API, simplifies signature verification, removes the legacy fallback code, and updates the corresponding documentation.

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