Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Sample App] Add Microsoft.Extensions.Http.Resilience #1643

Merged
merged 4 commits into from
Jan 14, 2024

Conversation

brminnick
Copy link
Collaborator

Description of Change

This PR replaces Microsoft.Extensions.Http.Polly with Microsoft.Extensions.Http.Resilience.

In .NET 8, Microsoft teamed up with the App-vNext.Polly team to create a more-performant version of Polly. The new Microsoft.Extensions.Http.Resilience library uses the new, more performant version of Polly.

Meanwhile [Microsoft.Extensions.Http.Polly](https://www.nuget.org/packages/Microsoft.Extensions.Http.Polly) uses the older, less performant, version of Polly now rebranded as Polly.Core.

To ensure our Sample App adheres to best practices recommended for all .NET MAUI developers, we are upgrading Microsoft.Extensions.Http.Polly with Microsoft.Extensions.Http.Resilience.

PR Checklist

  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard

Additional information

More information is available in this .NET Blog Post:

https://devblogs.microsoft.com/dotnet/building-resilient-cloud-services-with-dotnet-8/

Resilience packages

Co-authored-by: Pedro Jesus <pedrojesus.cefet@gmail.com>
@brminnick brminnick requested review from pictos and removed request for VladislavAntonyuk January 13, 2024 02:13
@brminnick brminnick merged commit 11e7122 into main Jan 14, 2024
7 checks passed
@brminnick brminnick deleted the Add-Microsoft.Extensions.Http.Resilience branch January 14, 2024 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants