Opened 6 years ago

Closed 6 years ago

#7645 closed defect (fixed)

Browse loses on comcast.net

Reported by: jg Owned by: erikos
Priority: blocker Milestone: 8.2.0 (was Update.2)
Component: browse-activity Version: not specified
Keywords: hulahop-0.4.3:+ joyride-2228:+ Cc: mstone, marco, simon
Blocked By: Blocking:
Deployments affected: Action Needed: qa signoff
Verified: no

Description

Visit www.comcast.net.
Notice you lose completely.

It appears, in a very obnoxious way that comcast is trying to do some animation of telling you to upgrade the browser.

a) comcast.net is broken that it doesn't tolerate some browser strings.
b) I'm glad they are trying to get people to upgrade their browser, some will use firefox. But... they do it in a horrid way....
c) we need some way to set the browser string, or for if other ISP's do likewise, we may not even be able to get internet connectivity.
d) this could be a support nightmare for g1g1 (I wanted to get to comcast.net to read my mail, and comcast is one of the U.S.'s biggest ISP's.

Change History (17)

comment:1 Changed 6 years ago by jg

  • Priority changed from normal to blocker

comment:2 Changed 6 years ago by jg

  • Summary changed from Browse loses on comcas.net to Browse loses on comcast.net

comment:3 Changed 6 years ago by marco

It seem to work fine in epiphany to me. Maybe something with our UA... will have to investigate.

comment:4 Changed 6 years ago by marco

  • Priority changed from blocker to high

comment:5 Changed 6 years ago by marco

Simon, Jim, can you update on this one? I've seen discussion about it in irc.

comment:6 Changed 6 years ago by jg

  • Priority changed from high to blocker

heh. Comcast loses, misinterprets the user agent string, and you end up with a blank page (that is reloading, eating your machine entirely).

Comcast is a major U.S. ISP; so this one is a real headache for G1G1 folks.

Note that this is the page you'd go to to log in to get your web mail....

The question is if we should redo our browser string in some fashion....

Or have a way to set it explicitly.

As it is such an issue for G1G1, I'm putting it back to blocker....

comment:7 Changed 6 years ago by marco

epiphany has this on Fedora:

Mozilla/5.0 (X11; U; Linux i686; en; rv:1.9.0.1) Gecko/20080528 Fedora/2.22.2-3.fc9 Epiphany/2.22 Firefox/3.0

And it works with comcast. We should compare it with Browse one and see why we lose.

comment:8 Changed 6 years ago by jg

We currently have:

Mozilla/5.0 (X11; U; Linux i586, en-US, rv:1.9) Gecko OLPC/Update.1 (XO)

Betcha the lack of Firefox/3 is doing it....

comment:9 Changed 6 years ago by erikos

We need to add the following line to prefs.js to make comcast work - epiphany is doing the same.

pref("general.useragent.extra.firefox", "Firefox/3.0.1");

comment:10 Changed 6 years ago by erikos

  • Action Needed changed from diagnose to review
  • Keywords r? added

Epiphany is using as well a string to distinct the epiphany version.

  • pref("general.useragent.extra.epiphany", "Epiphany/2.0.1");

We could do the same for hulahop - not sure we need it though.

comment:11 Changed 6 years ago by marco

  • Keywords r+ added; r? removed

I'd do both. It's nice for statistics.

comment:12 Changed 6 years ago by marco

Please provide a testcase.

comment:13 Changed 6 years ago by marco

  • Action Needed changed from review to code
  • Keywords r+ removed

Actually, it would be better to not hardcode the version number. We can talk about it later an see if we can autogenerate it.

comment:14 Changed 6 years ago by marco

  • Action Needed changed from code to review
  • Keywords r+ added

Firefox version can't be figured out dinamically. So I suggest to add the Firefox bit as an hardcoded pref and to just skip the extra.epiphany for now (not necessary to fix this issue). r+ on that

comment:15 Changed 6 years ago by erikos

  • Action Needed changed from review to package

|TestCase|

open browse and load the page comcast.net; the page should be able to load

comment:16 Changed 6 years ago by erikos

  • Action Needed changed from package to qa signoff
  • Keywords hulahop-0.4.3:+ joyride-2228:+ added; r+ removed

Tested in joyride-2228 as working.

comment:17 Changed 6 years ago by marco

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.