From 5e7323f389430beed5ffab915555c17416901351 Mon Sep 17 00:00:00 2001 From: Michael Jackson Date: Tue, 15 Jan 2019 07:59:53 -0800 Subject: [PATCH] Remove incrementCounter utility --- modules/middleware/fetchPackage.js | 13 ------------- modules/utils/incrementCounter.js | 13 ------------- 2 files changed, 26 deletions(-) delete mode 100644 modules/utils/incrementCounter.js diff --git a/modules/middleware/fetchPackage.js b/modules/middleware/fetchPackage.js index 2df1b56..edba38f 100644 --- a/modules/middleware/fetchPackage.js +++ b/modules/middleware/fetchPackage.js @@ -4,7 +4,6 @@ import addLeadingSlash from '../utils/addLeadingSlash'; import createPackageURL from '../utils/createPackageURL'; import createSearch from '../utils/createSearch'; import getNpmPackageInfo from '../utils/getNpmPackageInfo'; -// import incrementCounter from '../utils/incrementCounter'; function tagRedirect(req, res) { const version = req.packageInfo['dist-tags'][req.packageVersion]; @@ -59,14 +58,6 @@ function filenameRedirect(req, res) { ) { // Deprecated, see #63 filename = req.packageConfig[req.query.main]; - - // Count which packages are using this so we can warn them when we - // remove this functionality. - // incrementCounter( - // 'package-json-custom-main', - // req.packageSpec + '?main=' + req.query.main, - // 1 - // ); } else if ( req.packageConfig.unpkg && typeof req.packageConfig.unpkg === 'string' @@ -78,10 +69,6 @@ function filenameRedirect(req, res) { ) { // Deprecated, see #63 filename = req.packageConfig.browser; - - // Count which packages are using this so we can warn them when we - // remove this functionality. - // incrementCounter('package-json-browser-fallback', req.packageSpec, 1); } else { filename = req.packageConfig.main || '/index.js'; } diff --git a/modules/utils/incrementCounter.js b/modules/utils/incrementCounter.js deleted file mode 100644 index 3a1aac5..0000000 --- a/modules/utils/incrementCounter.js +++ /dev/null @@ -1,13 +0,0 @@ -import db from './data'; - -export default function incrementCounter(counter, key, by = 1) { - return new Promise((resolve, reject) => { - db.hincrby(counter, key, by, (error, value) => { - if (error) { - reject(error); - } else { - resolve(value); - } - }); - }); -}