Closed Bug 1024588 Opened 6 years ago Closed 6 years ago

UA override list system causes sporadic connections to dynamicua.cdn.mozilla.net

Categories

(Firefox OS Graveyard :: Gaia, defect)

All
Gonk (Firefox OS)
defect
Not set

Tracking

(firefox31 fixed, firefox32 fixed, firefox33 fixed, firefox-esr24 unaffected, b2g-v1.3 fixed, b2g-v1.3T fixed, b2g-v1.4 fixed, b2g-v2.0 fixed, b2g-v2.1 fixed)

RESOLVED FIXED
2.0 S4 (20june)
Tracking Status
firefox31 --- fixed
firefox32 --- fixed
firefox33 --- fixed
firefox-esr24 --- unaffected
b2g-v1.3 --- fixed
b2g-v1.3T --- fixed
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

References

Details

Attachments

(1 file)

With the patches in bug 995417 applied (which forbid network connections to non-local sites):

09:41:04     INFO -  *** UTM:SVC TimerManager:notify - notified timerID: user-agent-updates-timer
09:41:04     INFO -  Non-local network connections are disabled and a connection attempt to dynamicua.cdn.mozilla.net was made.  You should only access hostnames available via the test networking proxy (if running mochitests) or from a test-specific httpd.js server (if running xpcshell tests).
09:41:05     INFO -  TEST-INFO | Main app process: killed by SIGSEGV
09:41:05  WARNING -  TEST-UNEXPECTED-FAIL | /tests/dom/imptests/html/dom/nodes/test_case.html | application terminated with exit code 11

(The particular test isn't at fault, so far as I can tell, it's just when the update timer happens to go off that problems ensue.)

These updates should be disabled for all tests, and any tests which specifically depend on the updates being available should use a test-only local server.
Looking at the code, I think it does the right thing when the pref is off.
Testing this on try:

https://tbpl.mozilla.org/?tree=Try&rev=825ca25aa598

Getting tired of setting prefs in multiple places...
Attachment #8439453 - Flags: review?(jmaher)
(In reply to Nathan Froyd (:froydnj) from comment #1)
> Looking at the code, I think it does the right thing when the pref is off.
> Testing this on try:
> 
> https://tbpl.mozilla.org/?tree=Try&rev=825ca25aa598

Looks like the patch works, hooray!
Comment on attachment 8439453 [details] [diff] [review]
turn off useragent updates for tests, to avoid hitting the network

Review of attachment 8439453 [details] [diff] [review]:
-----------------------------------------------------------------

yes, so many prefs.  Should we do this for talos as well?
Attachment #8439453 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/09d24ac458bb
Assignee: nobody → nfroyd
Flags: in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/09d24ac458bb
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.0 S4 (20june)
You need to log in before you can comment on or make changes to this bug.