You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Due to edgurgel/httpoison#375 (which I created a PR to fix, and is now merged) if you use HTTPoison < v1.6.1 you will have a process leak when using watches whenever a GONE event is received from K8S.
Therefore should Kazan's deps be changed to force using this version or later of HTTPoison, or should this just be warned about in the docs?
I'm happy to create a PR for either option.
The text was updated successfully, but these errors were encountered:
Ah nice - I saw similar problems in a project I had based on HTTPoison a few years ago. Good to see someone fixing it.
I think it'd be fine to require > HTTPoison >= 1.6.1 - I'm assuming there's no breaking API changes, and people can always use an override: true dep if they really need an older verson.
Due to edgurgel/httpoison#375 (which I created a PR to fix, and is now merged) if you use HTTPoison < v1.6.1 you will have a process leak when using watches whenever a GONE event is received from K8S.
Therefore should Kazan's deps be changed to force using this version or later of HTTPoison, or should this just be warned about in the docs?
I'm happy to create a PR for either option.
The text was updated successfully, but these errors were encountered: