praeclarum
commented
2 years ago Sorry it's taken me so long to get to this.
This looks great!!! I'm so excited. I share your fears about the ever-filling disk. My hope is that because we are only using the temp directory, and because this server fails over at least once a week, the hard drive probably won't ever fail. Mark my words.
echoix
This restores the work that @praeclarum did in #144, which were undone when I added the download optimizations. The local caching should still provide a performance improvement. Caching of the zip archive table of contents, while the payload is very small, avoids 3 HTTP round trips required to discover the location of the TOC.
I'm slightly worried about what the behavior will be when the local disk eventually fills up. I think the behavior of the code will be fine, it should revert to pulling over HTTP. The full disk might lead to other issues though.
These bits can be previewed at https://pfluget-test.azurewebsites.net/ which is running in the smallest free Azure service plan. First hit might be slow, as the service will likely need to start up.