flagsmith icon indicating copy to clipboard operation
flagsmith copied to clipboard

infra(prod): Throttle segment endpoints

Open khvn26 opened this issue 1 week ago • 3 comments

Thanks for submitting a PR! Please check the boxes below:

  • [ ] I have added information to docs/ if required so people know about the feature!
  • [x] I have filled in the "Changes" section below?
  • [x] I have filled in the "How did you test this code" section below?
  • [x] I have used a Conventional Commit title for this Pull Request

Changes

This adds a default 30/minute throttle for /api/v1/segments/* endpoints, and throttles them at a rate of 5/minute for SaaS prod.

How did you test this code?

N/A

khvn26 avatar Dec 11 '25 17:12 khvn26

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

3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
docs Ignored Ignored Dec 11, 2025 5:56pm
flagsmith-frontend-preview Ignored Ignored Dec 11, 2025 5:56pm
flagsmith-frontend-staging Ignored Ignored Dec 11, 2025 5:56pm

vercel[bot] avatar Dec 11 '25 17:12 vercel[bot]

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-6400 Finished :white_check_mark: Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-6400 Finished :white_check_mark: Results :white_check_mark:
ghcr.io/flagsmith/flagsmith-e2e:pr-6400 Finished :white_check_mark: Skipped
ghcr.io/flagsmith/flagsmith-api:pr-6400 Finished :white_check_mark: Results :white_check_mark:
ghcr.io/flagsmith/flagsmith:pr-6400 Finished :white_check_mark: Results :white_check_mark:
ghcr.io/flagsmith/flagsmith-private-cloud:pr-6400 Finished :white_check_mark: Results :white_check_mark:

github-actions[bot] avatar Dec 11 '25 17:12 github-actions[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 98.06%. Comparing base (9e584f1) to head (08a9e41). :warning: Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6400   +/-   ##
=======================================
  Coverage   98.06%   98.06%           
=======================================
  Files        1289     1289           
  Lines       46293    46294    +1     
=======================================
+ Hits        45395    45396    +1     
  Misses        898      898           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Dec 11 '25 18:12 codecov[bot]

@khvn26 @gagantrivedi is this superseded now by #6401 ?

matthewelwell avatar Dec 15 '25 10:12 matthewelwell

Yes, closing for now.

khvn26 avatar Dec 15 '25 10:12 khvn26