ThreeMammals / Ocelot / 17938788732
93%

DEFAULT BRANCH: develop
Ran
Jobs 1
Files 309
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 90.766% (+5.0%) from 85.741%
17938788732

push

github

d5e6d9
web-flow 
#585 #1229 #1598 #1915 Rate limiting global configuration (#2294)

* feat: implement and configure global rate limit
* fix: fix nullability at constructor of `DownstreamRoute.cs`
* chore: update docs
* chore: update docs
---
Co-authored-by: Milad.Rv <miladrivandi73@gmail.com>

* Fix EOL
* Code review by @yjorayev
* Code review by @RaynaldM
* Remove business logic from the DownstreamRoute constructor
* Update src/Ocelot/Configuration/Creator/RateLimitOptionsCreator.cs
* Update src/Ocelot/Configuration/File/IRouteUpstream.cs
* Update src/Ocelot/Configuration/File/FileGlobalRateLimit.cs
* Update src/Ocelot/Configuration/File/FileDynamicRoute.cs
* 2nd code review by @RaynaldM
* Bump Ocelot.Testing package to 24.0.2
* Fix naming mistyping of the property of DownstreamRoute
* Review RouteOptions.EnableRateLimiting usage
* Review DownstreamRoute.EnableRateLimiting usage
* Remove RateLimitOptionsBuilder logic in favor of RateLimitOptions initialization via def ctor
* Code review by @raman-m
* Rename the rule's file model for rate limiting by a header feat
* Intro FileRateLimitRule as a base model for all rules
* Methods should not be included in the grouping of routes within the global configuration; instead, they could function as rules
* Create file models for a global rate-limiting feature to define a group based on a key, a std way of grouping used in aggregation
* Get ready to migrate from the DisableRateLimitHeaders property to the EnableHeaders prop
* Implement global configuration for the rate limiting-by-header feature, grouping routes by key
* Rename to ClientHeaderRateLimitingTests
* Refactor ClientHeaderRateLimitingTests
* Fix the bug with the X-Rate-Limit-Limit header value so that it correctly returns the rule's limit value
* Move sequential unit test to the same testing class
* Review the period string parsing in the rule to support milliseconds as a unit and floating-point expressions as values
* Get ready to migrate from the PeriodTimespan pr... (continued)

6301 of 6942 relevant lines covered (90.77%)

4679.03 hits per line

Subprojects
ID Flag name Job ID Ran Files Coverage
1 /jobs/171876648 17938788732.1 309
90.77
 GitHub Action Run
Source Files on build 17938788732