fix: more flexible user matching

Author: coltenkrauter

dist/index.js

@@ -81717,23 +81717,69 @@ var SlackAppUrl;
 ;// CONCATENATED MODULE: ./src/utils/slack/user-matchers.ts
 
 
+/**
+ * Redacts sensitive information for safe logging
+ * Replaces characters with * but keeps first and last character
+ * For emails, obfuscates both username and domain parts
+ */
+function redact(text) {
+    if (!text) {
+        return '';
+    }
+    // Handle email addresses specially
+    if (text.includes('@')) {
+        const [localPart, domainPart] = text.split('@');
+        const redactedLocal = redact(localPart);
+        // Obfuscate domain part too
+        const domainParts = domainPart.split('.');
+        const redactedDomain = domainParts
+            .map((part) => {
+            if (part.length <= 2) {
+                return part;
+            }
+            return `${part[0]}${'*'.repeat(part.length - 2)}${part[part.length - 1]}`;
+        })
+            .join('.');
+        return `${redactedLocal}@${redactedDomain}`;
+    }
+    // Handle regular text
+    if (text.length <= 2) {
+        return text;
+    }
+    return `${text[0]}${'*'.repeat(text.length - 2)}${text[text.length - 1]}`;
+}
 function customMappingMatcher(githubUsername, slackUsername) {
     return {
-        check: (user) => user.name?.toLowerCase() === slackUsername.toLowerCase() ||
-            user.profile?.display_name?.toLowerCase() === slackUsername.toLowerCase() ||
-            user.profile?.real_name?.toLowerCase() === slackUsername.toLowerCase(),
+        check: (user) => {
+            const name = user.name?.toLowerCase() ?? '';
+            const displayName = user.profile?.display_name?.toLowerCase() ?? '';
+            const realName = user.profile?.real_name?.toLowerCase() ?? '';
+            const slackNameLower = slackUsername.toLowerCase();
+            return (name.includes(slackNameLower) ||
+                slackNameLower.includes(name) ||
+                displayName.includes(slackNameLower) ||
+                slackNameLower.includes(displayName) ||
+                realName.includes(slackNameLower) ||
+                slackNameLower.includes(realName));
+        },
         description: 'custom user mapping',
         log: (user) => {
             (0,core.debug)(`Match found by custom mapping: GitHub username [${githubUsername}] to Slack username [${slackUsername}] for user [${user.id}]`);
+            (0,core.debug)(`Redacted debug info: GitHub username [${redact(githubUsername)}] to Slack username [${redact(slackUsername)}] matched with user name [${redact(user.name ?? '')}], display_name [${redact(user.profile?.display_name ?? '')}], real_name [${redact(user.profile?.real_name ?? '')}]`);
         },
     };
 }
 function displayNameMatcher(username) {
     return {
-        check: (user) => user.profile?.display_name?.toLowerCase() === username.toLowerCase(),
+        check: (user) => {
+            const displayName = user.profile?.display_name?.toLowerCase() ?? '';
+            const usernameLower = username.toLowerCase();
+            return displayName.includes(usernameLower) || usernameLower.includes(displayName);
+        },
         description: 'user.profile.display_name fields',
         log: (user) => {
             (0,core.debug)(`Match found by username [${username}] matching Slack displayName [${user.profile?.display_name}]`);
+            (0,core.debug)(`Redacted debug info: username [${redact(username)}] matched with display_name [${redact(user.profile?.display_name ?? '')}]`);
         },
     };
 }
@@ -81745,6 +81791,7 @@ function emailContainsMatcher(username) {
         description: 'user.profile.email contains check',
         log: (user) => {
             (0,core.debug)(`Match found by username [${username}] contained in Slack email [${user.profile?.email}]`);
+            (0,core.debug)(`Redacted debug info: username [${redact(username)}] matched with email [${redact(user.profile?.email ?? '')}]`);
         },
     };
 }
@@ -81754,15 +81801,21 @@ function emailMatcher(email) {
         description: 'user.profile.email fields',
         log: (user) => {
             (0,core.debug)(`Match found by email [${email}] with Slack email [${user.profile?.email}]`);
+            (0,core.debug)(`Redacted debug info: email [${redact(email)}] matched with Slack email [${redact(user.profile?.email ?? '')}]`);
         },
     };
 }
 function realNameMatcher(username) {
     return {
-        check: (user) => user.profile?.real_name?.toLowerCase() === username.toLowerCase(),
+        check: (user) => {
+            const realName = user.profile?.real_name?.toLowerCase() ?? '';
+            const usernameLower = username.toLowerCase();
+            return realName.includes(usernameLower) || usernameLower.includes(realName);
+        },
         description: 'user.profile.real_name fields',
         log: (user) => {
             (0,core.debug)(`Match found by username [${username}] matching Slack realName [${user.profile?.real_name}]`);
+            (0,core.debug)(`Redacted debug info: username [${redact(username)}] matched with real_name [${redact(user.profile?.real_name ?? '')}]`);
         },
     };
 }
