You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
RateLimit demo app reports Seconds instead of TotalSeconds #89
Describe the bug
In the RateLimit demo code a TimeSpan.Seconds is used to report the delay in the console. The TimeSpan.Seconds goes to 59 seconds and then starts over with 0 seconds. Thus if a delay of 70 seconds is calculated or returned in the response header then the RateLimit demo reports this as 10 seconds instead of 70 seconds.
AnsiConsole.MarkupLine($"[orange3]Delaying {apiType} request for {newTimeSpan(delayInTicks).Seconds} seconds, capacity left: {capacityLeft}%[/]");
Additional context
We had using TimeSpan.Seconds to log the RateLimit and Throttling delay. It was until we compared the log with a Fiddler log that this issue has come clear to us. It can easily be solved by using the TotalSeconds property.
Activity
Support for Rate-Limiting in the PnP Framework + unifiqation of the t…