Add version tip to docs (#1790)

Bump version 4.6.1

docs/source/07-02-troubleshooting-tips.md

@@ -3,6 +3,10 @@ title: Troubleshooting Tips
 permalink: /troubleshooting-tips/
 ---
 
+## How do I find out what version I'm running?
+
+If you visit https://<YOUR TALK INSTANCE>/api/v1, it will return the version you're running and the hash for the latest commit.
+
 ## I've installed Talk but I can't see the comment stream appear on my articles
 
 * Make sure you've adding the correct domains to your Permitted Domains in Configure > Tech Settings

graph/loaders/users.js

@@ -84,10 +84,10 @@ const getUsersByQuery = async (
 
     if (value.length > 0) {
       // Lowercase the search term and escape any regex characters.
-      value = escapeRegExp(value).toLowerCase();
+      value = escapeRegExp(value);
 
-      // Compile the prefix search regex.
-      const $regex = new RegExp(`^${value}`);
+      const lowercasedRegex = new RegExp(`^${value.toLowerCase()}`);
+      const notLowercasedRegex = new RegExp(`^${value}`);
 
       // Merge in the regex params.
       query.merge({
@@ -95,7 +95,7 @@ const getUsersByQuery = async (
           // Search by a prefix match on the username.
           {
             lowercaseUsername: {
-              $regex,
+              $regex: lowercasedRegex,
             },
           },
 
@@ -104,7 +104,7 @@ const getUsersByQuery = async (
             profiles: {
               $elemMatch: {
                 id: {
-                  $regex,
+                  $regex: lowercasedRegex,
                 },
                 provider: 'local',
               },
@@ -114,7 +114,7 @@ const getUsersByQuery = async (
           // Search by the displayName metadata field.
           {
             'metadata.displayName': {
-              $regex,
+              $regex: notLowercasedRegex,
             },
           },
         ],

graph/mutators/comment.js

@@ -293,7 +293,16 @@ const setStatus = async (ctx, { id, status }) => {
  */
 const editComment = async (
   ctx,
-  { id, asset_id, edit: { body, metadata = {} } }
+  {
+    id,
+    asset_id,
+    edit: {
+      body,
+      metadata = {},
+      status: commentStatus,
+      actions: commentActions = [],
+    },
+  }
 ) => {
   const {
     connectors: {
@@ -303,7 +312,13 @@ const editComment = async (
 
   // Build up the new comment we're setting. We need to check this with
   // moderation now.
-  let comment = { id, asset_id, body };
+  let comment = {
+    id,
+    asset_id,
+    body,
+    status: commentStatus,
+    actions: commentActions,
+  };
 
   // Determine the new status of the comment.
   const { actions, status } = await Moderation.process(ctx, comment);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "talk",
-  "version": "4.6.0",
+  "version": "4.6.1",
   "description": "A better commenting experience from Mozilla, The New York Times, and the Washington Post. https://coralproject.net",
   "main": "app.js",
   "private": true,

plugins/talk-plugin-akismet/server/hooks.js

@@ -25,85 +25,104 @@ let enabled = true;
 //   }
 // });
 
+async function checkForSpam(ctx, { asset_id, body }) {
+  const req = ctx.parent.parent;
+  const loaders = ctx.loaders;
+
+  //If the key validation failed, then we can't run with the client.
+  if (!enabled) {
+    debug('not enabled, passing');
+    return;
+  }
+
+  let spam = false;
+  try {
+    const user_ip = get(req, 'ip', false);
+    if (!user_ip) {
+      debug('no ip on request');
+      return;
+    }
+
+    // Get some headers from the request.
+    const user_agent = req.get('User-Agent');
+    if (!user_agent || user_agent.length === 0) {
+      debug('no user agent on request');
+      return;
+    }
+
+    const referrer = req.get('Referrer');
+    if (!referrer || referrer.length === 0) {
+      debug('no referrer on request');
+      return;
+    }
+
+    // Get the Asset that the comment is being made against.
+    const asset = await loaders.Assets.getByID.load(asset_id);
+    if (!asset) {
+      debug('asset not found for new comment');
+      return;
+    }
+
+    // Send off the comment to Akismet to check to see what they say.
+    spam = await client.checkSpam({
+      user_ip,
+      user_agent,
+      referrer,
+      permalink: asset.url,
+      comment_type: 'comment',
+      comment_content: body,
+      is_test: false,
+    });
+
+    debug(`comment analyzed as ${spam ? 'being' : 'not being'} spam`);
+
+    return spam;
+  } catch (err) {
+    console.trace(err);
+    return;
+  }
+}
+
+function handlePositiveSpam(input) {
+  // Attach reason information for the flag being added.
+  input.status = 'SYSTEM_WITHHELD';
+  input.actions =
+    input.actions && input.actions.length >= 0 ? input.actions : [];
+  input.actions.push({
+    action_type: 'FLAG',
+    user_id: null,
+    group_id: 'SPAM_COMMENT',
+    metadata: {},
+  });
+}
+
 module.exports = {
   RootMutation: {
-    createComment: {
-      async pre(_, { input }, ctx) {
-        const req = ctx.parent.parent;
-        const loaders = ctx.loaders;
-
-        //If the key validation failed, then we can't run with the client.
-        if (!enabled) {
-          debug('not enabled, passing');
-          return;
+    editComment: {
+      pre: async (_, { asset_id, edit: { body }, edit }, ctx) => {
+        const spam = await checkForSpam(ctx, { asset_id, body });
+        if (spam) {
+          // Mark the comment as positive spam.
+          handlePositiveSpam(edit);
         }
-
-        let spam = false;
-        try {
-          const user_ip = get(req, 'ip', false);
-          if (!user_ip) {
-            debug('no ip on request');
-            return;
+      },
+    },
+    createComment: {
+      pre: async (_, { input }, ctx) => {
+        const spam = await checkForSpam(ctx, input);
+        if (spam) {
+          if (input.checkSpam) {
+            throw new ErrSpam();
           }
 
-          // Get some headers from the request.
-          const user_agent = req.get('User-Agent');
-          if (!user_agent || user_agent.length === 0) {
-            debug('no user agent on request');
-            return;
-          }
-
-          const referrer = req.get('Referrer');
-          if (!referrer || referrer.length === 0) {
-            debug('no referrer on request');
-            return;
-          }
-
-          // Get the Asset that the comment is being made against.
-          const asset = await loaders.Assets.getByID.load(input.asset_id);
-          if (!asset) {
-            debug('asset not found for new comment');
-            return;
-          }
-
-          // Send off the comment to Akismet to check to see what they say.
-          spam = await client.checkSpam({
-            user_ip,
-            user_agent,
-            referrer,
-            permalink: asset.url,
-            comment_type: 'comment',
-            comment_content: input.body,
-            is_test: false,
-          });
-
-          debug(`comment analyzed as ${spam ? 'being' : 'not being'} spam`);
-        } catch (err) {
-          console.trace(err);
-          return;
+          // Mark the comment as positive spam.
+          handlePositiveSpam(input);
         }
 
         // Attach scores to metadata.
         input.metadata = merge({}, input.metadata || {}, {
           akismet: spam,
         });
-
-        if (spam) {
-          if (input.checkSpam) {
-            throw new ErrSpam();
-          }
-
-          // Attach reason information for the flag being added.
-          input.status = 'SYSTEM_WITHHELD';
-          input.actions =
-            input.actions && input.actions.length >= 0 ? input.actions : [];
-          input.actions.push({
-            action_type: 'FLAG',
-            user_id: null,
-            group_id: 'SPAM_COMMENT',
-            metadata: {},
-          });
-        }
       },
     },
   },

plugins/talk-plugin-profile-data/client/hocs/index.js

@@ -3,6 +3,8 @@ import { gql } from 'react-apollo';
 import moment from 'moment';
 import update from 'immutability-helper';
 
+import { scheduledDeletionDelayHours } from '../../config';
+
 export const withRequestDownloadLink = withMutation(
   gql`
     mutation DownloadCommentHistory {
@@ -48,7 +50,7 @@ export const withRequestAccountDeletion = withMutation(
             });
 
             const scheduledDeletionDate = moment()
-              .add(24, 'hours')
+              .add(scheduledDeletionDelayHours, 'hours')
               .toDate();
 
             const data = update(prev, {

plugins/talk-plugin-toxic-comments/server/hooks.js

@@ -1,40 +1,59 @@
 const { getScores, isToxic } = require('./perspective');
 const { ErrToxic } = require('./errors');
 
+function handlePositiveToxic(input) {
+  input.status = 'SYSTEM_WITHHELD';
+  input.actions =
+    input.actions && input.actions.length >= 0 ? input.actions : [];
+  input.actions.push({
+    action_type: 'FLAG',
+    user_id: null,
+    group_id: 'TOXIC_COMMENT',
+    metadata: {},
+  });
+}
+
+async function getScore(body) {
+  // Try getting scores.
+  let scores;
+  try {
+    scores = await getScores(body);
+  } catch (err) {
+    // Warn and let mutation pass.
+    console.trace(err); // TODO: log/handle this differently?
+    return;
+  }
+
+  return scores;
+}
+
 module.exports = {
   RootMutation: {
+    editComment: {
+      pre: async (_, { edit: { body }, edit }) => {
+        const scores = await getScore(body);
+        if (isToxic(scores)) {
+          handlePositiveToxic(edit);
+        }
+      },
+    },
     createComment: {
       async pre(_, { input }, _context, _info) {
-        // Try getting scores.
-        let scores;
-        try {
-          scores = await getScores(input.body);
-        } catch (err) {
-          // Warn and let mutation pass.
-          console.trace(err); // TODO: log/handle this differently?
-          return;
-        }
-
-        // Attach scores to metadata.
-        input.metadata = Object.assign({}, input.metadata, {
-          perspective: scores,
-        });
+        const scores = await getScore(input.body);
 
         if (isToxic(scores)) {
           if (input.checkToxicity) {
             throw new ErrToxic();
           }
 
-          input.status = 'SYSTEM_WITHHELD';
-          input.actions =
-            input.actions && input.actions.length >= 0 ? input.actions : [];
-          input.actions.push({
-            action_type: 'FLAG',
-            user_id: null,
-            group_id: 'TOXIC_COMMENT',
-            metadata: {},
-          });
+          // Mark the comment as positive toxic.
+          handlePositiveToxic(input);
         }
+
+        // Attach scores to metadata.
+        input.metadata = Object.assign({}, input.metadata, {
+          perspective: scores,
+        });
       },
     },
   },