Merge pull request #13 from Dvorinka/fix/selfhosted-email-optional

fix: self-hosted auth without mail transport, password-manager-friendly login

apps/auth/src/lib/auth.ts

@@ -94,12 +94,18 @@ export const auth = betterAuth({
     sendOnSignUp: true,
     autoSignInAfterVerification: true,
     async sendVerificationEmail({ user, url }) {
+      // Verification is optional — a misconfigured transport must not fail
+      // sign-up or sign-in.
+      try {
         await sendTransactionalEmail({
           to: user.email,
           subject: "Verify your Primora email",
           text: `Verify your Primora email: ${url}`,
           html: `<p>Verify your Primora email.</p><p><a href="${url}">Verify email</a></p>`,
         });
+      } catch (error) {
+        console.warn(JSON.stringify({ level: "warn", msg: "verification_email_failed", error }));
+      }
     },
   },
   // Enabled unconditionally — better-auth only turns this on when NODE_ENV is

apps/auth/src/lib/mail.ts

@@ -51,7 +51,12 @@ export async function sendTransactionalEmail(input: {
   }
 
   if (!("smtp" in cfg) || !cfg.smtp) {
-    throw new Error("No mail transport configured for auth service.");
+    // Self-hosted instances may run without mail — auth still works, the
+    // email simply never goes out.
+    console.warn(
+      JSON.stringify({ level: "warn", msg: "mail_skipped_no_transport", to: input.to, subject: input.subject }),
+    );
+    return;
   }
 
   await nodemailer.createTransport(cfg.smtp).sendMail({

apps/frontend/src/App.tsx

@@ -736,7 +736,7 @@ export default function App() {
       try {
         if (mode() === "sign-up") {
           await authClient.signUp.email({ email: email(), password: password(), name: name() });
-          setAuthMessage("Account created. Check Mailpit (or your SMTP provider) for the verification email.");
+          setAuthMessage("Account created — signing you in.");
         } else {
           await authClient.signIn.email({ email: email(), password: password() });
         }

apps/frontend/src/components/Input.tsx

@@ -13,16 +13,17 @@ const sizeClasses = {
 };
 
 export function Input(props: InputProps) {
-  const [local, rest] = splitProps(props, ["label", "error", "size", "class"]);
+  const [local, rest] = splitProps(props, ["label", "error", "size", "class", "id"]);
 
   const size = () => local.size ?? "md";
 
   return (
     <div class="w-full">
       <Show when={local.label}>
-        <label class="label">{local.label}</label>
+        <label class="label" for={local.id}>{local.label}</label>
       </Show>
       <input
+        id={local.id}
         class={`input ${sizeClasses[size()]} ${local.error ? "border-error focus:border-error focus:ring-error/20" : ""} ${local.class ?? ""}`}
         {...rest}
       />

apps/frontend/src/pages/LoginPage.tsx

@@ -124,6 +124,8 @@ export function LoginPage(props: LoginPageProps) {
             <Show when={!isSignIn()}>
               <Input
                 label="Name"
+                id="name"
+                name="name"
                 type="text"
                 placeholder="Ada Lovelace"
                 value={props.name}
@@ -134,15 +136,19 @@ export function LoginPage(props: LoginPageProps) {
             </Show>
             <Input
               label="Email"
+              id="email"
+              name="email"
               type="email"
               placeholder="[email protected]"
               value={props.email}
               onInput={(e) => props.onEmailChange(e.currentTarget.value)}
-              autocomplete="email"
+              autocomplete="username"
               required
             />
             <Input
               label="Password"
+              id="password"
+              name="password"
               type="password"
               placeholder="••••••••"
               value={props.password}