REST server should cache open documents #70
Assignees
Comments
ccxvii
added a commit
that referenced
this issue
Apr 23, 2024
Required us to remove the abbreviated express type imports, because we have to distinguish between the built-in fetch Response type and the express framework Response type.
ccxvii
added a commit
that referenced
this issue
Apr 25, 2024
Required us to remove the abbreviated express type imports, because we have to distinguish between the built-in fetch Response type and the express framework Response type.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Repeatedly doing fetch() on the same document from a third party server without caching is going to be slower than it needs to be. We should cache the most recently used documents and reuse the same array buffer that has already been fetched.
This caching can be handled in loadDocumentFromUrl, which can resolve to the cached document if it is in the cache.
Ideally we should use the fetch HTTP response headers to check for freshness as well, but that may be overkill for an example server.
The text was updated successfully, but these errors were encountered: