859030 - 302 error in browser when access 3g.qq.com browser can not redirected .

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect, P1)

Description

[qichao]

Attachment: the network information for this problem

1. Access 3g.qq.com
2. Browser can not redirect to 3gqq.qq.com
3. Nothing display in the browser.

Comment 1

[gary.ge]

Immanuel Zhu 2013-04-07 14:54:34 CST 
在抓包的数据分析中发现是304 HTTP/1.1 302 Found 这个信息。

类似于301，但新的URL应该被视为临时性的替代，而不是永久性的。注意，在HTTP1.0中对应的状态信息是“Moved Temporatily”。
出现该状态代码时，浏览器能够自动访问新的URL，因此它是一个很有用的状态代码。

注意这个状态代码有时候可以和301替换使用。例如，如果浏览器错误地请求http://host/~user（缺少了后面的斜杠），有的服务器返回301，有的则返回302。

严格地说，我们只能假定只有当原来的请求是GET时浏览器才会自动重定向。请参见307。

Comment 2

[Steven Yang [:styang]]

Hi Ian,
Please help to check this issue first. Thanks!

Comment 3

[[e-mail], [e-mail]]

Attachment: redirect successfully

I type in "3g.qq.com" to be url. Then, it works for me. May I know what the build version you run on?

Comment 4

[[e-mail], [e-mail]]

Looks like it going to wrong page. Just ignore my comment.

Comment 5

[[e-mail], [e-mail]]

I dump some log in function handleBrowserEvent.(https://github.com/mozilla-b2g/gaia/blob/master/apps/browser/js/browser.js#L368) There is no mozbrowsererror event during opening 3g.qq.com. The coming events are same with opening normal page.

Patrick and Ben,
Do you have any idea for the issue?

Comment 6

[Ben Francis [:benfrancis]]

That URL is serving a WML file instead of an HTML file, which Gecko doesn't support. I've noticed the same problem with vds.vivo.com.br which is one of Vivo's default bookmarks in the browser app!

I suspect this is an evangelism bug.

Comment 7

[[e-mail], [e-mail]]

I have tried to type in the url of 3g.qq.com(or vds.vivo.com.br) on Firefox for Ubuntu. It will pop out a downloading dialog for the link. The status sync with Comment 6.

Comment 8

[Patrick Wang (Chih-Kai Wang) (:kk1fff)]

(In reply to Ian Liu [:ianliu] from comment #7)
> I have tried to type in the url of 3g.qq.com(or vds.vivo.com.br) on Firefox
> for Ubuntu. It will pop out a downloading dialog for the link. The status
> sync with Comment 6.

The attached pcap file also says that we did follow the 302 response, and the final destination responsed with "Content-Type: text/vnd.wap.wml".

Comment 9

[[e-mail], [e-mail]]

Hi PM and devs,
Would we plan to support this kind of "Content-Type: text/vnd.wap.wml" in Browser app?

Comment 10

[Marco Chen [:mchen]]

Hi all,

I did a test on firefox browser on Desktop version. And it seems to be related to UA string.

  1. Open 3g.qq.com then browser prompt a dialog for saving "text/vnd.wap.wml".

  2. To install an add-on called "user agent switcher" then add "Android" into UA string.
     User-Agent: Mozilla/5.0 (Android; Mobile; rv:18.0) Gecko/18.0 Firefox/18.0

     After Opening 3g.qq.com again, the page is redirected to content-type with "application/xhtml+xml".

Conclusion:
  From google search page by 3q.qq.com, it show 3q.pp.com is a WML page. I suspect server side will re-direct some device into non-wml page according to UA with Android string. So if Firefox OS doesn't want to support WML then we need the ecosystem to support "redirect firefox OS device to non-wml page".

Comment 11

[Kai-Chih Hu [:khu]]

We don't have the detailed plan yet. But, it's worthy to have this on the list. Noted and thanks for the information.

Comment 12

[Kai-Chih Hu [:khu]]

Marked as koi? to see if it's possible to have this in koi. If not, let's push this to later versions with proper flags on this bug.

Comment 13

[[e-mail], [e-mail]]

re-nominating koi..

Comment 14

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Maybe we could the UA overrides to build/ua-override-prefs.js for this issue, but I am not sure what is the policy on what can go in that file.

ni Lawrence for any comments.

Dear Lawrence,

Could you please let us know what can go into the UA override rules?
Thanks.

Comment 15

[Lawrence Mandel [:lmandel] (use needinfo)]

We consider UA overrides for top global sites (typically using Alexa data) or other key sites for our launch locales. Alexa didn't return data for 3g.qq.com but qq.com is ranked #9 globally, so I would say it meets the criteria. I tested on Firefox for Android and did get mobile content as expected based on comment 10.

One of the requirements for the addition of an UA override is the creation of an evangelism bug and the promise to try to work with the site to fix their UA detection issue. Can someone on this thread commit to trying to speak to the site about correcting their UA detection to recognize Firefox OS as mobile?

Comment 16

[Jason Smith [:jsmith]]

(moving back just to remove nom)

Comment 17

[[e-mail], [e-mail]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9430

Pointer to Github pull-request

Comment 18

[[e-mail], [e-mail]]

Comment on attachment 742303 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9430

Hi Lawrence,
Could you please help to review my pr? And I will try to establish contact with 3g.qq.com website. Hoping the factory to support the issue, then we could remove the it from UA override string list. Thank you.

Hi qichao,
You could try to use the patch, if you think it's an critical issue in China immediately.

Comment 19

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 742303 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9430

If this is to be the bug for adding the UA override, we'll need a separate bug to track the outreach to the site. Please file a bug like bug 848854. Otherwise, looks good.

Comment 20

[[e-mail], [e-mail]]

Comment on attachment 742303 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9430

We have a separate bug to track the outreach to the site via Bug 866577. Thanks for Lawrence's point out.

Comment 21

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 742303 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9430

Looks good to me. Thanks for filing the new evangelism bug.

Comment 22

[[e-mail], [e-mail]]

https://github.com/mozilla-b2g/gaia/pull/9430
Since the pr is merged, we can close the issue now.