Prevent high values from being used as tiers

Paturages · Paturages · commit d7a6476f0e9d · 2021-03-31T18:07:13.000+02:00
diff --git a/src/drivers/google-drive.js b/src/drivers/google-drive.js
@@ -127,13 +127,11 @@ module.exports = async ({
       }
       // Store archives for processing
       // Archives might or might not be songs.
-      // The 200 MB threshold is just mostly here to not kill my bandwidth with multi-GB packs, which are therefore excluded.
-      // The good practice for such packs would be to rehost it (either by individual charters, or independently with separate
-      // folders/archives for each song)
-      // 2019-03-31: Increasing it to 500 MB to attempt taking modcharts/video backgrounds into account
+      // 2021-03-31: I have no idea what I'm doing with file limits anymore
+      const SIZE_LIMIT_MB = 69;
       if (
         ['rar', 'zip', '7z'].indexOf(item.fileExtension) >= 0 &&
-        item.size < 52428800 &&
+        item.size < SIZE_LIMIT_MB * (1024 * 1024) &&
         item.webContentLink
       ) {
         archives.push(item);
diff --git a/src/utils/db.js b/src/utils/db.js
@@ -1,4 +1,8 @@
 const Pg = require('./pg');
+// https://cdn.discordapp.com/attachments/616706064053829633/826847468737527878/unknown.png
+// https://cdn.discordapp.com/attachments/616706064053829633/826845717069955082/unknown.png
+// programming, I love that drug
+const MAX_DIFF = 32000;
 
 module.exports.getNbSongs = () =>
   Pg.q`SELECT COUNT(*) AS "nbSongs" FROM "Songs"`.then(
@@ -235,15 +239,15 @@ module.exports.upsertSongs = async (songs, noUpdateLastModified) => {
             ];
             return [
               ...wordableFields,
-              +diff_band >= 0 ? +diff_band >> 0 : null,
-              +diff_guitar >= 0 ? +diff_guitar >> 0 : null,
-              +diff_bass >= 0 ? +diff_bass >> 0 : null,
-              +diff_rhythm >= 0 ? +diff_rhythm >> 0 : null,
-              +diff_drums >= 0 ? +diff_drums >> 0 : null,
-              +diff_vocals >= 0 ? +diff_vocals >> 0 : null,
-              +diff_keys >= 0 ? +diff_keys >> 0 : null,
-              +diff_guitarghl >= 0 ? +diff_guitarghl >> 0 : null,
-              +diff_bassghl >= 0 ? +diff_bassghl >> 0 : null,
+              +diff_band >= 0 ? Math.min(+diff_band, MAX_DIFF) >> 0 : null,
+              +diff_guitar >= 0 ? Math.min(+diff_guitar, MAX_DIFF) >> 0 : null,
+              +diff_bass >= 0 ? Math.min(+diff_bass, MAX_DIFF) >> 0 : null,
+              +diff_rhythm >= 0 ? Math.min(+diff_rhythm, MAX_DIFF) >> 0 : null,
+              +diff_drums >= 0 ? Math.min(+diff_drums, MAX_DIFF) >> 0 : null,
+              +diff_vocals >= 0 ? Math.min(+diff_vocals, MAX_DIFF) >> 0 : null,
+              +diff_keys >= 0 ? Math.min(+diff_keys, MAX_DIFF) >> 0 : null,
+              +diff_guitarghl >= 0 ? Math.min(+diff_guitarghl, MAX_DIFF) >> 0 : null,
+              +diff_bassghl >= 0 ? Math.min(+diff_bassghl, MAX_DIFF) >> 0 : null,
               diffs.guitar,
               diffs.bass,
               diffs.rhythm,
