Closed
Bug 109138
Opened 23 years ago
Closed 23 years ago
platform detection not working on macintosh
Categories
(Bugzilla :: Creating/Changing Bugs, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: endico, Assigned: timeless)
References
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
bbaetz
:
review+
dkl
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
bbaetz
:
review+
gerv
:
review+
|
Details | Diff | Splinter Review |
This an email sent to me by Andy Lyttle the day after the b.m.o
upgrade to the bugzilla cvs tip. Did the mozilla user agent
change or did bugzilla detection?
---------------------------
I clicked the link to
http://bugzilla.mozilla.org/enter_bug.cgi?product=Browser and got this:
Possible bug database corruption has been detected. Please send mail to
endico@mozilla.org with details of what you were doing when this message
appeared. Thank you.
src = All Windows 3.1 Windows 95 Windows 98 Windows ME Windows 2000 Windows
NT Windows XP Mac System 7 Mac System 7.5 Mac System 7.6.1 Mac System 8.0
Mac System 8.5 Mac System 8.6 Mac System 9.x MacOS X Linux BSDI FreeBSD
NetBSD OpenBSD AIX BeOS HP-UX IRIX Neutrino OpenVMS OS/2 OSF/1 Solaris SunOS
other
default = Macintosh
I'm logged in as mozilla@phroggy.com.
-=[ Andy Lyttle,
Reporter | ||
Comment 1•23 years ago
|
||
Andy, what browser are you using? What version?
Comment 2•23 years ago
|
||
Mozilla 2001110808 on Mac OS 9.2.1. User-agent string is:
Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:0.9.5+) Gecko/20011108
Comment 3•23 years ago
|
||
The problem is on line 251 of enter_bug.cgi. This function returns an OS which
is not in Bugzilla's default list. Obviously they changed the Mac user-agent
string to be less specific about the OS at some point in the recent past.
We have two options: Add "Macintosh" to b.m.o as a possible OS (which seems very
sensible to me; a whole load of MacOS bugs are across all pre-OSX versions) or
change it to return "9.x" when it sees "Macintosh".
We also need to add OS detection code for MacOS X to that function while we are
there. I dimly remember a bug about the Windows XP detection - why's that
commented out?
Gerv
Comment 4•23 years ago
|
||
A few other user-agent strings for you. These have the same problem:
Mozilla/4.75C-CCK-MCD {C-UDP; EBM-APPLE} (Macintosh; U; PPC)
iCab/2.6 (Macintosh; I; PPC)
Opera/5.0 (Macintosh;US;PPC) TP [en]
These work fine:
Mozilla/4.0 (compatible; MSIE 5.0; Mac_PowerPC)
Lynx/2.7.1 (MacOS b1) libwww-FM/unknown
Reporter | ||
Comment 5•23 years ago
|
||
quoth dbaron:
-----------------------------------
I can't file new bugzilla bugs with the mach-o build. The user agent string of
that build is:
Mozilla/5.0 (Macintosh; U; Darwin Power Macintosh; en-US; rv:0.9.5+) Gecko/20011108
and bugzilla gives me the error message (after going to
enter_bug.cgi?product=Browser):
Possible bug database corruption has been detected. Please send mail to
endico@mozilla.org with details of what you were doing when this message
appeared. Thank you.
src = All Windows 3.1 Windows 95 Windows 98 Windows ME Windows 2000 Windows NT
Windows XP Mac System 7 Mac System 7.5 Mac System 7.6.1 Mac System 8.0 Mac
System 8.5 Mac System 8.6 Mac System 9.x MacOS X Linux BSDI FreeBSD NetBSD
OpenBSD AIX BeOS HP-UX IRIX Neutrino OpenVMS OS/2 OSF/1 Solaris SunOS other
default = Macintosh
-David
XP is commented out for two reasons.
#1 the bugzilla project CVS doesn't have XP in its list.
#2 i don't have a list of user agents from XP.
fwiw bugzilla.mozilla.org is running w/ the lines active.
http://bugzilla.mozilla.org/enter_bug.cgi?product=Browser&op_sys=Windows%20XP
As for this bug, it's mine. I suck.
Beyond that, what i'd like as part of some future bug is:
Hardware: MIPS, PPC, IA32, IA64, Alpha, 68k, ARM
Toolkit: Win16, Win32, MacOS/Carbon, MacOSX/Cocoa, Gtk, Qt, Xlib, Photon, BeOS
OS: <something like the current list>
What would then happen is I could get good hits for 2/3 of them and mediocre
hits for the 3rd in average cases.
Specifically...
For unices i'll be able to get Hardware and OS and i'll probably have to guess
(gtk) for Toolkit.
For windows i'll get Toolkit and usually OS (w/ 90% accuracy) and have to guess
Hardware for the NT4 case (95% accuracy for dumb luck, or perhaps not guessing
because the user agents will be nice. not sure, need info).
For macos i'll get Hardware and should get Toolkit for Classic (guesswork for
the OSX CFM v. Mach) with some guessing for OS (Classic, hopefully OSX will be
distinguishable).
For BeOS i'll get OS, and hopefully toolkit (unless people have fun and build an
X11 toolkit, in which case see unices), depending on niceness of browser i
should get Hardware (BeOS runs on IA32 and PPC).
For QNX i'll hopefully get OS (Opera lies!) and toolkit (X11 caveat...) and
depending on niceness of browser i should get Hardware (QNX runs on more
platforms than BeOS...).
Assignee: myk → timeless
Severity: normal → critical
Keywords: regression
OS: Linux → All
Hardware: PC → Macintosh
Target Milestone: --- → Bugzilla 2.16
Version: 2.10 → 2.15
For the Macintosh case, I'm going to go w/ 8.6 as the conservative guess unless
people really suggest 8.5 or 9.
For the OSX case, I'll gladly catch darwin, patch coming.
Does anyone know what 'TP' means from the Opera Useragent? I'm guessing
Technologoy Preview which would mean it's the Carbon release.
Status: NEW → ASSIGNED
Comment 8•23 years ago
|
||
I think Mac OS 9 is more common than 8.6; I'd suggest that as the Mac Classic
default.
Comment 9•23 years ago
|
||
"with some guessing for OS (Classic, hopefully OSX will be
distinguishable)."
If you're talking about distinguishing between Classic browsers running in OS9
or OSX, you can't. If you mean Carbon browsers, the browser might tell you, but
if it doesn't you'd have to guess (based on knowing things like MSIE 5.0 is
Classic and MSIE 5.1 is OSX-only):
Mozilla/4.0 (compatible; MSIE 5.0; Mac_PowerPC)
Mozilla/4.0 (compatible; MSIE 5.12; Mac_PowerPC)
Mozilla/4.5 (compatible; OmniWeb/4.0.5; Mac_PowerPC)
Comment 10•23 years ago
|
||
Also see bug 106993, which is the reason for the error. IF that was fixed, then
you'd get default values, but it would still work.
Assignee | ||
Comment 11•23 years ago
|
||
Ok, for consistency i'm picking 8.5 instead of 8.6.
Also fixing 68k => OS8.0 since 8.5 requires PPC
Comment 12•23 years ago
|
||
Comment on attachment 57144 [details] [diff] [review]
Darwin => OSX, random Macintosh => OS8.5
Without the patch I get the error. With the patch I don't, but the script
incorrectly detects my OS as "Mac System 8.5" even though I am running Mozilla
on MacOS X?
Still, it works like it used to now (i.e. it isn't broken), so r=myk on the
same patch with the "editusers.cgi" cruft removed.
Attachment #57144 -
Flags: review+
Comment 13•23 years ago
|
||
Comment on attachment 57144 [details] [diff] [review]
Darwin => OSX, random Macintosh => OS8.5
I don't have a mac, so I can't test. It looks right, though. The real bug is
bug 106993.
diff -0 makes it hard to review. Please don't use that option.
Attachment #57144 -
Flags: review+
Comment 14•23 years ago
|
||
I assume the strictvaluechecks have been turned on, so that's why we're getting
this error all of the sudden?
Comment 15•23 years ago
|
||
Strict value checks were on before.
Assignee | ||
Comment 16•23 years ago
|
||
Attachment #57144 -
Attachment is obsolete: true
Comment 17•23 years ago
|
||
Comment on attachment 57193 [details] [diff] [review]
per smfr and mpt picking 9.0 as the default, and adding a bit more case checking
r=bbaetz
Attachment #57193 -
Flags: review+
Comment 18•23 years ago
|
||
Comment on attachment 57193 [details] [diff] [review]
per smfr and mpt picking 9.0 as the default, and adding a bit more case checking
r=dkl
Attachment #57193 -
Flags: review+
Assignee | ||
Comment 19•23 years ago
|
||
Fixed. can someone please update bugzilla.mozilla.org asap so that I don't scare
away too many mac users?
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 20•23 years ago
|
||
b.m.o. has now been updated with this patch.
Comment 21•23 years ago
|
||
reopening per request of timeless, after i complained to him that Mozilla/5.0
(Macintosh; U; PPC Mac OS X; en-US; rv:0.9.7) Gecko/20011221 was being detected
as mac os 9.x.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 22•23 years ago
|
||
Comment 23•23 years ago
|
||
Comment on attachment 66723 [details] [diff] [review]
catch Mac OS X"
You know you should have used a new bug for this...
r=bbaetz, if you're tested (I can't, really)
Attachment #66723 -
Flags: review+
Comment 24•23 years ago
|
||
Comment on attachment 66723 [details] [diff] [review]
catch Mac OS X"
r=gerv. It can't break anything, and it probably does thejob :-)
Gerv
Attachment #66723 -
Flags: review+
Assignee | ||
Comment 25•23 years ago
|
||
committed
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Comment 26•23 years ago
|
||
*** Bug 124546 has been marked as a duplicate of this bug. ***
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•