ThreeMammals / Ocelot

94%

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
• 2339-fix-tests
• 24.0.1-Ocelot.Testing
• 24.1.0-beta
• 24.1.0-beta.1
• 24.1.0-beta.2
• Burgyn-patch-1
• allow-cache-filtering-based-on-reseponse-statuscode
• allow-empty-required-claim-value
• bug-2299
• bug-2304-PollKube
• bug/2346-query-param-id-regression
• feat-2319
• feat-2320-http-handler
• feat-2330
• feat_enableHttp2
• feat_rm_newtonsoft
• feature/1658---Global-UpstreamHeaderTransform-settings-in-GlobalConfiguration-section
• feature/2316-auth-opts
• feature/2338-qos-opts
• feature/651-merge-custom-properties
• feature/842-authenticationoptions-in-globalconfig
• feature/add-feature-for-ocelot-pipeline-configuration
• feature/add-global-rate-limit
• feature/downstream-service-name
• fix/aggregate-routekeys-nandan
• fix/roundrobin-null-service-name
• harris-patch-2
• issues-1869
• issues-930
• main
• merge-conflicts
• patch-1
• patch/24.0.1
• pre-release-24.1
• raman-m-patch-1
• raman/24.0
• release/24.0
• release/24.1
• release/24.1-dev
• service-discovery-path-prefix
• user/carl-aggregate-methods

Build # 21217359297

Build Type

push

github

Committed by web-flow

Commit Message

#2346 Fix problem with downstream URLs where query parameter names get messed up if they contain a placeholder that has a non-empty value (#2351)

* Update DownstreamUrlCreatorMiddleware.cs
* Update DownstreamUrlCreatorMiddlewareTests.cs
* Added more test Cases and refactor RemoveQueryStringParametersThatHaveBeenUsedInTemplate Logic
* Revert "Added more test Cases and refactor RemoveQueryStringParametersThatHaveBeenUsedInTemplate Logic"
  This reverts commit 21661a4ee.
* Replaced the regex-based logic in RemoveQueryStringParametersThatHaveBeenUsedInTemplate
* Revert "Replaced the regex-based logic in RemoveQueryStringParametersThatHaveBeenUsedInTemplate"
  This reverts commit 40358588b.
* Replaced the regex-based logic in RemoveQueryStringParametersThatHaveBeenUsedInTemplate
* Update RoutingWithQueryStringTests.cs
* Testing the "Catch All Query String" feat to make sure there is no regression
* Refactor algorithm to take full advantage of QueryHelpers
* Code review by @raman-m ~ improved testing
* Roll back the status badge targets to develop
---------
Co-authored-by: Raman Maksimchuk <dotnet044@gmail.com>

Run Details

6512 of 6964 relevant lines covered (93.51%)

5711.58 hits per line