HTTP requests to SendGrid can hang as SendGridAPIClient does not have a timeout.

[dashk]

Issue Summary

HTTP requests to SendGrid could hang, as SendGridAPIClient has no time out defined in case of a network anomaly.

Steps to Reproduce

This is a bit difficult to reproduce, as it could be caused by a network anomaly (Not necessarily SendGrid's issue).

When a network anomaly occurred, it is possible for API calls to SendGrid to hang indefinitely.

• By default, the HTTP client's timeout defaults to None.
• When SendGridAPIClient creates a client, it does not allow a timeout to be passed in.
• User of the client, unfortunately, has no way to provide a timeout, unless s/he creates a wrapper on top of SendGridAPIClient to override SendGridAPIClient.client or to have an outer wrapper to control timeout.

Expected Behavior
Either a reasonable timeout is set, or a timeout option should be allowed when creating a SendGridAPIClient.

Actual
None is available.

Technical details:

• sendgrid-python Version: master
• Python Version: 2.7
• python-http-client Version: master

Relevant Link:

• How HTTP Client is created in SendGridAPIClient:

  sendgrid-python/sendgrid/sendgrid.py

  Line 75 in 265b984

  self.client = python_http_client.Client(host=self.host,

• python-http-client: https://github.com/sendgrid/python-http-client/blob/master/python_http_client/client.py#L71

[thinkingserious]

Great catch! I've tagged this appropriately and the fix should happen on the v4 branch here..

With Best Regards,

Elmer

[erdnaxeli]

It seems it is still not fixed in the last versions, am I right?

[thinkingserious]

It did not @erdnaxeli, but thanks for the comment. Your comment helps us move this up the ladder in our backlog. Thanks!

[QuicksilverMachine]

We've had the same problem recently, for now I've fixed it with:

sendgrid_client = sendgrid.SendGridAPIClient()
sendgrid_client.client.timeout = 10

Hopefully this will be fixed soon, being able to set the timeout when initializing the client would be great.

[thinkingserious]

Since there has been no activity on this issue since March 1, 2020, we are closing this issue. Please feel free to reopen or create a new issue if you still require assistance. Thank you!

[matheusbrat]

I'm also having this issue on python 3.7 and sendgrid==6.6.0.