Closed Bug 1006830 Opened 11 years ago Closed 11 years ago

Tarako user agent says 28.0 not 28.1

Categories

(Firefox OS Graveyard :: Runtime, defect)

defect
Not set
normal

Tracking

(blocking-b2g:1.3T+, b2g-v1.3 unaffected, b2g-v1.3T verified, b2g-v1.4 unaffected, b2g-v2.0 unaffected)

VERIFIED FIXED
blocking-b2g 1.3T+
Tracking Status
b2g-v1.3 --- unaffected
b2g-v1.3T --- verified
b2g-v1.4 --- unaffected
b2g-v2.0 --- unaffected

People

(Reporter: clouserw, Assigned: ehsan.akhgari)

References

Details

Attachments

(3 files)

Attached image settings page
Over in bug 987450 we changed the Gecko Version to 28.1 so that Tarako could be detected via the User Agent on our sites (namely, the Marketplace). If you look at the Platform Version in the settings of the phone we see "28.1" as expected, but when looking at the User Agent on a website we see 28.0. Attaching screenshots. I'm marking this as 1.3T? because, as far as I know, the primary reason for having a 28.1 was the UA detection.
Attached image user agent screenshots
Blocks: 992404
We'll want this change. It will help post market if we should try to get other webpages to tune more for the tarako device like facebook and twitter and I think there will be little risk in a patch for this.
Flags: needinfo?(jcheng)
The app version comes from <http://mxr.mozilla.org/mozilla-central/source/netwerk/protocol/http/nsHttpHandler.cpp#281>. MOZILLA_UA_VERSION is set here <http://mxr.mozilla.org/mozilla-central/source/js/src/configure.in#240>, which is set in <http://mxr.mozilla.org/mozilla-b2g28_v1_3/source/config/milestone.pl#77>. Note the comment there, the patch level is intentionally hidden in the UA string.
Assignee: nobody → ehsan
This restores the code which was removed in bug 870868.
Attachment #8418483 - Flags: review?(fabrice)
Do we expose the detectable UA to the world? Why don't you use the site-override like bug 988725?
Comment on attachment 8418483 [details] [diff] [review] Show the patch level in the UA string for 1.3t; r=fabrice Review of attachment 8418483 [details] [diff] [review]: ----------------------------------------------------------------- I was about to ask you to review the same patch ;)
Attachment #8418483 - Flags: review?(fabrice) → review+
(In reply to Masatoshi Kimura [:emk] from comment #6) > Do we expose the detectable UA to the world? Why don't you use the > site-override like bug 988725? Yes we expose that to the world.
in general i agree with all the previous comments to plus this bug for tarako Since i am not very familiar with the impact of changing User Agent, ni? Lawrence Mandel for final input before plusing
Flags: needinfo?(jcheng) → needinfo?(lmandel)
I don't know what the landing rules for 1.3t are, but please land this patch *only* on 1.3t and on no other branches. Thanks!
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(lmandel)
Resolution: --- → FIXED
blocking-b2g: 1.3T? → 1.3T+
I'm late to comment but agree with this change. We have precedent as we took the same action in B2G 1.1. (See [1] for reference.) [1] https://developer.mozilla.org/en-US/docs/Gecko_user_agent_string_reference#Firefox_OS_version_number
Verified that User Agent is displaying appropriately on the web on the latest 1.3T build. User navigated to http://whatsmyuseragent.com and the site displayed the following user agent: User Agent: Mozilla/5.0 (Mobile; rv:28.1) Gecko/28.1 Firefox/28.1 1.3T Environmental Variables: Device: Tarako 1.3T BuildID: 20140602014001 Gaia: 335486c42498fa7a93c21e4d6121199728602ab8 Gecko: 55e4d83019e5 Version: 28.1 Firmware Version: sp6821a-gonk-4.0-5-12 User Agent: Mozilla/5.0 (Mobile; rv:28.1) Gecko/28.1 Firefox/28.1
Status: RESOLVED → VERIFIED
Blocks: 1026317
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: