max-age: 0 should be allowed?

[JeffBNimble]

Discussed in #3

^{Originally posted by JeffBNimble November 1, 2023}
Hello, I notice that if max age is 0, you don't write a cache-control header and per the HTTP caching specification here: https://datatracker.ietf.org/doc/html/rfc7234#section-5.2.2.8, the max-age is allowed to be any "non-negative integer". Can you explain the choice to not write a cache-control header on a response if max age is set to 0?

[landrade]

Hey @JeffBNimble,

First, thank you for opening this issue and bringing attention to this point. It's the first issue 🎉

After consulting the HTTP caching specification you mentioned, it's clear that the Cache-Control header should be included even when max-age is set to 0. This is indeed a bug that went unnoticed in the implementation.

Feel free to open a pull request with the fix if you want.