@@ -81772,6 +81825,7 @@ function userIdMatcher(userId) {
         description: 'user.id fields',
         log: (user) => {
             (0,core.debug)(`Match found by userId [${userId}] with Slack userId [${user.id}]`);
+            (0,core.debug)(`Redacted debug info: userId matched with Slack userId [${redact(user.id ?? '')}]`);
         },
     };
 }
@@ -81782,6 +81836,7 @@ const createUserMatchers = ({ email, userId, userMappings = [], username }) => {
         const matchingMappings = userMappings.filter((mapping) => mapping.githubUsername === username);
         if (matchingMappings.length > 0) {
             (0,core.debug)(`Found [${matchingMappings.length}] custom mappings for GitHub username [${username}]`);
+            (0,core.debug)(`Redacted debug info: Found [${matchingMappings.length}] custom mappings for GitHub username [${redact(username)}]`);
             // Add a matcher for each mapping
             matchingMappings.forEach((mapping) => {
                 matchers.push(customMappingMatcher(username, mapping.slackUsername));
@@ -81804,25 +81859,35 @@ const createUserMatchers = ({ email, userId, userMappings = [], username }) => {
 };
 const logFailedMatches = ({ email, userId, userMappings = [], username }, usersCount) => {
     console.log(`No user match found for [${username}] after checking against [${usersCount}] Slack ${(0,src.plural)('user', usersCount)}`);
+    // Redacted version
+    console.log(`Redacted debug info: No user match found for [${redact(username ?? '')}] after checking against [${usersCount}] Slack ${(0,src.plural)('user', usersCount)}`);
     // Log mapping failures
     if (username && userMappings.length > 0) {
         const matchingMappings = userMappings.filter((mapping) => mapping.githubUsername === username);
         if (matchingMappings.length > 0) {
             (0,core.debug)(`WARNING: Custom mappings for GitHub username [${username}] were defined but no matching Slack users were found:`);
+            (0,core.debug)(`Redacted debug info: WARNING: Custom mappings for GitHub username [${redact(username)}] were defined but no matching Slack users were found:`);
             // Show each mapping that failed
             matchingMappings.forEach((mapping) => {
                 (0,core.debug)(`  - Mapped to Slack username [${mapping.slackUsername}] but no Slack user with this name/display_name/real_name was found`);
+                (0,core.debug)(`  - Redacted debug info: Mapped to Slack username [${redact(mapping.slackUsername)}] but no Slack user with this name/display_name/real_name was found`);
             });
             (0,core.debug)(`Attempted to fall back to standard matching methods`);
         }
     }
     // Log standard matchers that were tried
-    if (userId)
+    if (userId) {
         (0,core.debug)(`Tried to match userId [${userId}] against Slack user.id fields`);
-    if (email)
+        (0,core.debug)(`Redacted debug info: Tried to match userId [${redact(userId)}] against Slack user.id fields`);
+    }
+    if (email) {
         (0,core.debug)(`Tried to match email [${email}] against Slack user.profile.email fields`);
-    if (username)
+        (0,core.debug)(`Redacted debug info: Tried to match email [${redact(email)}] against Slack user.profile.email fields`);
+    }
+    if (username) {
         (0,core.debug)(`Tried to match username [${username}] against Slack user.profile.email (contains), display_name and real_name fields`);
+        (0,core.debug)(`Redacted debug info: Tried to match username [${redact(username)}] against Slack user.profile.email (contains), display_name and real_name fields`);
+    }
     (0,core.debug)(`Since no Slack user match found, unable to @mention user or use their profile image`);
 };
 
@@ -90631,7 +90696,7 @@ module.exports = {"version":"3.17.0"};
 /***/ 8330:
 /***/ ((module) => {
 
-module.exports = /*#__PURE__*/JSON.parse('{"UU":"@krauters/github-notifier","rE":"1.3.2","TB":"https://buymeacoffee.com/coltenkrauter"}');
+module.exports = /*#__PURE__*/JSON.parse('{"UU":"@krauters/github-notifier","rE":"1.3.3","TB":"https://buymeacoffee.com/coltenkrauter"}');
 
 /***/ })
 

dist/index.js.map

@@ -1,7 +1,7 @@
 {
   "name": "@krauters/github-notifier",
   "description": "GitHub Notifier by Krauters – Post Open Pull Requests to Slack",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "author": "Colten Krauter <coltenkrauter>",
   "type": "module",
   "homepage": "https://buymeacoffee.com/coltenkrauter",