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)
Firefox OS Graveyard
Runtime
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)
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.
Reporter | ||
Comment 1•11 years ago
|
||
Updated•11 years ago
|
status-b2g-v1.3:
--- → unaffected
status-b2g-v1.3T:
--- → affected
status-b2g-v1.4:
--- → unaffected
status-b2g-v2.0:
--- → unaffected
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)
Assignee | ||
Comment 4•11 years ago
|
||
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
Assignee | ||
Comment 5•11 years ago
|
||
This restores the code which was removed in bug 870868.
Assignee | ||
Updated•11 years ago
|
Attachment #8418483 -
Flags: review?(fabrice)
Comment 6•11 years ago
|
||
Do we expose the detectable UA to the world? Why don't you use the site-override like bug 988725?
Comment 7•11 years ago
|
||
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+
Comment 8•11 years ago
|
||
(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.
Comment 9•11 years ago
|
||
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)
Assignee | ||
Comment 10•11 years ago
|
||
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!
Comment 11•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(lmandel)
Resolution: --- → FIXED
Updated•11 years ago
|
blocking-b2g: 1.3T? → 1.3T+
Comment 12•11 years ago
|
||
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
Comment 13•11 years ago
|
||
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
You need to log in
before you can comment on or make changes to this bug.
Description
•