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

%Mint.TransportError{reason: :closed} #42

Closed
chrisgreg opened this issue Dec 1, 2023 · 6 comments · Fixed by #66
Closed

%Mint.TransportError{reason: :closed} #42

chrisgreg opened this issue Dec 1, 2023 · 6 comments · Fixed by #66

Comments

@chrisgreg
Copy link
Contributor

chrisgreg commented Dec 1, 2023

I'm intermittently getting %Mint.TransportError{reason: :closed} as a result from do_api_request/4 - has this been seen before or are there any known ideas why?

I can fire off the exact same request afterwards and it works most of them time but 1% of calls are failing...

EDIT: My team found that this is an ongoing error with Finch. Look at the latest comments here.

Are we able to swap the HTTP client out for another?

@brainlid
Copy link
Owner

brainlid commented Dec 1, 2023

Should be resolved by PR #43. Let me know if you're still getting the error. For whatever reason, I haven't noticed this as an issue myself.

@brainlid brainlid closed this as completed Dec 1, 2023
@brainlid
Copy link
Owner

brainlid commented Dec 1, 2023

I cut a new release that includes this fix.

@chrisgreg
Copy link
Contributor Author

Still getting an error even with the retry - we're using this in production so there's a fair bit of usage coming but this error is still around even with a retry policy wrapping the LLMChain.run call.

Unsure of the appetite but might be worth swapping to HTTPoison for the underlying caller?

@brainlid
Copy link
Owner

I'm open to it. Thankfully it's a deeper dependency that application code doesn't really see or touch.

@brainlid
Copy link
Owner

I've started getting repeated errors of this.

@brainlid
Copy link
Owner

@chrisgreg I merged an explicit retry experiment into the main branch. If it works for you too, then it could be improved upon.

https://github.com/brainlid/langchain/pull/66/files#diff-98396b383d660bb5274db90a5fd13b1526d5d6b120a2d8ece0e8baf06e55b85eR336

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants