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

Curl transfers not completing #1156

Closed
docEdub opened this issue Oct 25, 2022 · 4 comments · Fixed by #1180
Closed

Curl transfers not completing #1156

docEdub opened this issue Oct 25, 2022 · 4 comments · Fixed by #1180
Labels
Milestone

Comments

@docEdub
Copy link
Contributor

docEdub commented Oct 25, 2022

In UrlRequest_Unix.cpp, curl transfers are not always completing. Gary and I looked at this for a while and suspect that we're not using the curl library correctly.

Steps to reproduce the behavior:

  1. On a Linux setup, start a script that runs the unit tests 1000 times.
  2. Eventually, one of the unit tests will get stuck and need to be killed manually.

Expected behavior:
The unit tests should never get stuck and need to be killed manually.

Other:

  • Platform: Linux
@CedricGuillemet
Copy link
Contributor

I'm taking a look.

@CedricGuillemet CedricGuillemet added the bug Something isn't working label Oct 27, 2022
@CedricGuillemet CedricGuillemet self-assigned this Oct 27, 2022
@thomlucc thomlucc added this to the 6.0 milestone Nov 1, 2022
@thomlucc
Copy link
Member

thomlucc commented Nov 1, 2022

@CedricGuillemet - please check with @bghgary

@CedricGuillemet
Copy link
Contributor

It's difficult to repro on my side. I'm even able to do 1000 runs without a failure. I let you fix it @docEdub

@docEdub docEdub removed their assignment Nov 15, 2022
@bghgary bghgary removed the bug Something isn't working label Dec 8, 2022
@bghgary
Copy link
Contributor

bghgary commented Dec 8, 2022

It's so rare, we are going to wait on fixing it.

@bghgary bghgary modified the milestones: 6.0, Future Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants