Error thrown when returning endpoint Response without body

[tctree333]

Describe the bug
If an endpoint returns an object without the body key specified, an error is thrown. This only seems to happen with a 200 status code.

// src/routes/example.json.js

export function get() {
	// does not work
	return {};
	// also does not work
	return { status: 200 };
	// works
	return { body: {} };
	// also works
	return { status: 404 };
}

Stack trace below.

Expected behavior
No error thrown, or at least a more descriptive one.

Stacktraces

Stack trace

Cannot read property 'length' of undefined
TypeError: Cannot read property 'length' of undefined
    at hash (file:///[project_path]/node_modules/@sveltejs/kit/dist/ssr.js:1518:11)
    at render (file:///[project_path]/node_modules/@sveltejs/kit/dist/ssr.js:1483:26)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.handle (/[project_path]/src/hooks.ts:18:14)
    at async respond (file:///[project_path]/node_modules/@sveltejs/kit/dist/ssr.js:1450:10)
    at async Immediate.<anonymous> (file:///[project_path]/node_modules/@sveltejs/kit/dist/chunks/index.js:3304:23)

Information about your SvelteKit Installation:

Tested on @sveltejs/[email protected], [email protected], [email protected].

Severity
This isn't too severe since it's very easy to return an empty object as the body, but the error message doesn't really help.

Additional context
Seems like it's trying to hash the body for the ETag header, but body is undefined.

[Rich-Harris]

Arguably we should provide a more informative error, but this is an error, no? When would a 200 GET response not have a body?

[Mlocik97]

I agree with Rich,... btw no body and empty body is not same thing.

[tctree333]

A more descriptive error message would be great since I ran into it a couple times and it wasn't clear what was going on. I do agree that it doesn't entirely make sense for there to be no body but when I'm just testing if something works sometimes I forget to add body stuff, and then the error makes it seem like the thing I'm testing doesn't work!

[jerrythomas]

I ran into the same issue today. The error message was "Cannot read property 'length' of undefined".

My mistake was that I copied this snippet which did not include the body in the response and didn't think while copying.

https://kit.svelte.dev/docs#ssr-and-javascript-ssr
To set multiple cookies in a single set of response headers, you can return an array:

return {
	headers: {
		'set-cookie': [cookie1, cookie2]
	}
};

[traed]

Arguably we should provide a more informative error, but this is an error, no? When would a 200 GET response not have a body?

I get this on a 200 DELETE though and in that case a body doesn't make sense, right?