tailcall icon indicating copy to clipboard operation
tailcall copied to clipboard

feat: adding rate limiter at field level

Open shashitnak opened this issue 6 months ago • 2 comments

Summary:
Adding the functionality for Rate Limiting at the field level

Issue Reference(s):

Build & Testing:

  • [x] I ran cargo test successfully.
  • [x] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

  • [ ] I have added relevant unit & integration tests.
  • [ ] I have updated the documentation accordingly.
  • [x] I have performed a self-review of my code.

shashitnak avatar Dec 28 '23 08:12 shashitnak

Action required: PR inactive for 2 days. Status update or closure in 5 days.

github-actions[bot] avatar Dec 31 '23 06:12 github-actions[bot]

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (036bc1d) 88.25% compared to head (898a447) 87.96%. Report is 21 commits behind head on main.

:exclamation: Current head 898a447 differs from pull request most recent head f4297e1. Consider uploading reports for the commit f4297e1 to get more accurate results

Files Patch % Lines
src/blueprint/definitions.rs 55.55% 4 Missing :warning:
src/config/from_document.rs 93.22% 4 Missing :warning:
src/rate_limiter/rate_limiter.rs 96.07% 4 Missing :warning:
src/blueprint/blueprint.rs 90.00% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #818      +/-   ##
==========================================
- Coverage   88.25%   87.96%   -0.30%     
==========================================
  Files         105      107       +2     
  Lines       11578    11736     +158     
==========================================
+ Hits        10218    10323     +105     
- Misses       1360     1413      +53     

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

codecov[bot] avatar Jan 02 '24 03:01 codecov[bot]

Action required: PR inactive for 2 days. Status update or closure in 5 days.

github-actions[bot] avatar Jan 06 '24 16:01 github-actions[bot]

Action required: PR inactive for 2 days. Status update or closure in 5 days.

github-actions[bot] avatar Jan 11 '24 18:01 github-actions[bot]

Can you resolve the conflicts

ssddOnTop avatar Jan 12 '24 08:01 ssddOnTop

Action required: PR inactive for 2 days. Status update or closure in 5 days.

github-actions[bot] avatar Feb 09 '24 06:02 github-actions[bot]

Changing the scope of rate limits to upstream only and not on entities.

tusharmath avatar Feb 09 '24 06:02 tusharmath