Strip dashes in Yahoo addresses (#6284)

2026-08-23 09:15:02 -05:00 · 2025-11-07 13:35:44 -05:00
parent d206a70b8a
commit 8f80996515
1 changed files with 8 additions and 1 deletions
--- a/servatrice/src/email_parser.cpp
+++ b/servatrice/src/email_parser.cpp
@@ -25,7 +25,7 @@ QPair<QString, QString> EmailParser::parseEmailAddress(const QString &dirtyEmail

    // Trim out dots and pluses from Google/Gmail domains
    if (capturedEmailAddressDomain.toLower() == "gmail.com") {
-        // Remove all content after first plus sign (as unnecessary with gmail)
+        // Remove all content after the first plus sign (as unnecessary with gmail)
        // https://gmail.googleblog.com/2008/03/2-hidden-ways-to-get-more-from-your.html
        const auto firstPlusSign = capturedEmailUser.indexOf("+");
        if (firstPlusSign != -1) {
@@ -36,6 +36,13 @@ QPair<QString, QString> EmailParser::parseEmailAddress(const QString &dirtyEmail
        // https://gmail.googleblog.com/2008/03/2-hidden-ways-to-get-more-from-your.html
        capturedEmailUser.replace(".", "");
    }
+    // Trim out minuses from Yahoo domains
+    else if (capturedEmailAddressDomain.toLower() == "yahoo.com") {
+        const auto firstMinusSign = capturedEmailUser.indexOf("-");
+        if (firstMinusSign != -1) {
+            capturedEmailUser = capturedEmailUser.left(firstMinusSign);
+        }
+    }

    return {capturedEmailUser, capturedEmailAddressDomain};
 }