From 2f54b14a5ccf34660597b1602e1ec6bbe6c1b43d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Fri, 23 Dec 2022 20:02:36 +0100 Subject: [PATCH 01/24] Check for log when submitting form --- src/components/molecules/EmailForm.svelte | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/components/molecules/EmailForm.svelte b/src/components/molecules/EmailForm.svelte index 5a24acd..df3d5b3 100644 --- a/src/components/molecules/EmailForm.svelte +++ b/src/components/molecules/EmailForm.svelte @@ -28,9 +28,9 @@ return async ({ result, update }) => { formStatus = result.data - if (dev) { - console.log(result) - } + console.log(result) + // if (dev) { + // } // If successful if (result.data.success) { From d604d892e23b4fff6a11a16293c38408717ec86d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Fri, 23 Dec 2022 20:41:09 +0100 Subject: [PATCH 02/24] Disable csrf checkOrigin to fix newsletter form submission --- svelte.config.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/svelte.config.js b/svelte.config.js index 6bedeed..03e6d17 100644 --- a/svelte.config.js +++ b/svelte.config.js @@ -17,6 +17,9 @@ const config = { }), kit: { + csrf: { + checkOrigin: false + }, adapter: adapter({ edge: true }), @@ -27,7 +30,6 @@ const config = { $utils: 'src/utils', $style: 'src/style', }, - csrf: false, } } From ca56ac7edaf45844e18533b106ffab1b8361ab65 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Fri, 23 Dec 2022 20:46:23 +0100 Subject: [PATCH 03/24] Log stuff --- src/utils/forms/subscribe.ts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/utils/forms/subscribe.ts b/src/utils/forms/subscribe.ts index 5a13258..d981df4 100644 --- a/src/utils/forms/subscribe.ts +++ b/src/utils/forms/subscribe.ts @@ -12,6 +12,8 @@ export default async ({ request }) => { const formData = await request.formData() const email = formData.get('email') + console.log(email) + // Newsletter API request const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { method: 'POST', @@ -23,6 +25,7 @@ export default async ({ request }) => { }) const res = await req.json() + console.log(res) if (res) { if (res && res.status !== 'PENDING') { From c5791fa07ca8205962d3e92093cf192b63d6d668 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Fri, 23 Dec 2022 20:50:12 +0100 Subject: [PATCH 04/24] Name action?! --- src/routes/(site)/[country]/[location]/+page.server.ts | 2 +- src/routes/(site)/locations/+page.server.ts | 2 +- src/routes/(site)/photos/+page.server.ts | 2 +- src/routes/(site)/subscribe/+page.server.ts | 2 +- src/routes/+page.server.ts | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/routes/(site)/[country]/[location]/+page.server.ts b/src/routes/(site)/[country]/[location]/+page.server.ts index 2474079..ea607be 100644 --- a/src/routes/(site)/[country]/[location]/+page.server.ts +++ b/src/routes/(site)/[country]/[location]/+page.server.ts @@ -100,5 +100,5 @@ export const load: PageServerLoad = async ({ params, setHeaders }) => { */ export const actions: Actions = { // Form newsletter subscription - subscribe, + subscribe: subscribe, } \ No newline at end of file diff --git a/src/routes/(site)/locations/+page.server.ts b/src/routes/(site)/locations/+page.server.ts index f97107a..946a413 100644 --- a/src/routes/(site)/locations/+page.server.ts +++ b/src/routes/(site)/locations/+page.server.ts @@ -4,5 +4,5 @@ import subscribe from '$utils/forms/subscribe' export const actions: Actions = { // Form newsletter subscription - subscribe, + subscribe: subscribe, } \ No newline at end of file diff --git a/src/routes/(site)/photos/+page.server.ts b/src/routes/(site)/photos/+page.server.ts index 8ff1373..389f5a8 100644 --- a/src/routes/(site)/photos/+page.server.ts +++ b/src/routes/(site)/photos/+page.server.ts @@ -97,5 +97,5 @@ export const load: PageServerLoad = async ({ url, setHeaders }) => { */ export const actions: Actions = { // Form newsletter subscription - subscribe, + subscribe: subscribe, } \ No newline at end of file diff --git a/src/routes/(site)/subscribe/+page.server.ts b/src/routes/(site)/subscribe/+page.server.ts index 5a4f168..47a0bf2 100644 --- a/src/routes/(site)/subscribe/+page.server.ts +++ b/src/routes/(site)/subscribe/+page.server.ts @@ -48,5 +48,5 @@ export const load: PageServerLoad = async ({ setHeaders }) => { */ export const actions: Actions = { // Form newsletter subscription - subscribe, + subscribe: subscribe, } \ No newline at end of file diff --git a/src/routes/+page.server.ts b/src/routes/+page.server.ts index a3e904a..77ca49a 100644 --- a/src/routes/+page.server.ts +++ b/src/routes/+page.server.ts @@ -65,5 +65,5 @@ export const load: PageServerLoad = async ({ setHeaders }) => { */ export const actions: Actions = { // Form newsletter subscription - subscribe, + subscribe: subscribe, } \ No newline at end of file From 869debe87e813b032acc2ee2aa64f611fbd48a35 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 13:55:57 +0100 Subject: [PATCH 05/24] =?UTF-8?q?Switch=20to=20API=20route=20for=20newslet?= =?UTF-8?q?ter=20form=20submission=20=F0=9F=AB=A0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit not using the latest SvelteKit use:enhance thingy but… hey! --- src/components/molecules/EmailForm.svelte | 39 +++++++++----- .../[country]/[location]/+page.server.ts | 12 +---- src/routes/(site)/locations/+page.server.ts | 8 --- src/routes/(site)/photos/+page.server.ts | 12 +---- src/routes/(site)/subscribe/+page.server.ts | 12 +---- src/routes/+page.server.ts | 12 +---- src/routes/api/newsletter/+server.ts | 37 +++++++++++++ src/utils/forms/subscribe.ts | 52 ------------------- svelte.config.js | 3 -- 9 files changed, 66 insertions(+), 121 deletions(-) delete mode 100644 src/routes/(site)/locations/+page.server.ts create mode 100644 src/routes/api/newsletter/+server.ts delete mode 100644 src/utils/forms/subscribe.ts diff --git a/src/components/molecules/EmailForm.svelte b/src/components/molecules/EmailForm.svelte index df3d5b3..8dcfbee 100644 --- a/src/components/molecules/EmailForm.svelte +++ b/src/components/molecules/EmailForm.svelte @@ -3,8 +3,6 @@ diff --git a/src/routes/api/newsletter/+server.ts b/src/routes/api/newsletter/+server.ts index 5ac5600..4f7070b 100644 --- a/src/routes/api/newsletter/+server.ts +++ b/src/routes/api/newsletter/+server.ts @@ -3,9 +3,8 @@ import type { RequestHandler } from './$types' import { error } from '@sveltejs/kit' export const POST = (async ({ request, fetch }) => { - const data = await request.json() + const data: { email: string } = await request.json() const { email } = data - console.log('server:', data, email) // No email if (!email) { @@ -16,25 +15,28 @@ export const POST = (async ({ request, fetch }) => { throw error(400, { message: 'INVALID_EMAIL' }) } - // Newsletter API request - const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { - method: 'POST', - headers: { 'content-type': 'application/json' }, - body: JSON.stringify({ - api_key: NEWSLETTER_API_TOKEN, - email_address: email, - }) - }) - const res = await req.json() - console.log('server:', res) - - // Other error - if (res && res.status !== 'PENDING') { - throw error(400, { message: res.error.code }) - } - return new Response(JSON.stringify({ - success: true, - message: res.status, + email, })) + + // // Newsletter API request + // const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { + // method: 'POST', + // headers: { 'Content-Type': 'application/json' }, + // body: JSON.stringify({ + // api_key: NEWSLETTER_API_TOKEN, + // email_address: email, + // }) + // }) + // const res = await req.json() + + // // Other error + // if (res && res.status !== 'PENDING') { + // throw error(400, { message: res.error.code }) + // } + + // return new Response(JSON.stringify({ + // success: true, + // message: res.status, + // })) }) satisfies RequestHandler \ No newline at end of file From a2afd38fda8a0814af81763949cca78b535b6f19 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 15:25:54 +0100 Subject: [PATCH 12/24] Try returning json via SK --- src/components/molecules/EmailForm.svelte | 18 +++++----- src/routes/api/newsletter/+server.ts | 43 +++++++++++------------ 2 files changed, 30 insertions(+), 31 deletions(-) diff --git a/src/components/molecules/EmailForm.svelte b/src/components/molecules/EmailForm.svelte index 397da17..e3da1b9 100644 --- a/src/components/molecules/EmailForm.svelte +++ b/src/components/molecules/EmailForm.svelte @@ -45,16 +45,16 @@ }) const result: FormStatus = await req.json() formStatus = result - console.log(result) + console.log('SK api response:', result) - // // If successful - // if (formStatus.success) { - // sendEvent('newsletterSubscribe') - // } else { - // // Hide message for errors - // clearTimeout(formMessageTimeout) - // formMessageTimeout = requestAnimationFrame(() => setTimeout(() => formStatus = null, 4000)) - // } + // If successful + if (formStatus.success) { + sendEvent('newsletterSubscribe') + } else { + // Hide message for errors + clearTimeout(formMessageTimeout) + formMessageTimeout = requestAnimationFrame(() => setTimeout(() => formStatus = null, 4000)) + } } } diff --git a/src/routes/api/newsletter/+server.ts b/src/routes/api/newsletter/+server.ts index 4f7070b..ff01efa 100644 --- a/src/routes/api/newsletter/+server.ts +++ b/src/routes/api/newsletter/+server.ts @@ -1,6 +1,6 @@ import { NEWSLETTER_API_TOKEN, NEWSLETTER_LIST_ID } from '$env/static/private' import type { RequestHandler } from './$types' -import { error } from '@sveltejs/kit' +import { error, json } from '@sveltejs/kit' export const POST = (async ({ request, fetch }) => { const data: { email: string } = await request.json() @@ -15,28 +15,27 @@ export const POST = (async ({ request, fetch }) => { throw error(400, { message: 'INVALID_EMAIL' }) } - return new Response(JSON.stringify({ - email, - })) + // return json(email) - // // Newsletter API request - // const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { - // method: 'POST', - // headers: { 'Content-Type': 'application/json' }, - // body: JSON.stringify({ - // api_key: NEWSLETTER_API_TOKEN, - // email_address: email, - // }) - // }) - // const res = await req.json() + // Newsletter API request + const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { + method: 'POST', + headers: { 'Content-Type': 'application/json' }, + body: JSON.stringify({ + api_key: NEWSLETTER_API_TOKEN, + email_address: email, + }) + }) + const res = await req.json() + console.log('server API response:', res) - // // Other error - // if (res && res.status !== 'PENDING') { - // throw error(400, { message: res.error.code }) - // } + // Other error + if (res && res.status !== 'PENDING') { + throw error(400, { message: res.error.code }) + } - // return new Response(JSON.stringify({ - // success: true, - // message: res.status, - // })) + return json({ + success: true, + message: res.status, + }) }) satisfies RequestHandler \ No newline at end of file From 3e1b3be2ba6b5dc4696bb49be6c7dfe92cd50bc4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 15:34:51 +0100 Subject: [PATCH 13/24] Disable csrf? --- svelte.config.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/svelte.config.js b/svelte.config.js index 0c9a3f4..03e6d17 100644 --- a/svelte.config.js +++ b/svelte.config.js @@ -17,6 +17,9 @@ const config = { }), kit: { + csrf: { + checkOrigin: false + }, adapter: adapter({ edge: true }), From 4f97e8bc8e768a09f5e1fca7c4e780154ddb5bd4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 15:42:03 +0100 Subject: [PATCH 14/24] Just return a json response --- src/routes/api/newsletter/+server.ts | 38 ++++++++++++++-------------- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/src/routes/api/newsletter/+server.ts b/src/routes/api/newsletter/+server.ts index ff01efa..1fea22f 100644 --- a/src/routes/api/newsletter/+server.ts +++ b/src/routes/api/newsletter/+server.ts @@ -15,27 +15,27 @@ export const POST = (async ({ request, fetch }) => { throw error(400, { message: 'INVALID_EMAIL' }) } - // return json(email) + return json(email) // Newsletter API request - const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { - method: 'POST', - headers: { 'Content-Type': 'application/json' }, - body: JSON.stringify({ - api_key: NEWSLETTER_API_TOKEN, - email_address: email, - }) - }) - const res = await req.json() - console.log('server API response:', res) + // const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { + // method: 'POST', + // headers: { 'Content-Type': 'application/json' }, + // body: JSON.stringify({ + // api_key: NEWSLETTER_API_TOKEN, + // email_address: email, + // }) + // }) + // const res = await req.json() + // console.log('server API response:', res) - // Other error - if (res && res.status !== 'PENDING') { - throw error(400, { message: res.error.code }) - } + // // Other error + // if (res && res.status !== 'PENDING') { + // throw error(400, { message: res.error.code }) + // } - return json({ - success: true, - message: res.status, - }) + // return json({ + // success: true, + // message: res.status, + // }) }) satisfies RequestHandler \ No newline at end of file From fd652da00aa948fbff228b5fddca076b711b1733 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 15:53:58 +0100 Subject: [PATCH 15/24] Always return json response and not error? --- src/routes/api/newsletter/+server.ts | 44 ++++++++++++++-------------- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/src/routes/api/newsletter/+server.ts b/src/routes/api/newsletter/+server.ts index 1fea22f..b96b34a 100644 --- a/src/routes/api/newsletter/+server.ts +++ b/src/routes/api/newsletter/+server.ts @@ -1,6 +1,6 @@ import { NEWSLETTER_API_TOKEN, NEWSLETTER_LIST_ID } from '$env/static/private' import type { RequestHandler } from './$types' -import { error, json } from '@sveltejs/kit' +import { json } from '@sveltejs/kit' export const POST = (async ({ request, fetch }) => { const data: { email: string } = await request.json() @@ -8,34 +8,34 @@ export const POST = (async ({ request, fetch }) => { // No email if (!email) { - throw error(400, { message: 'NO_EMAIL' }) + return json({ message: 'NO_EMAIL' }) } // Invalid email if (!email.match(/^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*$/)) { - throw error(400, { message: 'INVALID_EMAIL' }) + return json({ message: 'INVALID_EMAIL' }) } - return json(email) + // return json(email) // Newsletter API request - // const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { - // method: 'POST', - // headers: { 'Content-Type': 'application/json' }, - // body: JSON.stringify({ - // api_key: NEWSLETTER_API_TOKEN, - // email_address: email, - // }) - // }) - // const res = await req.json() - // console.log('server API response:', res) + const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { + method: 'POST', + headers: { 'Content-Type': 'application/json' }, + body: JSON.stringify({ + api_key: NEWSLETTER_API_TOKEN, + email_address: email, + }) + }) + const res = await req.json() + console.log('server API response:', res) - // // Other error - // if (res && res.status !== 'PENDING') { - // throw error(400, { message: res.error.code }) - // } + // Other error + if (res && res.status !== 'PENDING') { + return json({ message: res.error.code }) + } - // return json({ - // success: true, - // message: res.status, - // }) + return json({ + success: true, + message: res.status, + }) }) satisfies RequestHandler \ No newline at end of file From 0e4b8edd43ec5f107d21f13fe4e1620623ecf413 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 15:58:05 +0100 Subject: [PATCH 16/24] Is it CORS? --- vercel.json | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/vercel.json b/vercel.json index 9f9f663..b0e447d 100644 --- a/vercel.json +++ b/vercel.json @@ -1,5 +1,14 @@ { "headers": [ + { + "source": "/api/(.*)", + "headers": [ + { "key": "Access-Control-Allow-Credentials", "value": "true" }, + { "key": "Access-Control-Allow-Origin", "value": "*" }, + { "key": "Access-Control-Allow-Methods", "value": "GET,OPTIONS,PATCH,DELETE,POST,PUT" }, + { "key": "Access-Control-Allow-Headers", "value": "X-CSRF-Token, X-Requested-With, Accept, Accept-Version, Content-Length, Content-MD5, Content-Type, Date, X-Api-Version" } + ] + }, { "source": "/(.*)", "headers": [ From d23eac3bc42306d858675e67bb569f61f12cd8b9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 16:01:05 +0100 Subject: [PATCH 17/24] CORS everywhere? --- vercel.json | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/vercel.json b/vercel.json index b0e447d..b45a653 100644 --- a/vercel.json +++ b/vercel.json @@ -1,14 +1,5 @@ { "headers": [ - { - "source": "/api/(.*)", - "headers": [ - { "key": "Access-Control-Allow-Credentials", "value": "true" }, - { "key": "Access-Control-Allow-Origin", "value": "*" }, - { "key": "Access-Control-Allow-Methods", "value": "GET,OPTIONS,PATCH,DELETE,POST,PUT" }, - { "key": "Access-Control-Allow-Headers", "value": "X-CSRF-Token, X-Requested-With, Accept, Accept-Version, Content-Length, Content-MD5, Content-Type, Date, X-Api-Version" } - ] - }, { "source": "/(.*)", "headers": [ @@ -17,7 +8,11 @@ { "key": "X-XSS-Protection", "value": "1; mode=block" }, { "key": "Strict-Transport-Security", "value": "max-age=31536000; includeSubDomains; preload" }, { "key": "Referrer-Policy", "value": "no-referrer-when-downgrade" }, - { "key": "Feature-Policy", "value": "geolocation 'self'" } + { "key": "Feature-Policy", "value": "geolocation 'self'" }, + { "key": "Access-Control-Allow-Credentials", "value": "true" }, + { "key": "Access-Control-Allow-Origin", "value": "*" }, + { "key": "Access-Control-Allow-Methods", "value": "GET,OPTIONS,PATCH,DELETE,POST,PUT" }, + { "key": "Access-Control-Allow-Headers", "value": "X-CSRF-Token, X-Requested-With, Accept, Accept-Version, Content-Length, Content-MD5, Content-Type, Date, X-Api-Version" } ] }, { From ab965574e78802cc2cc26a4f03ec60331d76ac3d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 16:01:16 +0100 Subject: [PATCH 18/24] Try catch the request --- src/routes/api/newsletter/+server.ts | 27 +++++++++++++++++---------- 1 file changed, 17 insertions(+), 10 deletions(-) diff --git a/src/routes/api/newsletter/+server.ts b/src/routes/api/newsletter/+server.ts index b96b34a..c4913e7 100644 --- a/src/routes/api/newsletter/+server.ts +++ b/src/routes/api/newsletter/+server.ts @@ -26,16 +26,23 @@ export const POST = (async ({ request, fetch }) => { email_address: email, }) }) - const res = await req.json() - console.log('server API response:', res) - // Other error - if (res && res.status !== 'PENDING') { - return json({ message: res.error.code }) + try { + if (req.ok) { + const res = await req.json() + console.log('server API response:', res) + + // Other error + if (res && res.status !== 'PENDING') { + return json({ message: res.error.code }) + } + + return json({ + success: true, + message: res.status, + }) + } + } catch (err) { + console.error(err) } - - return json({ - success: true, - message: res.status, - }) }) satisfies RequestHandler \ No newline at end of file From 5af12a39d70443d13428c1d624a6372f51a361d9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 16:05:16 +0100 Subject: [PATCH 19/24] Return Response --- src/routes/api/newsletter/+server.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/routes/api/newsletter/+server.ts b/src/routes/api/newsletter/+server.ts index c4913e7..1ce6da5 100644 --- a/src/routes/api/newsletter/+server.ts +++ b/src/routes/api/newsletter/+server.ts @@ -37,9 +37,11 @@ export const POST = (async ({ request, fetch }) => { return json({ message: res.error.code }) } - return json({ + return new Response(JSON.stringify({ success: true, message: res.status, + }), { + status: 200 }) } } catch (err) { From f0018bead374da7bc49689de55d32977d0ffcab8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 16:08:09 +0100 Subject: [PATCH 20/24] Return Responses --- src/routes/api/newsletter/+server.ts | 38 ++++++++++------------------ 1 file changed, 14 insertions(+), 24 deletions(-) diff --git a/src/routes/api/newsletter/+server.ts b/src/routes/api/newsletter/+server.ts index 1ce6da5..1297d73 100644 --- a/src/routes/api/newsletter/+server.ts +++ b/src/routes/api/newsletter/+server.ts @@ -1,6 +1,5 @@ import { NEWSLETTER_API_TOKEN, NEWSLETTER_LIST_ID } from '$env/static/private' import type { RequestHandler } from './$types' -import { json } from '@sveltejs/kit' export const POST = (async ({ request, fetch }) => { const data: { email: string } = await request.json() @@ -8,15 +7,13 @@ export const POST = (async ({ request, fetch }) => { // No email if (!email) { - return json({ message: 'NO_EMAIL' }) + return new Response(JSON.stringify({ message: 'NO_EMAIL' }), { status: 400 }) } // Invalid email if (!email.match(/^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*$/)) { - return json({ message: 'INVALID_EMAIL' }) + return new Response(JSON.stringify({ message: 'INVALID_EMAIL' }), { status: 400 }) } - // return json(email) - // Newsletter API request const req = await fetch(`https://emailoctopus.com/api/1.6/lists/${NEWSLETTER_LIST_ID}/contacts`, { method: 'POST', @@ -26,25 +23,18 @@ export const POST = (async ({ request, fetch }) => { email_address: email, }) }) + const res = await req.json() + console.log('server API response:', res) - try { - if (req.ok) { - const res = await req.json() - console.log('server API response:', res) - - // Other error - if (res && res.status !== 'PENDING') { - return json({ message: res.error.code }) - } - - return new Response(JSON.stringify({ - success: true, - message: res.status, - }), { - status: 200 - }) - } - } catch (err) { - console.error(err) + // Other error + if (res && res.status !== 'PENDING') { + return new Response(JSON.stringify({ message: res.error.code }), { status: 400 }) } + + return new Response(JSON.stringify({ + success: true, + message: res.status, + }), { + status: 200 + }) }) satisfies RequestHandler \ No newline at end of file From 0a5261b0df5628b3e9ec6cc93bfa11920d8fd817 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 16:31:20 +0100 Subject: [PATCH 21/24] Is it the Edge? --- svelte.config.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/svelte.config.js b/svelte.config.js index 03e6d17..a577b53 100644 --- a/svelte.config.js +++ b/svelte.config.js @@ -21,7 +21,7 @@ const config = { checkOrigin: false }, adapter: adapter({ - edge: true + // edge: true }), alias: { $components: 'src/components', From c5803ae046036d3e8ec601bbad5295dbbcf76ada Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 16:37:53 +0100 Subject: [PATCH 22/24] Put back CSRF --- svelte.config.js | 3 --- 1 file changed, 3 deletions(-) diff --git a/svelte.config.js b/svelte.config.js index a577b53..79e8b44 100644 --- a/svelte.config.js +++ b/svelte.config.js @@ -17,9 +17,6 @@ const config = { }), kit: { - csrf: { - checkOrigin: false - }, adapter: adapter({ // edge: true }), From ff243d19a47259ca63d172099ef50cd3830075cc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 16:49:47 +0100 Subject: [PATCH 23/24] Switch to Cloudflare Pages --- package.json | 1 + pnpm-lock.yaml | 30 ++++++++++++++++++++++++++++++ svelte.config.js | 2 +- 3 files changed, 32 insertions(+), 1 deletion(-) diff --git a/package.json b/package.json index 4c9501a..6fc3ee3 100644 --- a/package.json +++ b/package.json @@ -27,6 +27,7 @@ }, "devDependencies": { "@sveltejs/adapter-auto": "^1.0.0", + "@sveltejs/adapter-cloudflare": "^1.0.0", "@sveltejs/adapter-node": "^1.0.0", "@sveltejs/adapter-vercel": "^1.0.0", "@sveltejs/kit": "^1.0.1", diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index fb9bfcd..713031b 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -3,6 +3,7 @@ lockfileVersion: 5.4 specifiers: '@studio-freight/lenis': ^0.2.28 '@sveltejs/adapter-auto': ^1.0.0 + '@sveltejs/adapter-cloudflare': ^1.0.0 '@sveltejs/adapter-node': ^1.0.0 '@sveltejs/adapter-vercel': ^1.0.0 '@sveltejs/kit': ^1.0.1 @@ -47,6 +48,7 @@ dependencies: devDependencies: '@sveltejs/adapter-auto': 1.0.0_@sveltejs+kit@1.0.1 + '@sveltejs/adapter-cloudflare': 1.0.0_@sveltejs+kit@1.0.1 '@sveltejs/adapter-node': 1.0.0_@sveltejs+kit@1.0.1 '@sveltejs/adapter-vercel': 1.0.0_@sveltejs+kit@1.0.1 '@sveltejs/kit': 1.0.1_svelte@3.55.0+vite@4.0.3 @@ -72,6 +74,10 @@ devDependencies: packages: + /@cloudflare/workers-types/4.20221111.1: + resolution: {integrity: sha512-BNV2wN8V6Zduvo7UzxcdjBbLQ906D2KhS804PDufLgx/sanGJCHVJMOIaLvS/b61JKtot1U7P/l1fjrjZ7/E3A==} + dev: true + /@csstools/normalize.css/12.0.0: resolution: {integrity: sha512-M0qqxAcwCsIVfpFQSlGN5XjXWu8l5JDZN+fPt1LeW5SZexQTgnaEvgXAY+CeygRw0EeppWHi12JxESWiWrB0Sg==} dev: true @@ -674,6 +680,17 @@ packages: import-meta-resolve: 2.2.0 dev: true + /@sveltejs/adapter-cloudflare/1.0.0_@sveltejs+kit@1.0.1: + resolution: {integrity: sha512-uSFmMXI8vSM/f6duNgqfo6pkR1BBEcdzw1t1f56DsM2b/SQI0N7onY0yXpQA6LmDnWO/qdKp/TbRE0DuUt27LQ==} + peerDependencies: + '@sveltejs/kit': ^1.0.0 + dependencies: + '@cloudflare/workers-types': 4.20221111.1 + '@sveltejs/kit': 1.0.1_svelte@3.55.0+vite@4.0.3 + esbuild: 0.16.10 + worktop: 0.8.0-next.14 + dev: true + /@sveltejs/adapter-node/1.0.0_@sveltejs+kit@1.0.1: resolution: {integrity: sha512-Q8an8CXEt5XlFbyT1NBM4xELNZD8xPVZfKCcgorCfPkeBP5ftDgPaK12JIokXA5koYJ54AJcNY4ams9TZ7yGxA==} peerDependencies: @@ -3010,6 +3027,11 @@ packages: picomatch: 2.3.1 dev: true + /regexparam/2.0.1: + resolution: {integrity: sha512-zRgSaYemnNYxUv+/5SeoHI0eJIgTL/A2pUtXUPLHQxUldagouJ9p+K6IbIZ/JiQuCEv2E2B1O11SjVQy3aMCkw==} + engines: {node: '>=8'} + dev: true + /regexpp/3.2.0: resolution: {integrity: sha512-pq2bWo9mVD43nbts2wGv17XLiNLya+GklZ8kaDLV2Z08gDCsGpnKn9BFMepvWuHCbyVvY7J5o5+BVvoQbmlJLg==} engines: {node: '>=8'} @@ -3561,6 +3583,14 @@ packages: engines: {node: '>=0.10.0'} dev: true + /worktop/0.8.0-next.14: + resolution: {integrity: sha512-RZgqHu1w/JcUdWOE/BUEAzarrUUHh39eWkLdX8XpA6MfgLJF6X5Vl26CV7/wcm4O/UpZvHMGJUtB9eYTqDjc9g==} + engines: {node: '>=12'} + dependencies: + mrmime: 1.0.1 + regexparam: 2.0.1 + dev: true + /wrappy/1.0.2: resolution: {integrity: sha512-l4Sp/DRseor9wL6EvV2+TuQn63dMkPjZ/sp9XkghTEbV9KlPS1xUsZ3u7/IQO4wxtcFB4bgpQPRcR3QCvezPcQ==} dev: true diff --git a/svelte.config.js b/svelte.config.js index 79e8b44..0af0c43 100644 --- a/svelte.config.js +++ b/svelte.config.js @@ -1,5 +1,5 @@ import preprocess from 'svelte-preprocess' -import adapter from '@sveltejs/adapter-vercel' +import adapter from '@sveltejs/adapter-cloudflare' import { fileURLToPath } from 'url' import { dirname } from 'path' From 19e18c4b2a300d2934ef932bb4338f16ac9e1ba6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fe=CC=81lix=20Pe=CC=81ault?= Date: Sun, 25 Dec 2022 17:01:46 +0100 Subject: [PATCH 24/24] Switch to adapter-auto --- svelte.config.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/svelte.config.js b/svelte.config.js index 0af0c43..b197064 100644 --- a/svelte.config.js +++ b/svelte.config.js @@ -1,5 +1,5 @@ import preprocess from 'svelte-preprocess' -import adapter from '@sveltejs/adapter-cloudflare' +import adapter from '@sveltejs/adapter-auto' import { fileURLToPath } from 'url' import { dirname } from 'path' @@ -18,7 +18,7 @@ const config = { kit: { adapter: adapter({ - // edge: true + // edge: true, }), alias: { $components: 'src/components',