957759 - Remove experiment to test reachability of SRV records

Status: RESOLVED FIXED

(Core :: Networking: DNS, defect)

Description

[u408661]

+++ This bug was initially created as a clone of Bug #942317 +++

We're done with the SRV experiment, we can get rid of the code.

Comment 1

[u408661]

Attachment: patch

This is for m-c. The patch for m-a will be almost identical, but since it never got the null check from bug 950888, that will change the patch ever so slightly.

Comment 2

[Patrick McManus [:mcmanus]]

Comment on attachment 8357358 [details] [diff] [review]
patch

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

this is fine, and I support aurora uplift to fix a topcrash there.

(for documentation purposes, this was designed to actually just be removed from nightly when we were done with it and naturally expire in earlier releases, but given that it needs a bugfix to be stable on those earlier releases it makes more sense to just back it out of aurora. beta is not impacted.)

Comment 3

[u408661]

try run: https://tbpl.mozilla.org/?tree=Try&rev=6fa332241c36

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/731d86661319

Comment 4

[u408661]

Attachment: patch for aurora

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 942317
User impact if declined: topcrash will continue (see bug 950888) without a different patch
Testing completed (on m-c, etc.): on m-c, this is purely a backout
Risk to taking this patch (and alternatives if risky): low 
String or IDL/UUID changes made by this patch: none

Version of the patch for aurora. It's effectively identical to the original patch (the original also had a simple fix for bug 950888, which makes this patch not quite text-identical), so I don't imagine Patrick's r+ will be an issue, but better safe than sorry.

Comment 5

[Patrick McManus [:mcmanus]]

Comment on attachment 8358015 [details] [diff] [review]
patch for aurora

r+ based on comment

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/731d86661319

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/237a6c25de39