Changeset 10476

Timestamp:

12/01/2020 12:23:34 AM (4 years ago)

Author:

dd32

Message:

Trac: Mentions Handler: Bail early when the user triggering the content match on Trac is unknown / doesn't exist.

Mentions requires that the user triggering the match be known.

This does not fix the core bug - Email parsing is failing when a comment only changes summary fields and doesn't leave a comment, but thankfully those changes don't need to trigger the mentions handler.

File:

• sites/trunk/api.wordpress.org/public_html/dotorg/trac/mentions-handler.php (modified) (3 diffs)

sites/trunk/api.wordpress.org/public_html/dotorg/trac/mentions-handler.php

@@ -5 +5 @@
 define( 'MENTIONED',  2 );
 
+require dirname( dirname( __DIR__ ) ) . '/wp-init.php';
 require dirname( dirname( __DIR__ ) ) . '/includes/slack-config.php';
 
@@ -12 +13 @@
 
 $payload = json_decode( $_POST['payload'] );
-
-$_SERVER['HTTP_HOST'] = 'wordpress.org';
-require WPORGPATH . 'wp-load.php';
 
 require_once WP_PLUGIN_DIR . '/wporg-notifications.php';
@@ -38 +36 @@
 
 $user = get_user_by( 'login', $user_login );
+if ( ! $user ) {
+    exit;
+}
 
 function wporg_user_has_visited_trac( $user_login ) {