Changeset 12493 for sites/trunk/api.wordpress.org/public_html/dotorg/helpscout/common.php

Timestamp:

03/24/2023 05:40:23 AM (3 years ago)

Author:

dd32

Message:

Helpscout: Better handling of bounce emails.

This fixes some edgecases where the wordpress user would incorrectly be matched.

File:

• sites/trunk/api.wordpress.org/public_html/dotorg/helpscout/common.php (modified) (3 diffs)

sites/trunk/api.wordpress.org/public_html/dotorg/helpscout/common.php

@@ -80 +80 @@
     $subject = $request->ticket->subject ?? '';
     $user    = get_user_by( 'email', $email );
-
-    // Ignore @wordpress.org "users", unless it's literally the only match (The ?? $email fallback at the end).
-    if ( $user && str_ends_with( $user->user_email, '@wordpress.org' ) ) {
-        $user = false;
-    }
 
     // If this is related to a slack user, fetch their details instead.
@@ -96 +91 @@
     // If the customer object has alternative emails listed, check to see if they have a profile.
     if ( ! $user && ! empty( $request->customer->emails ) ) {
-        foreach ( $request->customer->emails as $alt_email ) {
-            $user = get_user_by( 'email', $alt_email );
-            if ( $user ) {
-                break;
-            }
-        }
+        $user = get_user_from_emails( $request->customer->emails );
+    }
+
+    // Ignore @wordpress.org "users", unless it's literally the only match (The ?? $email fallback at the end).
+    if ( $user && str_ends_with( $user->user_email, '@wordpress.org' ) ) {
+        $user = false;
     }
 
@@ -185 +180 @@
     // Extract `To:`, `X-Orig-To:`, and fallback to all emails.
     $emails = [];
-    if ( preg_match( '!^(x-orig-to:|to:|Final-Recipient:(\s*rfc\d+;)?)\s*(?P<email>.+@.+)$!im', $text, $m ) ) {
-        $m['email'] = str_replace( [ '&lt;', '&gt;' ], '', $m['email'] );
-        $m['email'] = trim( $m['email'], '<> ' );
-
-        $emails = [ $m['email'] ];
-    } else {
+    if ( preg_match_all( '!^(x-orig-to:|to:|(Final|Original)-Recipient:(\s*rfc\d+;)?)\s*(?P<email>.+@.+)$!im', $text, $m ) ) {
+        $emails = $m['email'];
+    } elseif (
         // Ugly regex for emails, but it's good for mailer-daemon emails.
-        if ( preg_match_all( '![^\s;"]+@[^\s;&"]+\.[^\s;&"]+[a-z]!', $text, $m ) ) {
-            $emails = array_unique( array_diff( $m[0], [ $request->mailbox->email ] ) );
-        }
-    }
+        preg_match_all( '![^\s;"]+@[^\s;&"]+\.[^\s;&"]+[a-z]!', $text, $m )
+    ) {
+        $emails = $m[0];
+    }
+
+    // Clean them up.
+    foreach ( $emails as &$email ) {
+        $email = str_replace( [ '&lt;', '&gt;' ], '', $email );
+        $email = trim( $email, '<> ' );
+    }
+    $emails = array_unique( $emails );
+
+    // Remove any internal emails.
+    $emails = array_filter( $emails, function( $email ) use( $request ) {
+        return ! str_ends_with( $email, '@wordpress.org' ) &&
+            $email != $request->mailbox->email;
+    } );
 
     return $emails;