[FIX] code review fixes

src/controllers/routes.js

@@ -1,5 +1,7 @@
 'use strict'
 
+const co = require('co')
+const log = require('../common').log('routes')
 const requestUtil = require('five-bells-shared/utils/request')
 const routingTables = require('../services/routing-tables')
 const routeBroadcaster = require('../services/route-broadcaster')
@@ -15,8 +17,11 @@ exports.post = function * () {
 
   const connector = routes[0] && routes[0].connector
   if (connector && !knownConnectors[connector]) {
-    yield routeBroadcaster.broadcast()
     knownConnectors[connector] = true
+    co(routeBroadcaster.broadcast.bind(routeBroadcaster))
+      .catch(function (err) {
+        log.warn('error broadcasting routes: ' + err.message)
+      })
   }
 
   this.status = 200

src/lib/route-builder.js

@@ -37,9 +37,9 @@ class RouteBuilder {
     const _nextHop = this._findNextHop(query)
     if (!_nextHop) throwAssetsNotTradedError()
     if (query.sourceAmount) {
-      _nextHop.finalAmount = (+_nextHop.finalAmount * (1 - this.slippage)).toFixed(15)
+      _nextHop.finalAmount = (new BigNumber(_nextHop.finalAmount)).times(1 - this.slippage).toString()
     } else { // fixed destinationAmount
-      _nextHop.sourceAmount = (+_nextHop.sourceAmount * (1 + this.slippage)).toFixed(15)
+      _nextHop.sourceAmount = (new BigNumber(_nextHop.sourceAmount)).times(1 + this.slippage).toString()
     }
     const nextHop = yield this._roundHop(_nextHop)
 
@@ -62,6 +62,9 @@ class RouteBuilder {
   }
 
   /**
+   * Given a source transfer with an embedded final transfer, get the next
+   * transfer in the chain.
+   *
    * @param {Transfer} sourceTransfer
    * @returns {Transfer} destinationTransfer
    */