search

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

Current implementation relies on shell_exec() #49

Closed smentek closed 1 year ago

smentek commented 9 years ago

It comes from "ramsey/uuid" on Uuid.php.

Is it how it is going to be? To use it on production application server one will have to enable 'shell_exec()'method? New version uses 'passthru()' instead but in my opinion it is still not a way to go.

function_exists from Tracker.php line 154 should check function from the namespace in with is used? Whatever it supposed to do it does not work and (constant?) UUID_TYPE_TIME just does not exist.

alexanderdean commented 9 years ago

Is there another way of generating a UUID in PHP?

smentek commented 9 years ago

What about: https://github.com/webpatser/laravel-uuid ? I have not check it but I would go with whatever that does not call to OS directly. If your library calls one of 'disabled functions' it is like red flag. I'm afraid to set current version on prod...

alexanderdean commented 9 years ago

Looks promising!

mscwilson commented 1 year ago

Closing because this is very old.