Skip to content

fix: block alchemy_ prefixed methods#2990

Merged
grothem merged 1 commit into
mainfrom
fix/aav-14-json-rpc-method-injection-via-api-rpc-proxy
May 22, 2026
Merged

fix: block alchemy_ prefixed methods#2990
grothem merged 1 commit into
mainfrom
fix/aav-14-json-rpc-method-injection-via-api-rpc-proxy

Conversation

@sammdec
Copy link
Copy Markdown
Contributor

@sammdec sammdec commented May 21, 2026

Summary

  • Reject any /api/rpc-proxy request whose method is missing, non-string, or starts with alchemy_, so callers can't use the proxy to hit Alchemy-only RPCs

Test plan

  • curl -X POST .../api/rpc-proxy -d '{"chainId":1,"method":"alchemy_getTokenBalances","params":[...]}' returns 400 {"error":"Method not allowed"}
  • curl -X POST .../api/rpc-proxy -d '{"chainId":1,"method":"eth_blockNumber","params":[]}' still returns a block number

@odin-mjolnir
Copy link
Copy Markdown

odin-mjolnir Bot commented May 21, 2026

Linked Findings

AAV-14 JSON-RPC Method Injection via /api/rpc-proxy
Severity Medium
Status Mitigating
Discovered 15 May 2026

View in Odin

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
interface Ready Ready Preview, Comment May 21, 2026 1:08pm

Request Review

@github-actions
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

📦 Next.js Bundle Analysis for aave-ui

This analysis was generated by the Next.js Bundle Analysis action. 🤖

⚠️ Global Bundle Size Increased

Page Size (compressed)
global 1.15 MB (🟡 +77 B)
Details

The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!

@sammdec sammdec requested a review from mgrabina May 21, 2026 14:35
@grothem grothem merged commit 5f53e44 into main May 22, 2026
27 checks passed
@grothem grothem deleted the fix/aav-14-json-rpc-method-injection-via-api-rpc-proxy branch May 22, 2026 14:40
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.

3 participants