Guillaume Gnaegi
Guillaume Gnaegi
@raman-m This is the PR we talked about, including a generic polling manager.
> Thanks for the PR! 👍 > > You have to update your feature branch from **develop** one! ... because of this commit [e5f31ef](https://github.com/ThreeMammals/Ocelot/commit/e5f31ef030a4004579da8c54d663721ffb58fa22) after merging #1711 And, unfortunately you...
@raman-m a new provider project is needed for ServiceFabric... Can I create a new project in the solution could you add me as assignee?
> Wow! I remember this PR... 😄 It is not planned for any milestones.. What is the approximate readiness now? Still a bit of work, but we could foresee it...
@raman-m this should be part of Annual release, first address the consul long polling issue.
> [raman-m](https://github.com/raman-m) commented [on Apr 7](https://github.com/ThreeMammals/Ocelot/pull/1710#issuecomment-2041485040) @raman-m I think I will need to rewrite it since I'm preparing a PR with the long polling for consul...
> So, is this PR dependent on a PR which will be created soon? Why keep this open? Mark draft? > > Do you need to re-write this PR keeping...
> @ggnaegi Please review and approve! I believe the PR is ready for delivery. > I've got necessary explanations by Ray, and I have a good clarity now. > Seems,...
@raman-m commented [on Mar 9](https://github.com/ThreeMammals/Ocelot/pull/1914#issuecomment-1986833147): > Seems nothing to do with that. But in future we could move all package tests to separate testing project and release the package independently...
> @raman-m commented [on Mar 6](https://github.com/ThreeMammals/Ocelot/pull/1914#issuecomment-1981194346) I don't know, probably some design issues since I agree with you about the HttpContextAccessor. Probably the way the object is instantiated.