Closed
Bug 1030093
Opened 11 years ago
Closed 11 years ago
Talos should use a dummy experiments.manifest.uri to prevent external connections to telemetry-experiment.cdn.mozilla.net
Categories
(Testing :: Talos, defect)
Testing
Talos
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(1 file, 1 obsolete file)
1.38 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
In bug 996871, we set a dummy server URI for toolkit.telemetry.server, however it seems that the A/B testing experiments are under a different pref, so we need to do the same for that too, in order to prevent:
Non-local network connections are disabled and a connection attempt to telemetry-experiment.cdn.mozilla.net (23.203.212.61) was made.
https://tbpl.mozilla.org/php/getParsedLog.php?id=42428257&tree=Try#error0
It seems that the pref was set for mochitest in bug 989137, but not talos.
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 1•11 years ago
|
||
The talos equivalent of:
http://mxr.mozilla.org/mozilla-central/source/testing/profiles/prefs_general.js#61
Attachment #8445823 -
Flags: review?(benjamin)
Comment 2•11 years ago
|
||
Assignee: emorley → jmaher
Attachment #8445824 -
Flags: review?(emorley)
Updated•11 years ago
|
Attachment #8445824 -
Flags: review?(emorley)
Assignee | ||
Updated•11 years ago
|
Attachment #8445824 -
Attachment is obsolete: true
Assignee | ||
Updated•11 years ago
|
Assignee: jmaher → emorley
Comment 3•11 years ago
|
||
Comment on attachment 8445823 [details] [diff] [review]
Use a dummy experiments.manifest.uri to prevent external connections
Review of attachment 8445823 [details] [diff] [review]:
-----------------------------------------------------------------
stealing this as bsmedberg suggested this in bug 996871.
Attachment #8445823 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 4•11 years ago
|
||
Ty:-)
remote: https://hg.mozilla.org/build/talos/rev/610a752cfd46
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•