Skip to content

add support for CURLOPT_CONNECTTIMEOUT - #115

Merged
sgiehl merged 2 commits into
matomo-org:masterfrom
vpapaloukas:master
Mar 24, 2023
Merged

add support for CURLOPT_CONNECTTIMEOUT#115
sgiehl merged 2 commits into
matomo-org:masterfrom
vpapaloukas:master

Conversation

@vpapaloukas

Copy link
Copy Markdown
Contributor

Description:

add support for CURLOPT_CONNECTTIMEOUT. The default value it gets is the one set by curl lib (300 sec)

Review

Comment thread MatomoTracker.php Outdated

/**
* Returns the maximum number of seconds the tracker will spend trying to connect to Matomo.
* Defaults to 0 seconds (unlimited).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does default to 300, right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes sorry, fixing it

reference for default value https://curl.se/libcurl/c/CURLOPT_CONNECTTIMEOUT.html

@sgiehl
sgiehl merged commit c9e7816 into matomo-org:master Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants