ThreeMammals / Ocelot

91%

LAST BUILD ON BRANCH develop

branch: develop

CHANGE BRANCH

x

Reset

• develop
• 2080-adds-parameters-in-this-case-those-of-polly-v8-to-fine-tune-circuit-breaker-behavior
• 2168-watchkube
• 2254-coveralls
• 24.0.1-Ocelot.Testing
• bug-2299
• feature/1658---Global-UpstreamHeaderTransform-settings-in-GlobalConfiguration-section
• feature/842-authenticationoptions-in-globalconfig
• feature/add-global-rate-limit
• harris-patch-2
• issues-1869
• issues-930
• main
• patch-1
• patch/24.0.1
• raman/24.0
• release/24.0
• service-discovery-path-prefix

Build # 17938788732

Build Type

push

github

Specific Base d5e6d9

Committed by web-flow

Commit Message

#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)

Run Details

6301 of 6942 relevant lines covered (90.77%)

4679.03 hits per line