v1.0.0 - fix redix

connects #31

Author: cvburgess

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v1.0.1 - 09-18-2019
+
+- Fix issue with Redis keys and values
+
 ## v1.0.0 - 09-17-2019
 
 - Switch to an extended knex method (.cache) vs the old wrapper fns

index.js

@@ -1,3 +1,4 @@
+const crypto = require("crypto");
 const { DataSource } = require("apollo-datasource");
 const { InMemoryLRUCache } = require("apollo-server-caching");
 const Knex = require("knex");
@@ -32,13 +33,16 @@ class SQLDataSource extends DataSource {
   }
 
   cacheQuery(ttl = 5, query) {
-    const cacheKey = query.toString();
+    const cacheKey = crypto
+      .createHash("sha1")
+      .update(query.toString())
+      .digest("base64");
 
     return this.cache.get(cacheKey).then(entry => {
-      if (entry) return Promise.resolve(entry);
+      if (entry) return Promise.resolve(JSON.parse(entry));
 
       return query.then(rows => {
-        if (rows) this.cache.set(cacheKey, rows, { ttl });
+        if (rows) this.cache.set(cacheKey, JSON.stringify(rows), { ttl });
 
         return Promise.resolve(rows);
       });

package.json

@@ -1,6 +1,6 @@
 {
   "name": "datasource-sql",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "SQL DataSource for Apollo GraphQL projects",
   "main": "index.js",
   "scripts": {