Ssmith has submitted this change and it was merged.

Change subject: Fix caching
......................................................................


Fix caching

Was storing at the wrong key.  Oops.

Change-Id: I21f408c6a5c24ee6a40fb3118486928fd8709712
---
M routes/data.js
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ssmith: Looks good to me, approved



diff --git a/routes/data.js b/routes/data.js
index 6766dc5..78ef89f 100644
--- a/routes/data.js
+++ b/routes/data.js
@@ -357,7 +357,7 @@
                }
                result = { results: dbResults, sqlQuery: substituteParams( 
sqlQuery, values), timestamp: new Date().getTime() };
                logger.debug( 'Storing results at cache key ' + cacheKey );
-               cache.put( req.url, result, config.cacheDuration );
+               cache.put( cacheKey, result, config.cacheDuration );
                res.json( result );
        });
 };

-- 
To view, visit https://gerrit.wikimedia.org/r/199769
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I21f408c6a5c24ee6a40fb3118486928fd8709712
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/dash
Gerrit-Branch: master
Gerrit-Owner: Ejegg <eeggles...@wikimedia.org>
Gerrit-Reviewer: Ssmith <ssm...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to