wrap-resource should set basic response headers about ring HOT 4 CLOSED

This is actually one of the things that is planned for Ring 1.2.0. The new not-modified middleware will handle "304 Not Modified" responses for handlers that return a response with a "Last Modified" or "Etag" header.

The next step is to change ring.util.response/resource-response function to return a more headers, such as Last-Modified and, as you pointed out, setting the Content-Length would be a good idea as well.

from ring.

Here's a patch:
djpowell@89033af

• Adds Content-Length and Last-Modified to all resource-responses.
• Fixes a problem when directories paths are detected as resources.

from ring.

Would it be possible to open up a pull request for that? It makes keeping track of patches a little easier.

from ring.

This has been merged in.

from ring.