denoland / denoland/deploy_feedback

Support querying modification timestamps (`mtime`) for the Deploy filesystem

Open
#334 2 comments 4 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
79
Forks
5
PR merge metrics
No merged PRs in 30d

Description

I recently noticed that `std/http/file_server.ts` in Deploy doesn't add `Last-Modified` or `ETag` headers, which results in clients having to redownload the files when they might already have them cached. This seems to be because the Deploy filesystem doesn't support querying the modification timestamp ([`mtime`](https://deno.land/api@v1.30.3?s=Deno.FileInfo#prop_mtime)) of files.

Even returning the timestamp of the latest deployment would be a useful improvement in this regard, although of course it would be much better to return the timestamp of the first deployment that includes the commit that last modified the given file.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.