Changes
This commit is contained in:
@@ -185,6 +185,16 @@ export const api = {
|
|||||||
}),
|
}),
|
||||||
|
|
||||||
organizations: () => request<OrganizationsResponse>('/users/me/organizations'),
|
organizations: () => request<OrganizationsResponse>('/users/me/organizations'),
|
||||||
|
|
||||||
|
changePassword: (currentPassword: string, newPassword: string, newPasswordConfirm: string) =>
|
||||||
|
request<{ message: string }>('/users/me/password', {
|
||||||
|
method: 'POST',
|
||||||
|
body: JSON.stringify({
|
||||||
|
current_password: currentPassword,
|
||||||
|
new_password: newPassword,
|
||||||
|
new_password_confirm: newPasswordConfirm,
|
||||||
|
}),
|
||||||
|
}),
|
||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
@@ -1,16 +1,28 @@
|
|||||||
import { useState } from "react";
|
import { useState } from "react";
|
||||||
import { Lock, Fingerprint, Smartphone, Shield, Plus, CheckCircle } from "lucide-react";
|
import { Lock, Fingerprint, Smartphone, Shield, Plus, CheckCircle, Loader2 } from "lucide-react";
|
||||||
import { Button } from "@/components/ui/button";
|
import { Button } from "@/components/ui/button";
|
||||||
import { Input } from "@/components/ui/input";
|
import { Input } from "@/components/ui/input";
|
||||||
import { Label } from "@/components/ui/label";
|
import { Label } from "@/components/ui/label";
|
||||||
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from "@/components/ui/card";
|
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from "@/components/ui/card";
|
||||||
import { Badge } from "@/components/ui/badge";
|
import { Badge } from "@/components/ui/badge";
|
||||||
import { AddPasskeyWizard } from "@/components/security/AddPasskeyWizard";
|
import { AddPasskeyWizard } from "@/components/security/AddPasskeyWizard";
|
||||||
|
import { PasswordStrengthMeter, isPasswordValid } from "@/components/auth/PasswordStrengthMeter";
|
||||||
|
import { api, ApiError } from "@/lib/api";
|
||||||
|
import { useToast } from "@/hooks/use-toast";
|
||||||
|
|
||||||
export default function SecurityPage() {
|
export default function SecurityPage() {
|
||||||
const [showPasswordForm, setShowPasswordForm] = useState(false);
|
const [showPasswordForm, setShowPasswordForm] = useState(false);
|
||||||
const [showAddPasskey, setShowAddPasskey] = useState(false);
|
const [showAddPasskey, setShowAddPasskey] = useState(false);
|
||||||
|
|
||||||
|
// Password form state
|
||||||
|
const [currentPassword, setCurrentPassword] = useState("");
|
||||||
|
const [newPassword, setNewPassword] = useState("");
|
||||||
|
const [confirmPassword, setConfirmPassword] = useState("");
|
||||||
|
const [isChangingPassword, setIsChangingPassword] = useState(false);
|
||||||
|
const [passwordError, setPasswordError] = useState<string | null>(null);
|
||||||
|
|
||||||
|
const { toast } = useToast();
|
||||||
|
|
||||||
// Mock security data
|
// Mock security data
|
||||||
const security = {
|
const security = {
|
||||||
passwordLastChanged: "3 months ago",
|
passwordLastChanged: "3 months ago",
|
||||||
@@ -27,6 +39,68 @@ export default function SecurityPage() {
|
|||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const resetPasswordForm = () => {
|
||||||
|
setCurrentPassword("");
|
||||||
|
setNewPassword("");
|
||||||
|
setConfirmPassword("");
|
||||||
|
setPasswordError(null);
|
||||||
|
};
|
||||||
|
|
||||||
|
const handlePasswordChange = async () => {
|
||||||
|
setPasswordError(null);
|
||||||
|
|
||||||
|
// Client-side validation
|
||||||
|
if (!currentPassword) {
|
||||||
|
setPasswordError("Current password is required");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!isPasswordValid(newPassword)) {
|
||||||
|
setPasswordError("New password does not meet strength requirements");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (newPassword !== confirmPassword) {
|
||||||
|
setPasswordError("Passwords do not match");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
setIsChangingPassword(true);
|
||||||
|
|
||||||
|
try {
|
||||||
|
await api.users.changePassword(currentPassword, newPassword, confirmPassword);
|
||||||
|
|
||||||
|
toast({
|
||||||
|
title: "Password updated",
|
||||||
|
description: "Your password has been changed successfully.",
|
||||||
|
});
|
||||||
|
|
||||||
|
resetPasswordForm();
|
||||||
|
setShowPasswordForm(false);
|
||||||
|
} catch (err) {
|
||||||
|
console.error("Password change failed:", err);
|
||||||
|
|
||||||
|
if (err instanceof ApiError) {
|
||||||
|
if (err.type === "INVALID_CREDENTIALS" || err.code === 401) {
|
||||||
|
setPasswordError("Current password is incorrect");
|
||||||
|
} else if (err.type === "VALIDATION_ERROR") {
|
||||||
|
setPasswordError(err.message);
|
||||||
|
} else {
|
||||||
|
setPasswordError(err.message);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
setPasswordError("An unexpected error occurred. Please try again.");
|
||||||
|
}
|
||||||
|
} finally {
|
||||||
|
setIsChangingPassword(false);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
const handleCancelPasswordChange = () => {
|
||||||
|
resetPasswordForm();
|
||||||
|
setShowPasswordForm(false);
|
||||||
|
};
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="page-container">
|
<div className="page-container">
|
||||||
<div className="page-header">
|
<div className="page-header">
|
||||||
@@ -74,24 +148,58 @@ export default function SecurityPage() {
|
|||||||
</CardHeader>
|
</CardHeader>
|
||||||
{showPasswordForm && (
|
{showPasswordForm && (
|
||||||
<CardContent className="space-y-4 border-t pt-4">
|
<CardContent className="space-y-4 border-t pt-4">
|
||||||
|
{passwordError && (
|
||||||
|
<div className="p-3 rounded-md bg-destructive/10 text-destructive text-sm">
|
||||||
|
{passwordError}
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
<div className="space-y-2">
|
<div className="space-y-2">
|
||||||
<Label htmlFor="currentPassword">Current password</Label>
|
<Label htmlFor="currentPassword">Current password</Label>
|
||||||
<Input id="currentPassword" type="password" />
|
<Input
|
||||||
|
id="currentPassword"
|
||||||
|
type="password"
|
||||||
|
value={currentPassword}
|
||||||
|
onChange={(e) => setCurrentPassword(e.target.value)}
|
||||||
|
disabled={isChangingPassword}
|
||||||
|
/>
|
||||||
</div>
|
</div>
|
||||||
<div className="space-y-2">
|
<div className="space-y-2">
|
||||||
<Label htmlFor="newPassword">New password</Label>
|
<Label htmlFor="newPassword">New password</Label>
|
||||||
<Input id="newPassword" type="password" />
|
<Input
|
||||||
<p className="text-xs text-muted-foreground">
|
id="newPassword"
|
||||||
Minimum {security.policyRequirements.minPasswordLength} characters required by organization
|
type="password"
|
||||||
</p>
|
value={newPassword}
|
||||||
|
onChange={(e) => setNewPassword(e.target.value)}
|
||||||
|
disabled={isChangingPassword}
|
||||||
|
/>
|
||||||
|
<PasswordStrengthMeter password={newPassword} />
|
||||||
</div>
|
</div>
|
||||||
<div className="space-y-2">
|
<div className="space-y-2">
|
||||||
<Label htmlFor="confirmPassword">Confirm new password</Label>
|
<Label htmlFor="confirmPassword">Confirm new password</Label>
|
||||||
<Input id="confirmPassword" type="password" />
|
<Input
|
||||||
|
id="confirmPassword"
|
||||||
|
type="password"
|
||||||
|
value={confirmPassword}
|
||||||
|
onChange={(e) => setConfirmPassword(e.target.value)}
|
||||||
|
disabled={isChangingPassword}
|
||||||
|
/>
|
||||||
|
{confirmPassword && newPassword !== confirmPassword && (
|
||||||
|
<p className="text-xs text-destructive">Passwords do not match</p>
|
||||||
|
)}
|
||||||
</div>
|
</div>
|
||||||
<div className="flex gap-2">
|
<div className="flex gap-2">
|
||||||
<Button>Update password</Button>
|
<Button
|
||||||
<Button variant="outline" onClick={() => setShowPasswordForm(false)}>
|
onClick={handlePasswordChange}
|
||||||
|
disabled={isChangingPassword || !isPasswordValid(newPassword) || newPassword !== confirmPassword}
|
||||||
|
>
|
||||||
|
{isChangingPassword && <Loader2 className="w-4 h-4 mr-2 animate-spin" />}
|
||||||
|
Update password
|
||||||
|
</Button>
|
||||||
|
<Button
|
||||||
|
variant="outline"
|
||||||
|
onClick={handleCancelPasswordChange}
|
||||||
|
disabled={isChangingPassword}
|
||||||
|
>
|
||||||
Cancel
|
Cancel
|
||||||
</Button>
|
</Button>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Reference in New Issue
Block a user