How do I set the connect timeout and timeout values?

[matthew-gill]

We're in the process of implementing Sentry for our Symfony app and would like to set some v. conservative timeouts in the event that the Sentry server is unavailable or slow to respond.

I've been able to set

sentry:
  options:
    send_attempts: 1

Which makes a slight bit of difference, but it seems as if the DEFAULT_HTTP_CONNECT_TIMEOUT and DEFAULT_HTTP_TIMEOUT are hard coded in vendor/sentry/sentry/src/HttpClient/HttpClientFactory.php vendor/sentry/sentry/src/HttpClient/HttpClientFactory.php

Is there a way I'm missing to override this? Perhaps using the transport option (but I can't seem to find any documentation for it).

If you have an example of how to use the transport option that would be great. Otherwise, I'll try to raise a PR to set these two values by the option config.

[matthew-gill]

I've now raised both PRs above. I'm not sure if this is the way to do it, if you have a better way (or it's already possible) I'd love to know, thanks!

[Jean85]

I thought that this feature was not feasible since it would require handling that setting for each proper client implementation but, since we're already doing it (but with a hard coded value) I think it has its merits. @ste93cry do you agree?

I have some additional thoughts:

• it's a new feature, so both PRs have to target develop
• here on Symfony we will need to lock onto that newer sentry/sentry package version, or it will break
• [EDIT] we need to check if there's a standard name for options like this in Sentry SDKs
• there are some other corrections needed on the PRs, but I'll leave a contextual review there.

Thanks for the contribution!

[ste93cry]

@ste93cry do you agree?

Yes, let's do it once for all

Is there a way I'm missing to override this? Perhaps using the transport option

We don't support that option in this SDK as the initial idea was that users should have used the transport_factory option to customize the transport (and the HTTP client eventually) and we didn't want to provide multiple ways to arrive to the same end because it could be confusing. On the other side though, it's not as easy as we would like to set up an HttpTransport transport...

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

[Jean85]

Now both PRs are released! Thank you @matthew-gill!