Don't Use Polly in .NET Directly. Use this instead!

Поделиться
HTML-код
  • Опубликовано: 25 янв 2025

Комментарии • 146

  • @aweklin
    @aweklin 7 месяцев назад +16

    I love this. Finally learnt about resilience in an extremely easy way!
    Thanks Nick.

    • @jamesgoforth5644
      @jamesgoforth5644 3 месяца назад

      learnt, wtf. Have you tried learnting english just saying wow. Programmers who are illiterate is apparently a thing.

  • @AvenDonn
    @AvenDonn 7 месяцев назад +118

    Damnit, it was enough of an adventure getting my team to use Polly

    • @nickchapsas
      @nickchapsas  7 месяцев назад +26

      You're on the right path

    • @jojify
      @jojify 7 месяцев назад +8

      @@nickchapsas If a team plans to implement this in a microservice project with Kubernetes, I recommend using a service mesh mostly with CNCF products like lined/istio. It has all the necessary features, including the telemetry and observability MTLS, and all features can be implemented along with fine-tuning the behavior, like the number of retries, etc., just by YAML definition without touching the actual source code.

    • @marekmagath7090
      @marekmagath7090 7 месяцев назад +3

      But it's still using Polly in the background, or am I wrong? AddStandardResilienceHandler is using ResiliencePipelineBuilder in the background and ResiliencePipelineBuilder is from Polly. Microsoft just defined some "standard" resilience pipeline, that is all.

    • @AvenDonn
      @AvenDonn 7 месяцев назад

      @@marekmagath7090 Yeah for that standard case sure. But we're doing some much lower level stuff, not just typical network HTTP requests

  • @iBeechus
    @iBeechus 5 месяцев назад

    I wrote my own resilience class 7 years ago which does most of this. But time to modernise! Awesome content.

  • @mariacobretti
    @mariacobretti 7 месяцев назад +5

    why is using a jitter more important for exponential backoff than linear? or did i misunderstand that

    • @nickchapsas
      @nickchapsas  7 месяцев назад +16

      It's not. I mixed it up, it's actually less important but still important

  • @rngesus8057
    @rngesus8057 7 месяцев назад

    9:05 oh crap i normally just return tasks from lambdas expecting a task return. whats the difference between that and making it an async lambda like u have here?

    • @JoshWeeks
      @JoshWeeks 3 месяца назад

      The only real difference would be whether you can await inside the lambda. It's producing a Task either way.

  • @futurexjam2
    @futurexjam2 7 месяцев назад

    sometime ago, I wrote a wrapper class for polly to combine policies in a dynamic way :)

  • @vidhyajayavelu3688
    @vidhyajayavelu3688 10 дней назад

    How does this resilience handler works across different servers/instances? If application hits some transient error by calling an endpoint from server 1, will server 2 call the endpoint and hit the transient error again or will it know that server 1 had already made call and an error occurred? Thanks

  • @surajbangale839
    @surajbangale839 7 месяцев назад +6

    How this works with Refit C#

  • @1hotsteppa83
    @1hotsteppa83 3 месяца назад

    Great video and explanation. Thanks Nick!!

  • @amirhosseinahmadi3706
    @amirhosseinahmadi3706 7 месяцев назад +17

    Why did you delete the primitive obsession video?

    • @nickchapsas
      @nickchapsas  7 месяцев назад +16

      I wanna remake it with more context

    • @Wouldntyouliketoknow2
      @Wouldntyouliketoknow2 7 месяцев назад +16

      More dometrain context? 😅

    • @aweklin
      @aweklin 7 месяцев назад +1

      Thought I was the only one who noticed it. But I had finished watching & wanna comment before realizing it was deleted! It was great @Nick.

    • @phizc
      @phizc 7 месяцев назад

      ​@@aweklinsame. I got "This action is forbidden" when I tried to post a comment. Weirded me out. At least I saved the comment on OneNote for next time 🙂.

    • @Tsunami14
      @Tsunami14 7 месяцев назад

      ​@@aweklinSame! I went back to comment, and it said the video is now private.

  • @jwtly200
    @jwtly200 7 месяцев назад

    I can understand the first half of the video. What is the second half doing? I mean adding provider, and then removing http client and add the standard handler?

  • @kirillhorn3186
    @kirillhorn3186 7 месяцев назад +4

    Your video is just in time, Nick. I needed to implement this logic in api today)

  • @stephajn
    @stephajn 7 месяцев назад

    Very cool, but I have to ask, what if you are injecting that policy into multiple services that each make aj http request, and you send up those requests simultaneously using Task.WhenAll? Is the pipeline registered to be injected in a Transient scope, or are you getting a singleton pipeline, thus limiting you to one operation at a time?

    • @zakaria_ml
      @zakaria_ml 7 месяцев назад

      I think the pipeline is configured and implemented to handle multiple concurrent operations safely, and as mentioned in the doc, the pipeline is registered as a singleton but it is implemented to maintain separate state for each execution.
      For the Task.WhenAll, each request will go through the same pipeline instance, but it will have its own execution context and state.

  • @rodrigo6459
    @rodrigo6459 7 месяцев назад

    Is it a good idea to place that nice resilience pipeline inside a try/catch block?

  • @zoltanzorgo
    @zoltanzorgo 7 месяцев назад +3

    Great stuff! I am thinking about a very common situation: paralleism. Your method is called concurrently. let's say you have a named pipeline for a specific task. The policy is reusable, all right, but in an idempotent way, or does it have a state? If - for example - the circuit is broken in one call, it should (if needed for specific task cases) immediately fail for all the other subsequent calls until reset. In other scenarios the policy calls can be independent. Is there support for such a thing?

    • @Workshopshed
      @Workshopshed 7 месяцев назад +1

      For the circuitbreaker to work it needs to store state e.g. how many failures, is the circuit open or closed.
      If you use the same pipeline for all requests then the circuit will break for one and then all of the others too.
      If you want independent behaviour then you need separate pipelines

    • @Workshopshed
      @Workshopshed 7 месяцев назад

      For parallel calls then the concurrency option might be of interest

  • @ChrisWard74
    @ChrisWard74 7 месяцев назад +1

    I didn't know about this, and it looks very cool. Can it be used for Mongo/Cosmos retries, where the exception tells you how long you should wait before retrying?

  • @xaberue
    @xaberue 7 месяцев назад +1

    awesome sample! thanks for sharing!

  • @eugene3685
    @eugene3685 7 месяцев назад

    Hi! Is it really good idea to have resilience pipeline as a static instance, in such case it must be thread safe, because single pipeline instance has to serve many requests there?

  • @magicsmoke0
    @magicsmoke0 7 месяцев назад +1

    Can we add logic to detect retryable vs non-retryable errors?

    • @nickchapsas
      @nickchapsas  7 месяцев назад +5

      Absolutely. On the ShouldHandle method

  • @JonasJermann
    @JonasJermann 7 месяцев назад

    If I have many services I prefer to have the service setup within that service implementation and not part of it in the centralized DI setup. I guess that means I cant really use the HttpClient/HttpClientFactory approach with included resilience since I would have to set this up in the central DI setup, right?

    • @gileee
      @gileee 7 месяцев назад

      You can make an extension method for IServiceCollection in the same file as your service that wires that service and anything else it needs. So you have the code there and only call it in the startup method (I guess you could even avoid having to do that with automatic wiring using reflection).
      I generally prefer to use DI for everything if I'm already using DI. Why avoid the benefits of easy mocking, being able to see all the services in one big list... if you're already using it.

    • @jjermann2
      @jjermann2 7 месяцев назад

      @@gileee I currently use the approach to just inject the httpclientfactroy (yes setup globally using DI) and then in the service I would create a client and use a service specific resilience (the resilience not from DI), that way Program.cs/Startup.cs doesn't contain any "service specific" logic which I find nice. I am still curious if I can somehow improve on the pattern... I guess I could globally setup some basic resilience pipeline (not service specific) and then use that one in the service. I guess the key point is the HttpClient/HttpClientFactory which I don't like to setup service specific in the main Program even if the setup is from an extension method from the service code (it still would introduce a service specific setup in the main program (beyond the DI setup of IService -> Service implementation which is of course needed somewhere).

  • @jonathanblackwell42
    @jonathanblackwell42 7 месяцев назад

    To answer your question at the end of the video: not knowing of the name of this concept, we wrote our own stuff to do this. I'm going to forward this to the team so we can do better moving forward.

  • @jessecavada3015
    @jessecavada3015 7 месяцев назад +1

    2:36 the city query string parameter value is not url encoded. 🙂

  • @sebastianfriedrich754
    @sebastianfriedrich754 7 месяцев назад +1

    Great Content! Is there also a Microsoft builtin for token management?

  • @UrsEnzler
    @UrsEnzler 7 месяцев назад

    Nice video. However, what is now better than using Polly?

  • @Natusch_
    @Natusch_ 7 месяцев назад +4

    Hey Nick! Love your videos. Is there any reason in particular for you using Rider instead of Visual Studio? Would love to know.

    • @kabal911
      @kabal911 7 месяцев назад +5

      A lot of us Rider users probably started as Resharper users as .NET Core came about
      When I started using Rider(about 5 or 6 years) I found it to be significantly faster than Visual Studio, allowed me to use Linux, and I was coming from IntelliJ and PHP Storm

    • @mx0r
      @mx0r 7 месяцев назад +2

      It is much better IDE for non-desktop projects. Combines ReSharper with a simple strealined UI. And if you are user of JetBrains tools, you are at home in all of them.

    • @Natusch_
      @Natusch_ 7 месяцев назад

      @@mx0r could you be more specific on it being "better for non desktop apps?" are you saying it as in better for web apps, sockets/threads related stuff, etc.?

    • @DiddleDangle
      @DiddleDangle 7 месяцев назад +1

      i use it because I'm on mac and I don't like the visual studio for mac. don't like the performance hit of using it in a windows vm but that's what I used to do.

    • @mx0r
      @mx0r 7 месяцев назад

      @@Natusch_If you want to do XAML-based stuff, there is no visual editor or preview in Rider. So I usually did the desktop applications in VS+ReSharper and the usual backend things in Rider.

  • @fezhaez
    @fezhaez 7 месяцев назад

    I sadly still work in framework 4.7.2 and usually use Polly if I need a simple retry. Because I work with a lot of multi tenant to single tenant application lately I've been using queues to provide a rudimental retry mechanism and data persistency

  • @atziazas
    @atziazas 6 месяцев назад

    Nick thank you for this video. Can you give us what your favorite VS Code extensions are for C#?

  • @texasyobro
    @texasyobro 7 месяцев назад

    Is there a way to use this library in Blazor?

  • @dennisvandermeer8238
    @dennisvandermeer8238 7 месяцев назад

    Will this replace the result pattern as this can handle any exception so having both could be overkill, right?

    • @Workshopshed
      @Workshopshed 7 месяцев назад

      You can handle both results and/or exceptions.

  • @たろ羊
    @たろ羊 7 месяцев назад +1

    What happened to your previous video?

  • @alonmore28
    @alonmore28 7 месяцев назад

    Is it possible to chain pipelines? For example have a default pipeline but also I wish to have some specific handling in a specific request, and have that on top of the basic pipeline.
    Some cases I may want to override parts of the strategies of the default pipeline, and some cases I just want to add to it.

    • @volan4ik.
      @volan4ik. 7 месяцев назад

      I don't know if it's possible to chain pipelines, but at least it's very easy to make your own pipeline builder extensions to simplify the pipeline creation (so you will have some kind of duplicates of pipeline objects, but in memory, not in code)

    • @fdevl
      @fdevl 2 дня назад

      Did you find a solution for this?

  • @Velociapcior
    @Velociapcior 7 месяцев назад +3

    6:53 "by the power of one, by the power of two, by the power of maaaany" (sorry Nick, I had to :))

    • @KingOfBlades27
      @KingOfBlades27 6 месяцев назад

      From the bottom of all SW fans: fu 🖕😂

    • @Velociapcior
      @Velociapcior 6 месяцев назад

      @@KingOfBlades27 what did I do to deserve such a treatment from, as you claim "all SW fans?"

    • @KingOfBlades27
      @KingOfBlades27 6 месяцев назад

      @@Velociapcior That should be quite self-evident 😂

    • @Velociapcior
      @Velociapcior 6 месяцев назад

      @@KingOfBlades27 it's not evident to me, please enlighten me, what did I do wrong?

    • @KingOfBlades27
      @KingOfBlades27 6 месяцев назад

      @@Velociapcior Quoting a ridiculous scene from a show that overwhelming amount of viewers hate 😂

  • @henningerhenningstone691
    @henningerhenningstone691 7 месяцев назад

    Does this also have support for persistence and long-running retries? Like, if the request fails, retry it in a few hours, but in the meantime our application could potentially be restarted 😅

    • @Workshopshed
      @Workshopshed 7 месяцев назад

      Recommendation is not to have really long timeouts/retry intervals. For that you could look at a task scheduler like Quartz or Hangfire. If a job fails then schedule it for later. You could use the fallback for that.

  • @bensmith6004
    @bensmith6004 6 месяцев назад

    This actually looks really cool

  • @wusswuzz5818
    @wusswuzz5818 7 месяцев назад

    This sounds like a perfect way to handle jwt access token expiry. You can configure to retry with the refresh token one time on failure of access token due to expiry.

  • @antonmartyniuk
    @antonmartyniuk 7 месяцев назад

    Does Refit integrate with this Microsoft resilience package?

  • @Grymyrk
    @Grymyrk 7 месяцев назад +3

    In the weatherserivce class should you name it "_resiliantHttpClient" to distinguish it from a standard httpclient? How can we make sure the reader of the service knows it's resilient without them looking at the DI config?

    • @isnotnull
      @isnotnull 7 месяцев назад +2

      The truth is you always need to know what's in the config. What database is used, where logs are stored and so on. So appsettings.json and Program.cs (+optional Startup.cs) are places you need to learn by heart

    • @DustyBg
      @DustyBg 7 месяцев назад

      ​@@isnotnullExactly, naming it a 'resilientHttpClient' still does not provide you any clues as to which policies are applied without digging through the DI bootstrap.

  • @Chorniyko
    @Chorniyko 7 месяцев назад

    Is there a way how to test those policies? Polly has Simmy, so I was wondering if the same alternative exists or planned to be shipped in the future.
    P.S. I looooove that .NET finally offers such a nice API for such a basic problems!

  • @jakebradminster709
    @jakebradminster709 4 месяца назад

    I think they hardcoded 'Clouds' for London.

  • @kabal911
    @kabal911 7 месяцев назад +1

    That's very slick I must say

  • @extremedrone5365
    @extremedrone5365 7 месяцев назад +3

    You removed the previous video about using primitives

    • @nickchapsas
      @nickchapsas  7 месяцев назад +1

      No way

    • @paulkoopmans4620
      @paulkoopmans4620 7 месяцев назад +2

      ​@@nickchapsas yes way! :) correction: made it private.

  • @ThisIsStupid12312312
    @ThisIsStupid12312312 7 месяцев назад +5

    This is nice but lets go back to: On Error Resume Next! No need for error handling.

  • @jchandra74
    @jchandra74 7 месяцев назад

    So Polly is not needed anymore?

    • @eugene3685
      @eugene3685 7 месяцев назад

      Polly of version 8, has the same api and provides the same experience when build a pipeline. So, you can use any of packages at least for now.

    • @IAmFeO2x
      @IAmFeO2x 7 месяцев назад

      @@eugene3685In fact, Microsoft.Extensions.Resilience uses Polly.Core 8.x under the covers. You can just reference Polly.Core and that's it.

  • @paulovictor9439
    @paulovictor9439 7 месяцев назад +9

    Why you removed your latest video?

    • @nickchapsas
      @nickchapsas  7 месяцев назад +11

      To remake it with more context

    • @IamSystemko
      @IamSystemko 7 месяцев назад +1

      Primitive obsession feels like April Fools' day video. But It's June now)

  • @rodrigodearcayne
    @rodrigodearcayne 7 месяцев назад +4

    So why is this better than Polly?

    • @Mio2k10
      @Mio2k10 7 месяцев назад

      Not sure if he somewhere states that it is better, but -1 dependency

    • @eirik8440
      @eirik8440 7 месяцев назад +1

      @@Mio2k10 The title currently says "Don't Use Polly in .NET Directly. Use this instead!". I was wondering about the same thing.

    • @AndersBaumann
      @AndersBaumann Месяц назад

      Why is it -1 dependency? This is also a Nuget package.

  • @lalithprasadsrigiriraju
    @lalithprasadsrigiriraju 7 месяцев назад

    This is so good 🎉

  • @IssaFram
    @IssaFram 7 месяцев назад +1

    There were Polly references in the code though...

    • @kell7689
      @kell7689 7 месяцев назад +1

      Noticed this as well...

  • @amirhosseinahmadi3706
    @amirhosseinahmadi3706 7 месяцев назад +3

    Why are you still not using primary constructors for DI?

    • @luk4sz98
      @luk4sz98 7 месяцев назад +8

      because they are bad for DI readonly fields.. ?

    • @nickchapsas
      @nickchapsas  7 месяцев назад +19

      Because readonly is not supported yet

  • @GigAHerZ64
    @GigAHerZ64 7 месяцев назад +4

    Why is it not using keyed injection instead of injecting the provider? What a missed opportunity!

  • @robrider838
    @robrider838 7 месяцев назад +2

    Adding logging when each retry occurs would be nice. Log.Warning for each retry, then Log.Error when all retries have been exhausted.

    • @pazzuto
      @pazzuto 7 месяцев назад

      This I need. I need to log retries/failures to pass on that telemetry to my other team who support the service. I had to roll out my own even after looking at Polly.

    • @robrider838
      @robrider838 7 месяцев назад +1

      @@pazzuto You can do it with Polly. We do.

    • @pazzuto
      @pazzuto 7 месяцев назад

      @@robrider838 I see they added telemetry in v8. Nice! I'll be taking a look. Thanks so much for pointing it out.

    • @PaulNicholsAUS
      @PaulNicholsAUS 7 месяцев назад +1

      Yep it does log when it retries

  • @chris-pee
    @chris-pee 7 месяцев назад

    It's not bad, but it seems a bit more verbose compared to just using Polly.
    And they went with stringly typed DI again 🤨 but at least it seems you can use other types as key.

  • @csabraxas
    @csabraxas 6 месяцев назад

    Polly works with anything not just HTTP specific contexts.

  • @zeko77tz
    @zeko77tz 7 месяцев назад

    Beautiful.

  • @krigrtrue
    @krigrtrue 7 месяцев назад

    I can understand that there's an argument for supporting retry and circuit breakers for old school hosting of services or if you just have a need to do it all yourself.
    In my opinion there are better places for handling these problems than in the code.

  • @rngesus8057
    @rngesus8057 7 месяцев назад

    hi nick

  • @nocgod
    @nocgod 7 месяцев назад

    Polly lost me with V8 and their removal of the cache policy. Their whole API got strange and they omitted the cache and distributed cache policy which was golden when used with the request deduplication policy. Even the absence of bulkhead is a downgrade

  • @Lovecraft81
    @Lovecraft81 7 месяцев назад

    Finally!!!

  • @Milk-gw1zl
    @Milk-gw1zl 7 месяцев назад

    5:41

  • @PhilHeighes
    @PhilHeighes 7 месяцев назад +1

    Hello Weather examples are fine, how about a real world example using Refit with rate limit handling and logging; that was my task today

  •  7 месяцев назад

    I usually handroll my own stuff. Where I work the 3rd party packages are frowned upon.

    • @philosophersam
      @philosophersam 7 месяцев назад +1

      Lame! Refit and Polly make me so much more productive.

    • @fatlumlatifi2897
      @fatlumlatifi2897 7 месяцев назад

      @@philosophersam If you do contract work on critical software and your paycheck gets delayed from rejected security review you will understand why we have to 'handroll' our stuff, you also have to lick your screen before every commit for better luck🍀

    • @M0J0-RL236
      @M0J0-RL236 7 месяцев назад

      @@fatlumlatifi2897sounds like overzealous security policies. Why waste time and reinvent the wheel if there are good packages out there?

  • @vkovalyuk
    @vkovalyuk 7 месяцев назад +1

    Microsoft.Extensions.Http.Resilience uses Polly under the hood. Is this why you use word 'directly' in the title?

  • @androidsavior
    @androidsavior 7 месяцев назад

    Wowww

  • @T___Brown
    @T___Brown 7 месяцев назад +1

    Waters muddied 😭

  • @Ocean7653
    @Ocean7653 7 месяцев назад

    Hey nick I’d love a job 🙏

  • @orthodox-4-ever
    @orthodox-4-ever 7 месяцев назад +3

    Offtopic:
    Why you chose Insomnia over Postman? 😀

    • @mariocamspam72
      @mariocamspam72 7 месяцев назад +1

      Postman is pay2win 😂

    • @RtoipKa
      @RtoipKa 7 месяцев назад

      Builtin http client in Rider is a lot better than those two.

  • @KRTone123
    @KRTone123 7 месяцев назад

    OR! You can use service mesh.

    • @RtoipKa
      @RtoipKa 7 месяцев назад

      Or rewrite to Rust. Lol.

  • @mbalaganskiy
    @mbalaganskiy 7 месяцев назад

    ⚰️ polly

  • @jamesgoforth5644
    @jamesgoforth5644 3 месяца назад

    Man that voice is weird, really weird. I miss fingernails scraping the chalk board after hearing 2 seconds of it argh lol