[fix] adapter-netlify: handle undefined body

[benmccann]

Swap the condition so that when body is undefined, null, etc. we don't try to pass it into Buffer.from. Also, the old check wasn't quite correct in checking if the type was a string because you really should do typeof myVar === 'string' || myVar instanceof String

@Xenonym @Egnus do you want to check if this fix works for you?

[changeset-bot]

🦋 Changeset detected

Latest commit: c538704

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@sveltejs/adapter-netlify	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Xenonym]

@benmccann works for me! Thanks for fixing this 😓

However, I note that the Output of EndpointHandler should be JSONString | Uint8Array, which means that it should never be undefined / null in the first place right?

[Egnus]

Worked like a charm after enforcing these changes in the render function (via build)

[benmccann]

However, I note that the Output of EndpointHandler should be JSONString | Uint8Array, which means that it should never be undefined / null in the first place right?

body is optional though:

kit/packages/kit/types/endpoint.d.ts

Line 9 in 4101890

body?: Body;

[Xenonym]

body is optional though:

kit/packages/kit/types/endpoint.d.ts

Line 9 in 4101890

body?: Body;

@benmccann ah I missed that! Thanks for clarifying 😄