Skip to content

#270 Pairing endpoints are missing from the rate-limit table - #331

Open
bosedegift79-creator wants to merge 1 commit into
blockchain-maxis:mainfrom
bosedegift79-creator:fix/270-pairing-endpoints-rate-limit
Open

#270 Pairing endpoints are missing from the rate-limit table#331
bosedegift79-creator wants to merge 1 commit into
blockchain-maxis:mainfrom
bosedegift79-creator:fix/270-pairing-endpoints-rate-limit

Conversation

@bosedegift79-creator

Copy link
Copy Markdown

Summary

Added cli:pair:start and cli:pair:complete rate limit entries to LIMITS in apps/web/lib/rate-limit-http.ts.

  • Rate Limits: Configured to 12 requests per minute (matching sep10), protecting unauthenticated CLI pairing initialization and ed25519 signature verification steps against signature flooding and code minting attacks.
  • Unit Tests: Created apps/web/lib/rate-limit-http.test.ts to verify bucket configurations and ensure 429 responses with retry-after headers are returned when budgets are exceeded.

Closes #270

@netlify

netlify Bot commented Aug 30, 2026

Copy link
Copy Markdown

Deploy Preview for stellar-signet ready!

Name Link
🔨 Latest commit aeebf64
🔍 Latest deploy log https://app.netlify.com/projects/stellar-signet/deploys/6a940259042d0e00083918c7
😎 Deploy Preview https://deploy-preview-331--stellar-signet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@bosedegift79-creator Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

@Michvista is attempting to deploy a commit to the blockchainmaxis-8449's projects Team on Vercel.

A member of the Team first needs to authorize it.

@bosedegift79-creator
bosedegift79-creator force-pushed the fix/270-pairing-endpoints-rate-limit branch from 2bb44a9 to aeebf64 Compare August 30, 2026 10:13
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.

Pairing endpoints are missing from the rate-limit table

1 participant