Gender-neutral rate limit error message, better addMessage rate limit

withspectrum · Jan 16, 2019 · cfb8af2 · cfb8af2
1 parent 3aee31f
commit cfb8af2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/api/types/Message.js b/api/types/Message.js
@@ -58,7 +58,7 @@ const Message = /* GraphQL */ `
 
   extend type Mutation {
     addMessage(message: MessageInput!): Message
-      @rateLimit(max: 20, window: "1m")
+      @rateLimit(max: 30, window: "1m")
     deleteMessage(id: ID!): Boolean
     editMessage(input: EditMessageInput!): Message
   }

diff --git a/api/utils/rate-limit-directive.js b/api/utils/rate-limit-directive.js
@@ -8,8 +8,8 @@ export default createRateLimitDirective({
   identifyContext: ctx => (ctx.user && ctx.user.id) || getClientIp(ctx.request),
   store: new RedisStore(createRedis()),
   formatError: ({ fieldName, fieldIdentity, max, window }) =>
-    `Slow down there cowboy! You've called '${fieldName}' ${max} times in the past ${ms(
+    `Slow down there partner! You've called '${fieldName}' ${max} times in the past ${ms(
       window,
       { long: true }
-    )}, which is too much.`,
+    )}. Relax for a bit and try again later.`,
 });