Nandan Parmar

I can confirm the Context/memo scenario is fixed by #4738, but the Signals variant still repros on 10.27.1. ### Repro (no memo, Signals only) - StackBlitz (drop-in) create a signal...

Okay @raman-m Thenk you for the detailed feedback! Got it about the EOL part, I’ll leave that as is for now, and will check locally again regarding EOL, and will...

@raman-m I’ve added the acceptance test for #2248 covering the RouteKeys array case. It builds fine on my local setup for both net8.0 and net9.0. Please let me know if...

@raman-m sorry man for the trouble, especially fixing the EOL issues, cleaning up the branch, and rebasing everything. thanks man appreciate the effort you put in to make it right,...

@raman-m The new test `Should_expand_jsonpath_array_into_multiple_parameterized_calls ` checks that when a downstream service returns an array for example [{"userId":1},{"userId":2}], so that ocelot expands those values into multiple parameterized calls to /users/{userId}...

@raman-m I have added a new unit test inside MultiplexingMiddlewareTests that exercises the `ProcessRouteWithComplexAggregation` path in [MultiplexingMiddleware](https://coveralls.io/builds/76181663/source?filename=Ocelot%2FMultiplexer%2FMultiplexingMiddleware.cs#L261)

Added two additional unit tests in MultiplexingMiddlewareTests to cover the missing branches in `ProcessRoutesWithRouteKeysAsync` and `MapAsync`. @ggnaegi Let me know once you review the code

Fixed the mistake, updated the unit test which updated the code coverage

I added another small commit to adjust the new tests, reason being that AppVeyor CI doesn’t have SQL Server available, so the first version of the tests failed there. With...

Hi, just checking in to see if there’s any feedback or change you’d like here. I had eariler checked locally with SQL Server the fix works for both sync and...