1518890 - [autoconfig] Pass Exchange URL from account creation dialog to addon

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, enhancement, P3)

Description

[Ben Bucksch (:BenB)]

The account creation dialog runs the Exchange AutoDiscover methods and finds the Exhange protocol URLs (OWA and EWS URL). It then extracts the hostname from it, creates a normal nsIMsgIncomingServer and throws away the URLs. Owl the synthezises the URL from the hostname again. Obviously, that's information loss. Even though it happens to work in all cases we know, it's still not smart.

The account creation wizard, when creating the account, should set the preferences <server>.owa_url, <server>.ews_url and <server>.eas_url , if these URLs have been found during AutoDiscover. The Exchange addon can then use these URLs to contact the server.

Comment 1

[Ben Bucksch (:BenB)]

Attachment: Fix, v2

This also fixes up the existing POP3 code which was ...unnecessarily long (even when not considering the nsIMsgIncomingServer.set*Pref() API).

Comment 2

[Ben Bucksch (:BenB)]

Attachment: Fix, v3

Caught a leftover during self-review.

Comment 3

[[e-mail]]

Comment on attachment 9035420 [details] [diff] [review]
Fix, v3

+  if (config.incoming.owaURL) {
+    inServer.setCharValue("owa_url", config.incoming.owaURL);
+  }
+  if (config.incoming.ewsURL) {
+    inServer.setCharValue("ews_url", config.incoming.ewsURL);
+  }
+  if (config.incoming.easURL) {
+    inServer.setCharValue("eas_url", config.incoming.easURL);
  }

Please use setUnicharValue in case the URL contains Unicode. r=me with that fixed.

Comment 4

[Ben Bucksch (:BenB)]

Attachment: Fix, v4

• Use setUnicharValue()

Comment 5

[Ben Bucksch (:BenB)]

User impact if declined:
Minimal. This affects only users where the OWA URL is not https://host/owa/, but e.g. https://host/OWA/. Even in these cases, /owa/ might be redirected to /OWA/ and work anyway, just a tiny little slower due to the redirect. So, this is mostly a correctness fix.
Developer impact if declined:
Other autoconfig patches might be harder to backport later.
Risk:
Affects Owl and some advanced POP3 settings. Behavior for the POP3 should not change, just the code is streamlined.

Comment 6

[Jorg K (CEST = GMT+2)]

Why the need for tracking? Request beta uplift on the patch.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/1ff4cbc59aae
[autoconfig] Pass Exchange URL from account creation dialog to addon. r=Neil

Comment 8

[Jorg K (CEST = GMT+2)]

Ben, can you please request uplifts as required. I am constantly monitoring potential uplifts and the sooner you request, the better, avoids missing the next uplift I'm doing. Currently I have like five bugs, so it's going to happen soon.

Comment 9

[Ben Bucksch (:BenB)]

Comment on attachment 9035652 [details] [diff] [review]
Fix, v4

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: None

User impact if declined: Minimal. This affects only users where the OWA URL is not https://host/owa/, but e.g. https://host/OWA/. Even in these cases, /owa/ might be redirected to /OWA/ and work anyway, just a tiny little slower due to the redirect. So, this is mostly a correctness fix.

Developer impact if declined:
Other autoconfig patches might be harder to backport later.

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: Yes

If yes, steps to reproduce:

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Affects Owl and some advanced POP3 settings. Behavior for the POP3 should not change, just the code is streamlined.

String changes made/needed:

Comment 10

[Ben Bucksch (:BenB)]

oops, some of the fields defaulted to wrong values.

Comment 11

[Ben Bucksch (:BenB)]

Comment on attachment 9035652 [details] [diff] [review]
Fix, v4

Sorry, I'm totally confused about all these flags in different places. (I do understand the differences, I was just confused.) Sorry for the noise. This should be the right one.

[Approval Request Comment]
See comment 5

Comment 12

[Jorg K (CEST = GMT+2)]

TB 65 beta 3:
https://hg.mozilla.org/releases/comm-beta/rev/492bea4ded8961c2171ba05a185555c5e6deb7af

Comment 13

[Jorg K (CEST = GMT+2)]

Comment on attachment 9035652 [details] [diff] [review]
Fix, v4

Using rebased patch from Neil's try. Same as this one with white-space adjustments.

Comment 14

[Jorg K (CEST = GMT+2)]

TB 60.5.0 ESR:
https://hg.mozilla.org/releases/comm-esr60/rev/794569c5127d5eb3d7cbdcc0a2c7c2a71a8557fd