Kraken: Rate limiting for backfilling and warning instead of error on…

… API retries (#217) * Kraken: Rate limiting for backfilling and warning instead of error on API retries * Typo
DeviaVir · Jun 4, 2017 · dad44bc · dad44bc
1 parent c3125b1
commit dad44bc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/commands/backfill.js b/commands/backfill.js
@@ -167,7 +167,11 @@ module.exports = function container (get, set, clear) {
                       console.log('\ndownload complete!\n')
                       process.exit(0)
                     }
-                    setImmediate(getNext)
+                    if (exchange.backfillRateLimit) {
+                      setTimeout(getNext, exchange.backfillRateLimit)
+                    } else {
+                      setImmediate(getNext)
+                    }
                   })
                 })
               }
@@ -201,4 +205,4 @@ module.exports = function container (get, set, clear) {
         })
       })
   }
-}
+}
diff --git a/extensions/kraken/exchange.js b/extensions/kraken/exchange.js
@@ -38,7 +38,7 @@ module.exports = function container(get, set, clear) {
       var timeout = 2500
     }
 
-    console.error(('\nKraken API error - unable to call ' + method + ' (' + error + '), retrying in ' + timeout / 1000 + 's').red)
+    console.warn(('\nKraken API warning - unable to call ' + method + ' (' + error + '), retrying in ' + timeout / 1000 + 's').yellow)
     setTimeout(function () {
       exchange[method].apply(exchange, args)
     }, timeout)
@@ -50,6 +50,8 @@ module.exports = function container(get, set, clear) {
     name: 'kraken',
     historyScan: 'forward',
     makerFee: 0.16,
+    // The limit for the public API is not documented, 1750 ms between getTrades in backfilling seems to do the trick to omit warning messages.
+    backfillRateLimit: 1750,
 
     getProducts: function () {
       return require('./products.json')