Mirrors/chatmail-core
1
0
Fork 0
mirror of https://github.com/chatmail/core.git synced 2026-10-26 06:48:01 +03:00
Code Issues Packages Projects Releases Wiki Activity

feat: remove ?emailaddress argument from autoconfig URL that is not using a dedicated domain

Browse Source 
This brings the implementation closer to the specification <https://datatracker.ietf.org/doc/draft-ietf-mailmaint-autoconfig/06/>
as it is confirmed that `?emailaddress` should not be included.
This commit is contained in:
link2xt
2026-06-29 18:07:17 +00:00
committed by l
parent b4471f2434
commit a14e617059
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/configure.rs
+4 -4
View File
@@ -721,12 +721,12 @@ async fn get_autoconfig(
}
progress!(ctx, 300);
// `?emailaddress=` query string is excluded on purpose.
// It is not part of the URL according to <https://datatracker.ietf.org/doc/draft-ietf-mailmaint-autoconfig/06/>.
// Related discussion confirming this is at <https://github.com/benbucksch/autoconfig-spec/issues/17>.
if let Ok(res) = moz_autoconfigure(
ctx,
// the doc does not mention `emailaddress=`, however, Thunderbird adds it, see <https://releases.mozilla.org/pub/thunderbird/>, which makes some sense
&format!(
"https://{param_domain}/.well-known/autoconfig/mail/config-v1.1.xml?emailaddress={param_addr_urlencoded}"
),
&format!("https://{param_domain}/.well-known/autoconfig/mail/config-v1.1.xml"),
&param.addr,
accept_invalid_certificates,
)