jose-lpa / django-tracking-analyzer

User actions tracking and analytics for Django sites :bar_chart:
GNU General Public License v3.0
96 stars 32 forks source link

Crashes with certain user agent strings #17

Closed owenh000 closed 4 years ago

owenh000 commented 4 years ago

Certain user agent strings, such as xxxxxxxxxxxxxxxxxxxxxxxxxxxxbot, crash this app, causing a 500 error for the visitor. Tested with version 1.0.2.

  1. The example string is parsed by python-user-agents as a browser family string that is longer than 30 characters.
  2. A Tracker object is created using that string for the browser field.
  3. A DataError: value too long for type character varying(30) exception is raised.
jose-lpa commented 4 years ago

This should have been fixed in PR #24 :+1: