-
Notifications
You must be signed in to change notification settings - Fork 23
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
sift.client.ApiException: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer')) #82
Comments
Hello @ashwinv11 thanks for letting use know.
Thanks :) |
Hello! |
Thanks @StevenLudwig we are investigating this issue. |
Hi @vincent-sift just reached out to support with some additional information. Thanks! |
Thanks @ashwinv11. I am following up with the engineering team as well. |
Hi there, is there any update on this one? 🙏🏾 Are there any recommendations on how to handle this until it's fixed? Should we be retrying API calls if we get this error? |
@ashwinv11 we are working on getting this work scheduled. Our current suggestion is to retry these requests when you get these exceptions. Thanks for your patience. |
Hey @vincent-sift any update on this? |
@ashwinv11 a few questions:
Thanks! |
Hey @viaskal-sift sure! Python version – Here's an example of how we have it setup. Looks like we set a longer timeout than the default, and don't build the import sift
client = sift.Client(SIFT_SCIENCE_API_KEY, timeout=3)
def send_to_sift(data):
try:
response = client.track('$create_order', data, return_workflow_status=True)
if response and not response.is_ok():
logger.error('...')
except Exception:
logger.exception('..')
return None
return response Hope this helps! |
@ashwinv11 Could you pls try to add some pre-configuration before creating a Sift Client using the code snipped below? Basically, it will configure the Session object particularly and only for the Sift Client, and only requests to the Sift API will be affected by that.
|
@ashwinv11 did you have a chance to try this one? Thanks! |
Hi Viaskal, we're hoping to tackle this guy soon. |
@stephenflynn any updates from your side? |
Hi @stephenflynn wanted to check if @viaskal-sift 's recommendation was able to resolve the issue? Thank you! |
Hello there!
We're seeing some 104 Connection Reset errors from the v205 Events POST endpoint. We're using version 5.0.1 of the SDK. It looks like this is an issue with the underlying calls to requests. Is this a known issue? What's the best way to handle these exceptions?
Thanks!
The text was updated successfully, but these errors were encountered: