tailcall
tailcall copied to clipboard
feat: adding rate limiter at field level
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.
Action required: PR inactive for 2 days. Status update or closure in 5 days.
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.
Action required: PR inactive for 2 days. Status update or closure in 5 days.
Action required: PR inactive for 2 days. Status update or closure in 5 days.
Can you resolve the conflicts
Action required: PR inactive for 2 days. Status update or closure in 5 days.
Changing the scope of rate limits to upstream only and not on entities.