snowplow / snowplow-php-tracker

Snowplow event tracker for PHP. Add analytics into your PHP apps and scripts
http://snowplowanalytics.com
34 stars 36 forks source link

Fix curl_multi_exec deprecated functionality error (close #121) #133

Closed mscwilson closed 1 year ago

mscwilson commented 1 year ago

For issue #121.

A Fatal error has been reported, caused by deprecation changes in the behaviour of curl_multi_exec(). This is a known problem, see e.g. this StackOverflow question.

I wasn't able to reproduce the error.

The problem appears to be the $running variable which is passed to curl_multi_exec(): it should be an Int, but is null. Strangely, $running wasn't actually defined in the code. This change declares it as an Int.