feat: password unlock screen on refresh instead of full logout
Build & Release / Docker / backend (push) Successful in 1m3s
Build & Release / Docker / frontend (push) Successful in 2m47s
Build & Release / Deploy to server (push) Has been cancelled
Build & Release / Create release (push) Has been cancelled
Build & Release / Desktop / Linux & Windows (push) Has been cancelled
Build & Release / Docker / backend (push) Successful in 1m3s
Build & Release / Docker / frontend (push) Successful in 2m47s
Build & Release / Deploy to server (push) Has been cancelled
Build & Release / Create release (push) Has been cancelled
Build & Release / Desktop / Linux & Windows (push) Has been cancelled
On page reload, tokens persist but crypto keys are lost (by design). Now shows a password-only unlock screen instead of the full login form, so users don't need to re-enter their email after a refresh. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
1 parent
b95cc7dd54
commit
515bef4fc9
3 files changed
+56
-27
No files matched your search
@@ -6,12 +6,13 @@ interface AuthState {
|
|||||||
refreshToken: string | null;
|
refreshToken: string | null;
|
||||||
userId: string | null;
|
userId: string | null;
|
||||||
username: string | null;
|
username: string | null;
|
||||||
|
email: string | null;
|
||||||
isAdmin: boolean;
|
isAdmin: boolean;
|
||||||
// Crypto keys — in-memory only, never persisted
|
// Crypto keys — in-memory only, never persisted
|
||||||
masterKey: CryptoKey | null;
|
masterKey: CryptoKey | null;
|
||||||
privateKey: CryptoKey | null; // X25519 private key (non-extractable CryptoKey)
|
privateKey: CryptoKey | null;
|
||||||
publicKey: string | null; // X25519 public key (base64)
|
publicKey: string | null;
|
||||||
setAuth: (data: { accessToken: string; refreshToken: string; userId: string; isAdmin?: boolean }) => void;
|
setAuth: (data: { accessToken: string; refreshToken: string; userId: string; isAdmin?: boolean; email?: string }) => void;
|
||||||
setTokens: (access: string, refresh: string) => void;
|
setTokens: (access: string, refresh: string) => void;
|
||||||
setMasterKey: (key: CryptoKey) => void;
|
setMasterKey: (key: CryptoKey) => void;
|
||||||
setKeyPair: (publicKeyB64: string, privateKey: CryptoKey) => void;
|
setKeyPair: (publicKeyB64: string, privateKey: CryptoKey) => void;
|
||||||
@@ -26,12 +27,13 @@ export const useAuthStore = create<AuthState>()(
|
|||||||
refreshToken: null,
|
refreshToken: null,
|
||||||
userId: null,
|
userId: null,
|
||||||
username: null,
|
username: null,
|
||||||
|
email: null,
|
||||||
isAdmin: false,
|
isAdmin: false,
|
||||||
masterKey: null,
|
masterKey: null,
|
||||||
privateKey: null as CryptoKey | null,
|
privateKey: null as CryptoKey | null,
|
||||||
publicKey: null,
|
publicKey: null,
|
||||||
setAuth: ({ accessToken, refreshToken, userId, isAdmin }) =>
|
setAuth: ({ accessToken, refreshToken, userId, isAdmin, email }) =>
|
||||||
set({ accessToken, refreshToken, userId, isAdmin: isAdmin ?? false }),
|
set({ accessToken, refreshToken, userId, isAdmin: isAdmin ?? false, ...(email ? { email } : {}) }),
|
||||||
setTokens: (accessToken, refreshToken) => set({ accessToken, refreshToken }),
|
setTokens: (accessToken, refreshToken) => set({ accessToken, refreshToken }),
|
||||||
setMasterKey: (masterKey) => set({ masterKey }),
|
setMasterKey: (masterKey) => set({ masterKey }),
|
||||||
setKeyPair: (publicKey, privateKey) => set({ publicKey, privateKey }),
|
setKeyPair: (publicKey, privateKey) => set({ publicKey, privateKey }),
|
||||||
@@ -41,6 +43,7 @@ export const useAuthStore = create<AuthState>()(
|
|||||||
accessToken: null,
|
accessToken: null,
|
||||||
refreshToken: null,
|
refreshToken: null,
|
||||||
userId: null,
|
userId: null,
|
||||||
|
email: null,
|
||||||
masterKey: null,
|
masterKey: null,
|
||||||
privateKey: null,
|
privateKey: null,
|
||||||
publicKey: null,
|
publicKey: null,
|
||||||
@@ -54,6 +57,7 @@ export const useAuthStore = create<AuthState>()(
|
|||||||
refreshToken: s.refreshToken,
|
refreshToken: s.refreshToken,
|
||||||
userId: s.userId,
|
userId: s.userId,
|
||||||
username: s.username,
|
username: s.username,
|
||||||
|
email: s.email,
|
||||||
isAdmin: s.isAdmin,
|
isAdmin: s.isAdmin,
|
||||||
}),
|
}),
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
import { useState, FormEvent } from 'react';
|
import { useState, FormEvent } from 'react';
|
||||||
import { Link, useNavigate } from 'react-router-dom';
|
import { Link, useNavigate, useLocation } from 'react-router-dom';
|
||||||
import { Shield } from 'lucide-react';
|
import { Shield, Lock } from 'lucide-react';
|
||||||
import api from '../lib/api';
|
import api from '../lib/api';
|
||||||
import { useAuthStore } from '../lib/store';
|
import { useAuthStore } from '../lib/store';
|
||||||
import { deriveKeyFromPassword, decryptMasterKey, decryptPrivateKeyToCryptoKey } from '../crypto';
|
import { deriveKeyFromPassword, decryptMasterKey, decryptPrivateKeyToCryptoKey } from '../crypto';
|
||||||
@@ -8,7 +8,12 @@ import { socket } from '../lib/ws';
|
|||||||
|
|
||||||
export default function LoginPage() {
|
export default function LoginPage() {
|
||||||
const navigate = useNavigate();
|
const navigate = useNavigate();
|
||||||
const { setAuth, setMasterKey, setKeyPair, setUsername } = useAuthStore();
|
const location = useLocation();
|
||||||
|
const { setAuth, setMasterKey, setKeyPair, setUsername, username, email: storedEmail } = useAuthStore();
|
||||||
|
|
||||||
|
// Reauth: tokens still valid but crypto keys lost on refresh
|
||||||
|
const isReauth = location.state?.reauth === true && !!storedEmail;
|
||||||
|
|
||||||
const [email, setEmail] = useState('');
|
const [email, setEmail] = useState('');
|
||||||
const [password, setPassword] = useState('');
|
const [password, setPassword] = useState('');
|
||||||
const [totpCode, setTotpCode] = useState('');
|
const [totpCode, setTotpCode] = useState('');
|
||||||
@@ -23,7 +28,7 @@ export default function LoginPage() {
|
|||||||
|
|
||||||
try {
|
try {
|
||||||
const res = await api.post('/auth/login', {
|
const res = await api.post('/auth/login', {
|
||||||
email,
|
email: isReauth ? storedEmail : email,
|
||||||
password,
|
password,
|
||||||
...(totpCode ? { totpCode } : {}),
|
...(totpCode ? { totpCode } : {}),
|
||||||
});
|
});
|
||||||
@@ -34,18 +39,15 @@ export default function LoginPage() {
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
const { accessToken, refreshToken, userId, username, kdfSalt, encryptedMasterKey, isAdmin } = res.data;
|
const { accessToken, refreshToken, userId, username: uname, kdfSalt, encryptedMasterKey, isAdmin } = res.data;
|
||||||
|
|
||||||
// Re-derive master key from password — never leaves the browser
|
|
||||||
const derived = await deriveKeyFromPassword(password, kdfSalt);
|
const derived = await deriveKeyFromPassword(password, kdfSalt);
|
||||||
const masterKey = await decryptMasterKey(encryptedMasterKey, derived);
|
const masterKey = await decryptMasterKey(encryptedMasterKey, derived);
|
||||||
|
|
||||||
// Set tokens first so the next API call has a Bearer token
|
setAuth({ accessToken, refreshToken, userId, isAdmin, email: isReauth ? storedEmail! : email });
|
||||||
setAuth({ accessToken, refreshToken, userId, isAdmin });
|
|
||||||
setMasterKey(masterKey);
|
setMasterKey(masterKey);
|
||||||
setUsername(username);
|
setUsername(uname);
|
||||||
|
|
||||||
// Fetch and decrypt the X25519 private key (stored encrypted on server for cross-device support)
|
|
||||||
const keyRes = await api.get('/auth/my-key');
|
const keyRes = await api.get('/auth/my-key');
|
||||||
const privateKey = await decryptPrivateKeyToCryptoKey(keyRes.data.encrypted_private_key, masterKey);
|
const privateKey = await decryptPrivateKeyToCryptoKey(keyRes.data.encrypted_private_key, masterKey);
|
||||||
setKeyPair(keyRes.data.public_key, privateKey);
|
setKeyPair(keyRes.data.public_key, privateKey);
|
||||||
@@ -63,17 +65,27 @@ export default function LoginPage() {
|
|||||||
<div className="min-h-screen flex items-center justify-center p-4">
|
<div className="min-h-screen flex items-center justify-center p-4">
|
||||||
<div className="w-full max-w-sm">
|
<div className="w-full max-w-sm">
|
||||||
<div className="flex flex-col items-center mb-8">
|
<div className="flex flex-col items-center mb-8">
|
||||||
<Shield className="w-12 h-12 text-accent mb-3" />
|
{isReauth ? (
|
||||||
|
<Lock className="w-12 h-12 text-accent mb-3" />
|
||||||
|
) : (
|
||||||
|
<Shield className="w-12 h-12 text-accent mb-3" />
|
||||||
|
)}
|
||||||
<h1 className="text-2xl font-bold">CloudLink</h1>
|
<h1 className="text-2xl font-bold">CloudLink</h1>
|
||||||
<p className="text-slate-400 text-sm mt-1">End-to-end encrypted</p>
|
<p className="text-slate-400 text-sm mt-1">
|
||||||
|
{isReauth ? `Welcome back, ${username}` : 'End-to-end encrypted'}
|
||||||
|
</p>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<form onSubmit={submit} className="bg-surface-1 rounded-xl p-6 space-y-4 border border-slate-700">
|
<form onSubmit={submit} className="bg-surface-1 rounded-xl p-6 space-y-4 border border-slate-700">
|
||||||
<h2 className="font-semibold text-lg">Sign in</h2>
|
<h2 className="font-semibold text-lg">
|
||||||
|
{isReauth ? 'Enter your password to unlock' : 'Sign in'}
|
||||||
|
</h2>
|
||||||
|
|
||||||
{error && <p className="text-red-400 text-sm">{error}</p>}
|
{error && <p className="text-red-400 text-sm">{error}</p>}
|
||||||
|
|
||||||
{!require2fa ? (
|
{isReauth ? (
|
||||||
|
<Field label="Password" type="password" value={password} onChange={setPassword} autoFocus />
|
||||||
|
) : !require2fa ? (
|
||||||
<>
|
<>
|
||||||
<Field label="Email" type="email" value={email} onChange={setEmail} />
|
<Field label="Email" type="email" value={email} onChange={setEmail} />
|
||||||
<Field label="Password" type="password" value={password} onChange={setPassword} />
|
<Field label="Password" type="password" value={password} onChange={setPassword} />
|
||||||
@@ -95,21 +107,33 @@ export default function LoginPage() {
|
|||||||
)}
|
)}
|
||||||
|
|
||||||
<button type="submit" disabled={loading} className="btn-primary w-full">
|
<button type="submit" disabled={loading} className="btn-primary w-full">
|
||||||
{loading ? 'Signing in…' : require2fa ? 'Verify' : 'Sign in'}
|
{loading ? (isReauth ? 'Unlocking…' : 'Signing in…') : isReauth ? 'Unlock' : require2fa ? 'Verify' : 'Sign in'}
|
||||||
</button>
|
</button>
|
||||||
|
|
||||||
|
{isReauth && (
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
onClick={() => { useAuthStore.getState().logout(); navigate('/login'); }}
|
||||||
|
className="w-full text-center text-sm text-slate-500 hover:text-slate-300"
|
||||||
|
>
|
||||||
|
Sign in as a different user
|
||||||
|
</button>
|
||||||
|
)}
|
||||||
</form>
|
</form>
|
||||||
|
|
||||||
<p className="text-center text-sm text-slate-400 mt-4">
|
{!isReauth && (
|
||||||
No account?{' '}
|
<p className="text-center text-sm text-slate-400 mt-4">
|
||||||
<Link to="/register" className="text-accent hover:underline">Register</Link>
|
No account?{' '}
|
||||||
</p>
|
<Link to="/register" className="text-accent hover:underline">Register</Link>
|
||||||
|
</p>
|
||||||
|
)}
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
function Field({ label, type, value, onChange }: {
|
function Field({ label, type, value, onChange, autoFocus }: {
|
||||||
label: string; type: string; value: string; onChange: (v: string) => void;
|
label: string; type: string; value: string; onChange: (v: string) => void; autoFocus?: boolean;
|
||||||
}) {
|
}) {
|
||||||
return (
|
return (
|
||||||
<div>
|
<div>
|
||||||
@@ -119,6 +143,7 @@ function Field({ label, type, value, onChange }: {
|
|||||||
type={type}
|
type={type}
|
||||||
value={value}
|
value={value}
|
||||||
onChange={(e) => onChange(e.target.value)}
|
onChange={(e) => onChange(e.target.value)}
|
||||||
|
autoFocus={autoFocus}
|
||||||
required
|
required
|
||||||
/>
|
/>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
@@ -52,7 +52,7 @@ export default function RegisterPage() {
|
|||||||
});
|
});
|
||||||
|
|
||||||
const { accessToken, refreshToken, userId } = res.data;
|
const { accessToken, refreshToken, userId } = res.data;
|
||||||
setAuth({ accessToken, refreshToken, userId });
|
setAuth({ accessToken, refreshToken, userId, email });
|
||||||
setMasterKey(masterKey);
|
setMasterKey(masterKey);
|
||||||
setKeyPair(publicKeyB64, privateKey);
|
setKeyPair(publicKeyB64, privateKey);
|
||||||
setUsername(username);
|
setUsername(username);
|
||||||
|
|||||||
Reference in new issue
Block a user