Closed Bug 1024588 Opened 6 years ago Closed 6 years ago

UA override list system causes sporadic connections to


(Firefox OS Graveyard :: Gaia, defect)

Gonk (Firefox OS)
Not set


(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)

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


(Reporter: froydnj, Assigned: froydnj)




(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 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:

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:

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+
Assignee: nobody → nfroyd
Flags: in-testsuite-
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.