From ea54fe5a58e5bf524591201447a5e0b8ac5a6ef7 Mon Sep 17 00:00:00 2001 From: CarlosFaria94 Date: Thu, 6 Apr 2017 12:46:54 +0100 Subject: [PATCH] Replace ServerWithAuth by Server in getFiatRate To get a fiat rate there is no need to pass by the authentication mechanisms. Also, previously the BWS was waiting for a 'source' in query, but the BWC it is sending a 'provider' query, not 'source', turns out that it is impossible to the client to query a specific provider, because fiatRateService do not recognize the variable 'source'. --- lib/expressapp.js | 24 ++++++++++++++---------- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/lib/expressapp.js b/lib/expressapp.js index 966ac4d..aded96e 100644 --- a/lib/expressapp.js +++ b/lib/expressapp.js @@ -660,16 +660,20 @@ ExpressApp.prototype.start = function(opts, cb) { }); router.get('/v1/fiatrates/:code/', function(req, res) { - getServerWithAuth(req, res, function(server) { - var opts = { - code: req.params['code'], - source: req.query.source, - ts: +req.query.ts, - }; - server.getFiatRate(opts, function(err, rates) { - if (err) return returnError(err, res, req); - res.json(rates); - }); + var server; + var opts = { + code: req.params['code'], + provider: req.query.provider, + ts: +req.query.ts, + }; + try { + server = getServer(req, res); + } catch (ex) { + return returnError(ex, res, req); + } + server.getFiatRate(opts, function(err, rates) { + if (err) return returnError(err, res, req); + res.json(rates); }); });