Compare commits

..

2 Commits

Author SHA1 Message Date
f7411d3dc4 2.3.30
All checks were successful
Build & Deploy / 🔍 Prepare (push) Successful in 10s
Build & Deploy / 🧪 QA (push) Successful in 1m13s
Build & Deploy / 🏗️ Build (push) Successful in 2m31s
Build & Deploy / 🚀 Deploy (push) Successful in 16s
Build & Deploy / 🔔 Notify (push) Successful in 2s
Build & Deploy / 🧪 Post-Deploy Verification (push) Successful in 1m21s
2026-07-17 15:54:38 +02:00
7be2ec605c fix(contact): correct pino logger argument order to pass typescript compiler 2026-07-17 15:54:35 +02:00
2 changed files with 37 additions and 58 deletions

View File

@@ -32,24 +32,18 @@ export async function sendContactFormAction(formData: FormData) {
const productName = formData.get('productName') as string | null;
if (!name || !email || !message) {
logger.warn(
{
name: !!name,
email: !!email,
message: !!message,
},
'Missing required fields in contact form',
);
logger.warn('Missing required fields in contact form', {
name: !!name,
email: !!email,
message: !!message,
});
return { success: false, error: 'Missing required fields' };
}
logger.info(
{
type: productName ? 'product_quote' : 'contact',
email,
},
'Payload CMS saving skipped because it has been removed',
);
logger.info('Payload CMS saving skipped because it has been removed', {
type: productName ? 'product_quote' : 'contact',
email,
});
// 1.5. Simple Fail-Safe Backup to Disk
// To ensure leads are never lost if email fails or Gotify is down, we append them to a local JSON Lines file.
@@ -68,13 +62,13 @@ export async function sendContactFormAction(formData: FormData) {
message,
};
fs.appendFileSync(backupFile, JSON.stringify(leadData) + '\n');
logger.info({ backupFile }, 'Successfully saved lead to local backup file');
logger.info('Successfully saved lead to local backup file', { backupFile });
} catch (backupError) {
logger.error({ error: String(backupError) }, 'Failed to write to local leads backup');
logger.error('Failed to write to local leads backup', { error: String(backupError) });
}
// 2. Send Emails
logger.info({ email, productName }, 'Sending branded emails');
logger.info('Sending branded emails', { email, productName });
const notificationSubject = productName
? `Product Inquiry: ${productName}`
@@ -94,7 +88,7 @@ export async function sendContactFormAction(formData: FormData) {
);
if (!isTestSubmission) {
logger.info({ recipients: env.MAIL_RECIPIENTS }, 'Sending internal notification');
logger.info('Sending internal notification', { recipients: env.MAIL_RECIPIENTS });
const notificationResult = await sendEmail({
replyTo: email,
subject: notificationSubject,
@@ -102,21 +96,15 @@ export async function sendContactFormAction(formData: FormData) {
});
if (notificationResult.success) {
logger.info(
{
messageId: notificationResult.messageId,
},
'Notification email sent successfully',
);
logger.info('Notification email sent successfully', {
messageId: notificationResult.messageId,
});
} else {
logger.error(
{
error: notificationResult.error,
subject: notificationSubject,
recipients: env.MAIL_RECIPIENTS,
},
'Notification email DELIVERY FAILED',
);
logger.error('Notification email DELIVERY FAILED', {
error: notificationResult.error,
subject: notificationSubject,
recipients: env.MAIL_RECIPIENTS,
});
services.errors.captureException(
new Error(`Notification email failed: ${notificationResult.error}`),
{
@@ -127,7 +115,7 @@ export async function sendContactFormAction(formData: FormData) {
);
}
} else {
logger.info({ email }, 'Skipping notification email for test submission');
logger.info('Skipping notification email for test submission', { email });
}
// 2b. Send confirmation to Customer (branded as KLZ Cables)
@@ -140,7 +128,7 @@ export async function sendContactFormAction(formData: FormData) {
);
if (!isTestSubmission) {
logger.info({ to: email }, 'Sending customer confirmation');
logger.info('Sending customer confirmation', { to: email });
const confirmationResult = await sendEmail({
to: email,
subject: confirmationSubject,
@@ -148,28 +136,22 @@ export async function sendContactFormAction(formData: FormData) {
});
if (confirmationResult.success) {
logger.info(
{
messageId: confirmationResult.messageId,
},
'Confirmation email sent successfully',
);
logger.info('Confirmation email sent successfully', {
messageId: confirmationResult.messageId,
});
} else {
logger.error(
{
error: confirmationResult.error,
subject: confirmationSubject,
to: email,
},
'Confirmation email DELIVERY FAILED',
);
logger.error('Confirmation email DELIVERY FAILED', {
error: confirmationResult.error,
subject: confirmationSubject,
to: email,
});
services.errors.captureException(
new Error(`Confirmation email failed: ${confirmationResult.error}`),
{ action: 'sendContactFormAction_confirmation', email },
);
}
} else {
logger.info({ email }, 'Skipping confirmation email for test submission');
logger.info('Skipping confirmation email for test submission', { email });
}
// Notify via Gotify (Internal)
@@ -187,13 +169,10 @@ export async function sendContactFormAction(formData: FormData) {
return { success: true };
} catch (error) {
const errorMsg = error instanceof Error ? error.message : String(error);
logger.error(
{
error: errorMsg,
stack: error instanceof Error ? error.stack : undefined,
},
'Failed to send branded emails',
);
logger.error('Failed to send branded emails', {
error: errorMsg,
stack: error instanceof Error ? error.stack : undefined,
});
services.errors.captureException(error, { action: 'sendContactFormAction', email });

View File

@@ -116,7 +116,7 @@
"prepare": "husky",
"preinstall": "npx only-allow pnpm"
},
"version": "2.3.29",
"version": "2.3.30",
"pnpm": {
"onlyBuiltDependencies": [
"@parcel/watcher",