Skip to content

Require signed auth for feed writes#25

Open
lawyered0 wants to merge 1 commit intoJejuNetwork:mainfrom
lawyered0:codex/feed-auth
Open

Require signed auth for feed writes#25
lawyered0 wants to merge 1 commit intoJejuNetwork:mainfrom
lawyered0:codex/feed-auth

Conversation

@lawyered0
Copy link

Summary:

  • Require signed auth headers for write operations in /api/feed
  • Add signed-request helpers in the Factory web API client
  • Update Farcaster feed mutations to sign requests before posting

Security:
Previously, clients could spoof x-wallet-address and perform write actions on behalf of another user. This change requires a wallet signature for feed write endpoints.

Testing:

  • Not run (auth plumbing changes only)

@lawyered0
Copy link
Author

Why this should merge:

  • Security: prior to this change, any client could spoof and perform write actions on .
  • This change preserves all read paths and only hardens write endpoints.
  • Client signing uses the existing message format already validated by the backend.

Repro (before fix):

  • Send with any header to like on behalf of that address.

Now: write actions require a signed header set.

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