New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client detection #5413

Open
sgiehl opened this Issue Jul 4, 2014 · 4 comments

Comments

Projects
None yet
4 participants
@sgiehl
Member

sgiehl commented Jul 4, 2014

Currently Piwik is only able to detect browsers.
Instead of detecting only browsers we should open up piwik to client based detection.
Clients can be browsers, as well as media players, mobile apps, personal information managers (like mail clients), feed readers and so on.
The DeviceDetector used by Piwik is already able to detect such clients.
As it is impossible to define short codes for all those possible clients, we might need to move away from only saving short codes (like it is actually done for browsers)

With client based detection there will be more reports possible within Piwik. So it would be nice to see which client types mostly visits a website and os on.

@sgiehl sgiehl added this to the 2.x - The Great Piwik 2.x Backlog milestone Jul 8, 2014

@ThaDafinser

This comment has been minimized.

Show comment
Hide comment
@ThaDafinser

ThaDafinser Jul 29, 2014

Contributor

@sgiehl i would go further...save all client $_SERVER vars serialized in one field inside piwik_log_visit.
Pro:

  • at an upgrade you can extract them, if other table fields need to be field
  • you can update the short codes or other things at an upgrade

Con:

  • possible overflow (TEXT can be too small....i had it one time as LARGETEXT....)
  • needs a lot of space
Contributor

ThaDafinser commented Jul 29, 2014

@sgiehl i would go further...save all client $_SERVER vars serialized in one field inside piwik_log_visit.
Pro:

  • at an upgrade you can extract them, if other table fields need to be field
  • you can update the short codes or other things at an upgrade

Con:

  • possible overflow (TEXT can be too small....i had it one time as LARGETEXT....)
  • needs a lot of space

@mattab mattab removed the c: Core label Aug 3, 2014

@mattab mattab added this to the Short term milestone Aug 3, 2014

@mattab mattab added the c: Platform label Sep 9, 2014

@mattab

This comment has been minimized.

Show comment
Hide comment
@mattab

mattab Apr 7, 2015

Member

Hi @sgiehl - great idea of a new report for Piwik.

Do you maybe have some idea on approx estimate for this issue?

Member

mattab commented Apr 7, 2015

Hi @sgiehl - great idea of a new report for Piwik.

Do you maybe have some idea on approx estimate for this issue?

@sgiehl

This comment has been minimized.

Show comment
Hide comment
@sgiehl

sgiehl Apr 7, 2015

Member

I think that would take a few days for implementation. We need to built a switch between the different client types and therefor adjust the table structure as we need additional information. Currently all client types other than browser will be reported as "unknown" browser, which will then be excluded from the browser reports. Maybe I can provide a detailed list of todos as soon as I have some time for that.

I would love to see that in the near future!

Member

sgiehl commented Apr 7, 2015

I think that would take a few days for implementation. We need to built a switch between the different client types and therefor adjust the table structure as we need additional information. Currently all client types other than browser will be reported as "unknown" browser, which will then be excluded from the browser reports. Maybe I can provide a detailed list of todos as soon as I have some time for that.

I would love to see that in the near future!

@mattab mattab modified the milestones: Mid term, Short term Apr 7, 2015

@mattab mattab added duplicate and removed Major labels Dec 5, 2016

@mattab mattab modified the milestones: Long term, Mid term Dec 5, 2016

@mattab mattab removed the duplicate label Dec 5, 2016

@mattab mattab added the Major label Feb 28, 2017

@bigretromike

This comment has been minimized.

Show comment
Hide comment
@bigretromike

bigretromike May 17, 2017

is near feature now 👍 ?

bigretromike commented May 17, 2017

is near feature now 👍 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment