From 7f5589c01baf8d8b1af78765fcf2120a62982244 Mon Sep 17 00:00:00 2001 From: Travis Ralston Date: Tue, 14 May 2019 12:53:13 -0600 Subject: [PATCH] Clarify comment on is_url and hs_url handling We don't actually need to do anything because the app transparently handles this. See https://github.com/vector-im/riot-web/issues/9290 --- src/vector/index.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/vector/index.js b/src/vector/index.js index 9c5eb061ac..17cf27a3a5 100644 --- a/src/vector/index.js +++ b/src/vector/index.js @@ -449,8 +449,9 @@ async function loadLanguage() { async function verifyServerConfig() { console.log("Verifying homeserver configuration"); - // TODO: TravisR - Handle query string arguments for hs_url and is_url - // We probably don't want to handle them unless the user is logged out though? + // Note: the query string may include is_url and hs_url - we only respect these in the + // context of email validation. Because we don't respect them otherwise, we do not need + // to parse or consider them here. const config = SdkConfig.get(); let wkConfig = config['default_server_config']; // overwritten later under some conditions