Email parsing improved

Sources/Common/Extensions/URLExtension.swift

@@ -176,7 +176,23 @@ extension URL {
                urlWithScheme.port != nil || urlWithScheme.user != nil {
                 // could be a local domain but user needs to use the protocol to specify that
                 // make exception for "localhost"
-                guard urlWithScheme.host?.contains(".") == true || urlWithScheme.host == .localhost else { return nil }
+                let hasDomain = urlWithScheme.host?.contains(".") == true
+                guard hasDomain || urlWithScheme.host == .localhost else { return nil }
+
+                let isInvalidUserInfo = {
+                    let hasUser = urlWithScheme.user != nil
+                    let hasPassword = urlWithScheme.password != nil
+                    let hasPath = !urlWithScheme.path.isEmpty
+                    let hasPort = urlWithScheme.port != nil
+                    let hasFragment = urlWithScheme.fragment != nil
+
+                    return hasUser && !hasPassword && !hasPath && !hasPort && !hasFragment
+                }()
+
+                if isInvalidUserInfo {
+                    return nil
+                }
+
                 self = urlWithScheme
                 return
 
@@ -223,6 +239,10 @@ extension URL {
 
         guard let (authData, urlPart, query) = Self.fixupAndSplitURLString(s) else { return nil }
 
+        if (authData?.contains(" ") == true) || urlPart.contains(" ") {
+            return nil
+        }
+
         let componentsWithoutQuery = urlPart.split(separator: "/").map(String.init)
         guard !componentsWithoutQuery.isEmpty else {
             return nil

Tests/CommonTests/Extensions/URLExtensionTests.swift

@@ -551,6 +551,14 @@ final class URLExtensionTests: XCTestCase {
         XCTAssertEqual(result.last, .init(name: "another_item", value: "test_2"))
     }
 
+    func testWhenUserInfoDoesNotContaintPassword_ThenNavigateToSearch() {
+        XCTAssertNil(URL(trimmedAddressBarString: "[email protected]"))
+        XCTAssertNil(URL(trimmedAddressBarString: "user: @domain.com"))
+
+        XCTAssertEqual(URL(trimmedAddressBarString: "user:,,@domain.com")?.host, "domain.com")
+        XCTAssertEqual(URL(trimmedAddressBarString: "user:[email protected]")?.host, "domain.com")
+    }
+
 }
 
 extension String {