feat: add email status check to verify page
Security Enhancement: - Server-side validation of email status before allowing verification - Only allow verifying emails in VerificationPending state - Show "Action not possible" message for invalid states - Extract and validate share-id on server side - Return 404 for invalid share-ids or missing tenant emails Implementation: - Convert page.tsx to async server component - Fetch location and check tenantEmailStatus - Pass isPending prop to client component - Add bilingual "not-allowed" translations (same as unsubscribe page) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -7,9 +7,10 @@ import { CheckCircleIcon } from '@heroicons/react/24/outline';
|
|||||||
|
|
||||||
interface EmailVerifyPageProps {
|
interface EmailVerifyPageProps {
|
||||||
shareId: string;
|
shareId: string;
|
||||||
|
isPending: boolean;
|
||||||
}
|
}
|
||||||
|
|
||||||
export default function EmailVerifyPage({ shareId }: EmailVerifyPageProps) {
|
export default function EmailVerifyPage({ shareId, isPending }: EmailVerifyPageProps) {
|
||||||
const t = useTranslations('email-verify-page');
|
const t = useTranslations('email-verify-page');
|
||||||
const [isVerifying, setIsVerifying] = useState(false);
|
const [isVerifying, setIsVerifying] = useState(false);
|
||||||
const [isVerified, setIsVerified] = useState(false);
|
const [isVerified, setIsVerified] = useState(false);
|
||||||
@@ -61,6 +62,17 @@ export default function EmailVerifyPage({ shareId }: EmailVerifyPageProps) {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (!isPending) {
|
||||||
|
return (
|
||||||
|
<div className="card bg-base-100 shadow-xl max-w-2xl mx-auto mt-8">
|
||||||
|
<div className="card-body">
|
||||||
|
<h2 className="card-title text-warning">{t('not-allowed.title')}</h2>
|
||||||
|
<p>{t('not-allowed.message')}</p>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="card bg-base-100 shadow-xl max-w-2xl mx-auto mt-8">
|
<div className="card bg-base-100 shadow-xl max-w-2xl mx-auto mt-8">
|
||||||
<div className="card-body">
|
<div className="card-body">
|
||||||
|
|||||||
@@ -1,12 +1,44 @@
|
|||||||
import { Suspense } from 'react';
|
import { Suspense } from 'react';
|
||||||
import EmailVerifyPage from './EmailVerifyPage';
|
import EmailVerifyPage from './EmailVerifyPage';
|
||||||
import { Main } from '@/app/ui/Main';
|
import { Main } from '@/app/ui/Main';
|
||||||
|
import { getDbClient } from '@/app/lib/dbClient';
|
||||||
|
import { BillingLocation, EmailStatus } from '@/app/lib/db-types';
|
||||||
|
import { extractShareId, validateShareChecksum } from '@/app/lib/shareChecksum';
|
||||||
|
import { notFound } from 'next/navigation';
|
||||||
|
|
||||||
export default async function Page({ params: { id } }: { params: { id: string } }) {
|
export default async function Page({ params: { id } }: { params: { id: string } }) {
|
||||||
|
// Extract and validate share ID
|
||||||
|
const extracted = extractShareId(id);
|
||||||
|
if (!extracted) {
|
||||||
|
notFound();
|
||||||
|
}
|
||||||
|
|
||||||
|
const { locationId, checksum } = extracted;
|
||||||
|
|
||||||
|
// Validate checksum
|
||||||
|
if (!validateShareChecksum(locationId, checksum)) {
|
||||||
|
notFound();
|
||||||
|
}
|
||||||
|
|
||||||
|
// Fetch location to check email status
|
||||||
|
const dbClient = await getDbClient();
|
||||||
|
const location = await dbClient.collection<BillingLocation>("lokacije")
|
||||||
|
.findOne(
|
||||||
|
{ _id: locationId },
|
||||||
|
{ projection: { tenantEmail: 1, tenantEmailStatus: 1 } }
|
||||||
|
);
|
||||||
|
|
||||||
|
if (!location || !location.tenantEmail) {
|
||||||
|
notFound();
|
||||||
|
}
|
||||||
|
|
||||||
|
// Check if email is pending verification
|
||||||
|
const isPending = location.tenantEmailStatus === EmailStatus.VerificationPending;
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<Main>
|
<Main>
|
||||||
<Suspense fallback={<div className="text-center p-8">Loading...</div>}>
|
<Suspense fallback={<div className="text-center p-8">Loading...</div>}>
|
||||||
<EmailVerifyPage shareId={id} />
|
<EmailVerifyPage shareId={id} isPending={isPending} />
|
||||||
</Suspense>
|
</Suspense>
|
||||||
</Main>
|
</Main>
|
||||||
);
|
);
|
||||||
|
|||||||
@@ -455,6 +455,10 @@
|
|||||||
"error": {
|
"error": {
|
||||||
"title": "Verification Failed",
|
"title": "Verification Failed",
|
||||||
"unknown": "An error occurred during verification. Please try again or contact your landlord."
|
"unknown": "An error occurred during verification. Please try again or contact your landlord."
|
||||||
|
},
|
||||||
|
"not-allowed": {
|
||||||
|
"title": "Action not possible",
|
||||||
|
"message": "The selected action cannot be performed or the passed information is invalid."
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
"email-unsubscribe-page": {
|
"email-unsubscribe-page": {
|
||||||
|
|||||||
@@ -452,6 +452,10 @@
|
|||||||
"error": {
|
"error": {
|
||||||
"title": "Potvrda Nije Uspjela",
|
"title": "Potvrda Nije Uspjela",
|
||||||
"unknown": "Došlo je do greške prilikom potvrde. Molimo pokušajte ponovno ili kontaktirajte vašeg vlasnika nekretnine."
|
"unknown": "Došlo je do greške prilikom potvrde. Molimo pokušajte ponovno ili kontaktirajte vašeg vlasnika nekretnine."
|
||||||
|
},
|
||||||
|
"not-allowed": {
|
||||||
|
"title": "Akcija nije moguća",
|
||||||
|
"message": "Odabrana akcija nije moguća ili zadani podaci nisu ispravni."
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
"email-unsubscribe-page": {
|
"email-unsubscribe-page": {
|
||||||
|
|||||||
Reference in New Issue
Block a user