Question: isn't return null correct compared to empty string?

[adityapatadia]

https://github.com/fastify/fastify-etag/blob/master/index.js#L35

[adityapatadia]

https://github.com/fastify/fastify/blob/master/docs/Hooks.md#the-onsend-hook this link describes sending null.

[mcollina]

Good spot! Would you like to send a PR?

[kurtextrem]

Looks like this can be closed, looking at the closed PR?

[Uzlopak]

@kurtextrem

I tink this issue is still relevant, and the PR was closed due to more than 1 year inactivity.

[kurtextrem]

@Uzlopak I'm not entirely sure. The docs mention Content-length is not set when you return null, but @mcollina mentions we should set content-length manually to 0 if we want to return null.
So for me it's the same just different, which tells me we should keep it like it is?