Skip to content

Poor handling of timeouts when trying to send messages with large attachments #33

@astiskala

Description

@astiskala

Hi There,

I have been seeing timeouts in some cases when a message has many attachments. The timeouts surface themselves as a NullReferenceException as such:

System.NullReferenceException: Object reference not set to an instance of an object. at Typesafe.Mailgun.Http.MailgunHttpResponse.ReadJson() at Typesafe.Mailgun.Http.MailgunHttpRequest.GetResponse() at Typesafe.Mailgun.MailgunCommand1.Invoke()`

I believe it would be best if the library threw either a standard web exception or something more explicit in these cases.

It would also be good to be able to set a custom timeout value on the MailgunClient to override the default.

I'd be happy to put together a PR for this if you're accepting changes.

  • Adam

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions