peter-csala

It can't be implemented in a nice way without introducing a breaking change.

Just to clarify: which Polly version are we talking about? The `RateLimiterRejectedException` was introduced in the V8 API. V7 API uses _policies_ whereas the V8 uses _strategies_. I educated guess...

Could you please describe the desired behavior in a bit more detail (how many strategies are chain, in what order, how are they configured, etc.)? Or you could also share...

Well, as far as I know there is only a single place where Polly throws `RateLimiterRejectedException`: https://github.com/App-vNext/Polly/blob/d36058d0ac55ac221c1f0d9405aff932998c045a/src/Polly.RateLimiting/RateLimiterResilienceStrategy.cs#L68 I think we could extend the RLRE with a `Source` property and populate...

@lonix1 Do you want to give it a try and file a PR? > Unrelated side issue: is it correct to have the concurrency or rate limiter first? The ordering...

> PR: I don't know... I'm still new to Polly and there's many things I don't yet grok. Sure, no problem. @martincostello could you please assign to me this issue?

Hi @lonix1 we had made the required changes and it will be available as part of the 8.5.0 release. The new property name will be `TelemetrySource` (the `Source` was already...

I'll be off in the next two weeks, but I can start work on this request on the 16th of September week if that's not too late.

As I can see no one has jump on this request in my absence so, @martincostello please assign this ticket to me.

## The current state ### `OnRetry` The `OnRetry` telemetry events are always reported as `Warning`: https://github.com/App-vNext/Polly/blob/957444f8a3ce923b298e9dee1ec653447ec4091d/src/Polly.Core/Retry/RetryResilienceStrategy.cs#L81 - These are reported after a failed and handled attempt but before the next...