Closed Bug 1573564 Opened 3 months ago Closed 3 months ago
Config` tries to await the DNS lookup
Comment on attachment 9085130 [details] [diff] [review] Proposed patch Thanks for catching and fixing this. The patch looks good to me. This obsolete comment from the `getMX` function doc should be removed: ``` * The promise-based async call to `DNS.mx` (imported from DNS.jsm) is at odds * with the other code here that works with callbacks and `Abortable` objects. * That makes the code less coherent, but otherwise it just means we can't * abort the initial MX query. ```
Attachment #9085130 - Flags: feedback+
Comment on attachment 9085130 [details] [diff] [review] Proposed patch r+ BenB @Neil: Could you please make a diff to the state before bug 1349337? I'd like to verify that really only the implementation of `getMX()` has changed.
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 70.0
You need to log in before you can comment on or make changes to this bug.