Skip to content

fix : Implement API Rate Limiting Middleware (closes #875)#900

Open
Pratikshya32 wants to merge 1 commit into
JhaSourav07:mainfrom
Pratikshya32:chore-fix-issue-875
Open

fix : Implement API Rate Limiting Middleware (closes #875)#900
Pratikshya32 wants to merge 1 commit into
JhaSourav07:mainfrom
Pratikshya32:chore-fix-issue-875

Conversation

@Pratikshya32
Copy link
Copy Markdown
Contributor

Summary

  • Add Express middleware tracking request IP thresholds in-memory
  • Implement automatic cooldown triggers and 429 status response

Closes #875

@Pratikshya32
Copy link
Copy Markdown
Contributor Author

Hello! I have created a pull request resolving this issue: #900.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 28, 2026

@Pratikshya32 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions github-actions Bot added the needs-details This PR is missing required description details. label May 28, 2026
@github-actions
Copy link
Copy Markdown

👋 Hey @Pratikshya32, it looks like you didn't use our PR template!

The section ## Description is missing from your PR description.

Please update your PR description to include all required sections so we can review this properly:

  • ## Description — What does this PR do? Which issue does it fix?
  • ## Pillar — Which contribution pillar does this fall under?
  • ## Checklist — Have you ticked off the quality checklist?

You can find the full template in CONTRIBUTING.md. Just edit your PR description and the needs-details label will be removed automatically. 🙌

@github-actions github-actions Bot added the status:blocked This PR is blocked due to a failing CI check. label May 28, 2026
@github-actions
Copy link
Copy Markdown

🚨 Hey @Pratikshya32, the CI Pipeline is failing on this PR and it has been marked as status:blocked.

Please fix the issues before this can be reviewed. Here's how:

1. Run checks locally before pushing:

npm run format:check   # Check Prettier formatting
npm run lint           # Run ESLint
npm run typecheck      # TypeScript type check
npm run test           # Run unit tests (Vitest)
npm run build          # Verify production build passes

2. Auto-fix common issues:

npm run format         # Auto-fix formatting with Prettier
npm run lint -- --fix  # Auto-fix lint errors where possible

3. Check the full failure log here:
👉 View CI Run

Once you push a fix and the CI passes, the status:blocked label will be removed automatically. 💪

@Aamod007
Copy link
Copy Markdown
Collaborator

@Pratikshya32 Make PR as per guidelines #87 refer this

@github-actions github-actions Bot added the type:bug Something isn't working as expected label May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-details This PR is missing required description details. status:blocked This PR is blocked due to a failing CI check. type:bug Something isn't working as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Implement API rate-limiting middleware to protect endpoints of commitpulse

2 participants