DX-2002: use eval instead of scriptload and evalsha (#137)

* fix: use eval instead of scriptload and evalsha

old version uses two commands while it is possible with a single command. also, I think script load doesn't increment ryw counter so it's possible to try to use evalsha on a script which hasn't actually loaded in a read replica

* fix: test

Author: CahidArda

src/cache.test.ts

@@ -38,7 +38,7 @@ test("ephemeral cache", async () => {
   }
 
   expect(passes).toBeLessThanOrEqual(10);
-  expect(metrics.evalsha).toBe(12);
+  expect(metrics.evalsha).toBe(11);
   expect(reasons).toContain("cacheBlock")
 
   await new Promise((r) => setTimeout(r, 5000));

src/hash.ts

@@ -22,17 +22,7 @@ export const safeEval = async (
     return await ctx.redis.evalsha(script.hash, keys, args)
   } catch (error) {
     if (`${error}`.includes("NOSCRIPT")) {
-      const hash = await ctx.redis.scriptLoad(script.script)
-
-      if (hash !== script.hash) {
-        console.warn(
-          "Upstash Ratelimit: Expected hash and the hash received from Redis"
-          + " are different. Ratelimit will work as usual but performance will"
-          + " be reduced."
-        );
-      }
-
-      return await ctx.redis.evalsha(hash, keys, args)
+      return await ctx.redis.eval(script.script, keys, args)
     }
     throw error;
   }