New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Newer version of Netscape is required"... :s #92

Closed
thomthom opened this Issue Mar 14, 2014 · 3 comments

Comments

Projects
None yet
2 participants
@thomthom
Owner

thomthom commented Mar 14, 2014

2014-03-14_08h50_13

Strange user report. Is it the user agent string of older IE versions that spoofs it's identity? Or some other software that has changed it?

@thomthom thomthom added this to the Version 1.0 milestone Mar 14, 2014

@thomthom thomthom added the bug label Mar 14, 2014

@noelwarr

This comment has been minimized.

noelwarr commented Mar 14, 2014

I had no idea SKUI required some minimum IE version. That's great. Any
idea what the "is" in Netcapeis is?

On 14 March 2014 08:51, Thomas Thomassen notifications@github.com wrote:

[image: 2014-03-14_08h50_13]https://f.cloud.github.com/assets/192418/2418793/5dfaf4e8-ab4d-11e3-9ae3-ec1a29439fca.png

Strange user report. Is it the user agent string of older IE versions that
spoofs it's identity? Or some other software that has changed it?

Reply to this email directly or view it on GitHubhttps://github.com//issues/92
.

@thomthom

This comment has been minimized.

Owner

thomthom commented Mar 14, 2014

Missing space typo.

Looks like MS changed the useragent string in IE11 again:
http://msdn.microsoft.com/en-us/library/ie/bg182625(v=vs.110).aspx

Also the navigator.appName was changed - which might explain Netscape.

But the user agent string doesn't match what I get in the webdialog:

Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.2; WOW64; Trident/7.0; .NET4.0E; .NET4.0C; .NET CLR 3.5.30729; .NET CLR 2.0.50727; .NET CLR 3.0.30729) SketchUp/14.0 (PC)

Sigh... I'm not getting a reliable version number here.

I expect the change is because there are so many broken IE detection that checked only the first character of the version so it picks up IE10 and IE11 as 1 - but this change breaks the correct detection.... argh!

@noelwarr

This comment has been minimized.

noelwarr commented Mar 14, 2014

lol.

Doesn't it make you want to take cover in all that SU C++ code? I hate the
DOM. Not with a passion because I haven't had to deal with it that much.
But I don't think I could live as a web developer without turning to
something like this... http://www.ispot.tv/ad/7oAZ/abilify-add-abilify

On 14 March 2014 09:10, Thomas Thomassen notifications@github.com wrote:

Missing space typo.

Looks like MS changed the useragent string in IE11 again:
http://msdn.microsoft.com/en-us/library/ie/bg182625(v=vs.110).aspx

Also the navigator.appName was changed - which might explain Netscape.

But the user agent string doesn't match what I get in the webdialog:
Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.2; WOW64; Trident/7.0;
.NET4.0E; .NET4.0C; .NET CLR 3.5.30729; .NET CLR 2.0.50727; .NET CLR
3.0.30729) SketchUp/14.0 (PC)

Sigh... I'm not getting a reliable version number here.

I expect the change is because there are so many broken IE detection that
checked only the first character of the version so it picks up IE10 and
IE11 as 1 - but this change breaks the correct detection.... argh!

Reply to this email directly or view it on GitHubhttps://github.com//issues/92#issuecomment-37624654
.

@thomthom thomthom self-assigned this Mar 14, 2014

@thomthom thomthom closed this in 3c72040 Mar 14, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment