Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop Using a Deprecated Docker Client #83

Closed
RealTYPICAL opened this issue Jan 24, 2024 · 2 comments
Closed

Stop Using a Deprecated Docker Client #83

RealTYPICAL opened this issue Jan 24, 2024 · 2 comments
Labels
enhancement New feature or request
Milestone

Comments

@RealTYPICAL
Copy link

Currently the shiny proxy uses com.spotify:docker-client:jar:8.16.0 which is considered deprecated as of writing https://github.com/spotify/docker-client

Would it be possible to use its canonical replacement found here: https://github.com/docker-java/docker-java

The reason is that the spotify:docker-client has many issues associated with it in regards to the socket hanging when trying to listen for the response. It looks like it's a common problem with older versions of the spotify docker lib.

fabric8io/docker-maven-plugin#344
fabric8io/docker-maven-plugin#981
fabric8io/docker-maven-plugin#552
fabric8io/docker-maven-plugin#862

Some explanations in older issues for other projects indicate that it's related to an interaction between the native socket library, the system, and how it's being used (the spotify docker lib). In TCP HTTP the connection will close/once it has reached the end even if you try to read too much, but the domain socket will not.

Thanks in advance.

@LEDfan
Copy link
Member

LEDfan commented Jan 25, 2024

Hi, thanks for raising this issue. As mentioned in openanalytics/shinyproxy#474 we are looking into replacing the deprecated library for the next release. We have not yet decided which library to use, since we also want to avoid to have rewrite all the code interacting with the docker containers. I'll update this issue once we have replaced the library.

@LEDfan LEDfan added the enhancement New feature or request label Jan 25, 2024
@LEDfan LEDfan added this to the Next milestone Jan 25, 2024
@LEDfan
Copy link
Member

LEDfan commented May 7, 2024

Hi, this is now part of the ShinyProxy 3.1.0 we released today. Thank you for opening this issue.

@LEDfan LEDfan closed this as completed May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants