-
Notifications
You must be signed in to change notification settings - Fork 69
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
Comments
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. |
I cut a new release that includes this fix. |
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? |
I'm open to it. Thankfully it's a deeper dependency that application code doesn't really see or touch. |
I've started getting repeated errors of this. |
@chrisgreg I merged an explicit retry experiment into the main branch. If it works for you too, then it could be improved upon. |
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?
The text was updated successfully, but these errors were encountered: