Skip to content

Commit 3599dad

Browse files
committed
fix send mail
1 parent 739740f commit 3599dad

File tree

4 files changed

+5
-10
lines changed

4 files changed

+5
-10
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ You can provide a site name to be used in the email subject and body.
131131

132132
### Docker
133133

134-
You can use the docker image to run the serverless form server.
134+
You can use the [docker image available on docker hub](https://hub.docker.com/repository/docker/lexoyo/serverless-forms) to run the serverless form server.
135135

136136
```
137137
docker run -e EMAIL_USER="username" \

lib/email.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@ function createTransporter(mailOptions) {
55
host: mailOptions.host,
66
port: Number(mailOptions.port),
77
secure: mailOptions.port === '465',
8-
auth: mailOptions.user && mailOptions.pass && {
8+
auth: mailOptions.user && mailOptions.pass ? {
99
user: mailOptions.user,
1010
pass: mailOptions.pass,
11-
},
11+
} : undefined,
1212
});
1313
}
1414

lib/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import 'dotenv/config';
22
import startServer from './server.js';
33

44
if (!process.env.EMAIL_HOST || !process.env.EMAIL_PORT || !process.env.EMAIL_USER || !process.env.EMAIL_PASS || !process.env.TO) {
5-
console.log('Missing required env vars. Please set EMAIL_HOST, EMAIL_PORT, EMAIL_USER, EMAIL_PASS and TO.');
5+
console.error('Missing required env vars. Please set EMAIL_HOST, EMAIL_PORT, EMAIL_USER, EMAIL_PASS and TO.');
66
process.exit(1);
77
}
88

lib/post.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,7 @@ export async function handleForm(fields, refererHeader, options) {
7777
return;
7878
}
7979
const referer = fields[options.siteField] || refererHeader || 'your website';
80-
console.log('Received form submission', { fields, referer, options });
8180
const html = generateEmailBody(fields, referer, options);
8281
const to = getRecipient(fields, options);
83-
await sendMail(html, to, `New form submission on ${referer}`, {
84-
host: options.mail.host,
85-
port: options.mail.port,
86-
user: options.mail.user,
87-
});
82+
await sendMail(html, to, `New form submission on ${referer}`, options.mail);
8883
}

0 commit comments

Comments
 (0)