Opened 9 years ago

Closed 9 years ago

Last modified 20 months ago

#5269 closed enhancement (fixed)

Browser Identification is not descriptive

Reported by: QuantumG Owned by: marco
Priority: normal Milestone:
Component: browse-activity Version:
Keywords: browser, identification, HTTP_USER_AGENT Cc: erikos
Blocked By: Blocking:
Deployments affected: Action Needed:
Verified: no


HTTP_USER_AGENT for Browse activity should include the string "OLPC" and perhaps the OS build and laptop version, etc.

This will aid web developers to layout content appropriately and provide a good browser statistic metric.

Attachments (1)

hulahop-prefs.diff (466 bytes) - added by QuantumG 9 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 9 years ago by QuantumG

This can be done by adding the following to /usr/share/hulahop/prefs.js

pref("general.useragent.vendor", "OLPC");
pref("general.useragent.vendorSub", "1.1");
pref("general.useragent.vendorComment", "XO2");

and I also notice that the locale currently isn't set, so:

pref(general.useragent.locale", "en-US");

for example, otherwise you send the unhelpful string of "chrome://navigator/locale/" for locale.

Changed 9 years ago by QuantumG

comment:2 Changed 9 years ago by QuantumG

Here's a patch.

comment:3 Changed 9 years ago by jg

  • Milestone changed from Never Assigned to Update.1

comment:4 Changed 9 years ago by erikos

Thanks QuantumG works great will add it to the hulahop package.

comment:5 Changed 9 years ago by erikos

  • Cc erikos added
  • Owner changed from erikos to ApprovalForUpdate

Tested in joyride 1416.

Way to verify: document.write(navigator.userAgent))

Who verifies in update.1: erikos

The package that needs to go into update.1 for this fix is:

comment:6 Changed 9 years ago by jg

  • Owner changed from ApprovalForUpdate to dgilmore


comment:7 Changed 9 years ago by marco

This is not in the build yet, please get it in as soon as possible.

comment:8 Changed 9 years ago by dgilmore

  • Owner changed from dgilmore to marco

please test build 661

comment:9 Changed 9 years ago by erikos

  • Resolution set to fixed
  • Status changed from new to closed

Tested as working fine in 661.

comment:10 Changed 20 months ago by Quozl

  • Milestone Update.1 deleted

Milestone Update.1 deleted

Note: See TracTickets for help on using tickets.