Use env variables instead of checking if isdev

packages/backend-core/src/security/auth.ts

@@ -1,20 +1,15 @@
 import { env } from ".."
 
-export const PASSWORD_MIN_LENGTH = 8
+export const PASSWORD_MIN_LENGTH = +(process.env.PASSWORD_MIN_LENGTH || 8)
-export const PASSWORD_MAX_LENGTH = 512
+export const PASSWORD_MAX_LENGTH = +(process.env.PASSWORD_MAX_LENGTH || 512)
 
 export function validatePassword(
   password: string
 ): { valid: true } | { valid: false; error: string } {
-  if (env.isDev() && !env.isTest() && password) {
-    // We accept any password while on development
-    return { valid: true }
-  }
-
   if (!password || password.length < PASSWORD_MIN_LENGTH) {
     return {
       valid: false,
-      error: "Password invalid. Minimum eight characters.",
+      error: `Password invalid. Minimum ${PASSWORD_MIN_LENGTH} characters.`,
     }
   }
 

packages/backend-core/src/security/tests/auth.spec.ts

@@ -14,21 +14,18 @@ describe("auth", () => {
     ])("%s returns unsuccessful", (_, password) => {
       expect(validatePassword(password as string)).toEqual({
         valid: false,
-        error: "Password invalid. Minimum eight characters.",
+        error: "Password invalid. Minimum 8 characters.",
       })
     })
 
     it.each([
       generator.word({ length: PASSWORD_MAX_LENGTH }),
       generator.paragraph().substring(0, PASSWORD_MAX_LENGTH),
-    ])(
+    ])(`can use passwords up to 512 characters in length`, password => {
-      `can use passwords up to ${PASSWORD_MAX_LENGTH} characters in length`,
+      expect(validatePassword(password)).toEqual({
-      password => {
+        valid: true,
-        expect(validatePassword(password)).toEqual({
+      })
-          valid: true,
+    })
-        })
-      }
-    )
 
     it.each([
       generator.word({ length: PASSWORD_MAX_LENGTH + 1 }),

packages/server/scripts/dev/manage.js

@@ -48,6 +48,7 @@ async function init() {
     HTTP_MIGRATIONS: "0",
     HTTP_LOGGING: "0",
     VERSION: "0.0.0+local",
+    PASSWORD_MIN_LENGTH: "1",
   }
 
   config = { ...config, ...existingConfig }

packages/worker/scripts/dev/manage.js

@@ -30,6 +30,7 @@ async function init() {
     ENABLE_EMAIL_TEST_MODE: "1",
     HTTP_LOGGING: "0",
     VERSION: "0.0.0+local",
+    PASSWORD_MIN_LENGTH: "1",
   }
 
   config = { ...config, ...existingConfig }