Guillaume Gnaegi

Results 75 comments of Guillaume Gnaegi

@raman-m The error handling is hiding the 413 exception payload too large. I've tested this with Kestrel, it's throwing a `BadHttpRequestException`. I will create a PR as soon as I...

> @ggnaegi Oh, no! > We forgot to check this user scenario before the release 😏 Ouch...

@raman-m @camilocalderont @RaynaldM By design, if only one route, then when don't make any further checks and process the downstream request. It's a bit straight forward, but that's the way...

@nikhillkumar @raman-m you still have an option, it's adding your own logic here: https://github.com/ThreeMammals/Ocelot/blob/develop/src/Ocelot/Requester/MessageInvokerPool.cs#L85 You could even go a step further and use DI. ```c# RemoteCertificateValidationCallback = (sender, certificate, chain,...

@raman-m Yes, I just gave a solution to @nikhillkumar, but the feature should be implemented imo.

> @ggnaegi > > > the feature should be implemented imo. > > By whom? By Mr. Kumar? I think we should first convert this to a discussion and talk...

> Gui, thanks for the new issue! > > > (Consul, Eureka...) > > What about ServiceFabric? > Can this upgrade be applied to ServiceFabric? @raman-m yes, but should I...

@raman-m We could keep it like that for now. I don't think polling has been foreseen for ServiceFabric. It's not that complicated to extract the provider though, but as you...