Skip to content
This repository has been archived by the owner on Nov 17, 2018. It is now read-only.

Update Polly dependency to v5.9.0 #93

Merged
merged 2 commits into from
Apr 10, 2018
Merged

Conversation

reisenberger
Copy link
Contributor

Update Polly dependency to v5.9.0

  • Update Polly dependency
  • Update corresponding Polly.Extensions.Http dependency
  • Adopt Polly v5.9.0 usages within HttpClientFactory (no functional change)

Addresses #92

+ Updates Polly references to v5.9.0.
+ Updates Polly.Extensions.Http to corresponding version.
Use features new to Polly v5.9.0 in PolicyHttpMessageHandler.  The
primary change is improved syntax
policy.AsAsyncPolicy<HttpResponseMessage>() for the case when a
non-generic IAsyncPolicy needs conversion to a generic
IAsyncPolicy<HttpResponseMessage> for use with HttpClientFactory.
Copy link
Member

@Eilon Eilon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OSS approved.

@rynowak
Copy link
Member

rynowak commented Apr 10, 2018

Changes here look great

@rynowak
Copy link
Member

rynowak commented Apr 10, 2018

Will merge this shortly.

@rynowak rynowak merged commit 60a2338 into aspnet:dev Apr 10, 2018
@rynowak
Copy link
Member

rynowak commented Apr 10, 2018

Thanks!

@reisenberger reisenberger deleted the pollyv590 branch April 10, 2018 20:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants