final fix: handle expired ZIM downloads with user-friendly error

Author: Best2Two

wp1-frontend/src/components/MyLists.vue

@@ -86,7 +86,7 @@
                 v-else-if="item.z_url && !hasDeletedZim(item)"
                 :class="{ 'outdated-zim': hasOutdatedZim(item) }"
               >
-                <a :href="item.z_url">Download ZIM</a>
+                <a href="#" @click.prevent="downloadZim(item)">Download ZIM</a>
                 <span
                   v-if="hasOutdatedZim(item)"
                   data-toggle="tooltip"
@@ -269,6 +269,17 @@ export default {
     localDate: function (date) {
       return localDate(date);
     },
+    downloadZim: async function (item) {
+      const response = await fetch(item.z_url);
+      if (response.status === 410) {
+        this.$router.push({
+          path: `/selections/${item.id}/zim`,
+          query: { expired: true },
+        });
+      } else {
+        window.location.href = item.z_url;
+      }
+    },
   },
   created: function () {
     this.getLists();

wp1-frontend/src/components/ZimFile.vue

@@ -83,7 +83,10 @@
             page and the My Selections page will automatically update with a URL
             to download your ZIM file once it is ready.
           </p>
-          <p v-if="isDeleted && status == 'FILE_READY'" class="errors">
+          <p
+            v-if="forceExpiredDisplay || (isDeleted && status == 'FILE_READY')"
+            class="errors"
+          >
             Your ZIM file has expired and needs to be re-created. ZIM file
             download links are only valid for 2 weeks. You can re-create your
             ZIM file with the button below.
@@ -108,7 +111,8 @@
       <div
         v-if="
           !activeSchedule &&
-          (status === 'NOT_REQUESTED' ||
+          (forceExpiredDisplay ||
+            status === 'NOT_REQUESTED' ||
             status === 'FAILED' ||
             (status != 'REQUESTED' && isDeleted))
         "
@@ -294,7 +298,10 @@
         </div>
       </div>
       <div
-        v-else-if="status === 'FILE_READY' || status === 'REQUESTED'"
+        v-else-if="
+          !forceExpiredDisplay &&
+          (status === 'FILE_READY' || status === 'REQUESTED')
+        "
         class="row"
       >
         <div class="col-lg-6 col-md-9 mx-4">
@@ -364,11 +371,15 @@ export default {
       repetitionPeriodInMonths: 1,
       numberOfRepetitions: 1,
       scheduleEmail: '',
+      forceExpiredDisplay: false,
     };
   },
   created: async function () {
     await this.getBuilder();
     await this.getStatus();
+    if (this.$route.query.expired) {
+      this.forceExpiredDisplay = true;
+    }
     await this.loadUserEmail();
     this.ready = true;
   },

wp1/web/builders.py

@@ -356,6 +356,10 @@ def latest_zim_file_for_builder(builder_id):
     if not url:
         flask.abort(404)
 
+    head = requests.head(url)
+    if head.status_code == 404:
+        flask.abort(410)
+
     return flask.redirect(url, code=302)