Tracking API setVisitorId should throw exception if the visitor ID is not an hexadecimal string #3787

Closed
mattab opened this Issue Mar 3, 2013 · 1 comment

1 participant

@mattab
Piwik Open Source Analytics member

per title. Otherwise users might think it's OK, but in fact it won't necessarily work as expected.

@mattab
Piwik Open Source Analytics member

In 12c3fbe: Fixes #3776, Fixes #3787

Adding integration test to test this code.

I also updated [ tracking api reference] to explain "cid" parameter.

cid (requires token_auth to be set) defines the visitor ID for this request. You must set this value to exactly a 16 character hexadecimal string (containing only characters 01234567890abcdefABCDEF). When specified, the Visitor ID will be enforced. This means that if there is no recent visit with this visitor ID, a new one will be created. If a visit is found in the last 30 minutes with your specified Visitor Id, then the new action will be recorded to this existing visit.

@mattab mattab added this to the 1.11 - Piwik 1.11 milestone Jul 8, 2014
@mattab mattab self-assigned this Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment