Merge pull request #316 from BeaudanBrown/fix-filtering

Fix silly filtering bug, add filtering for swarm nodes and lint

js/modules/loki_message_api.js

@@ -283,7 +283,7 @@ class LokiMessageAPI {
       options
     );
     return result.messages || [];
-  };
+  }
 
   async startLongPolling(numConnections, callback) {
     this.ourSwarmNodes = {};
@@ -293,7 +293,9 @@ class LokiMessageAPI {
       nodes = await lokiSnodeAPI.getSwarmNodesForPubKey(this.ourKey);
     }
     for (let i = 0; i < nodes.length; i += 1) {
-      const lastHash = await window.Signal.Data.getLastHashBySnode(nodes[i].address);
+      const lastHash = await window.Signal.Data.getLastHashBySnode(
+        nodes[i].address
+      );
       this.ourSwarmNodes[nodes[i].address] = {
         lastHash,
         ip: nodes[i].ip,

js/modules/loki_snode_api.js

@@ -92,10 +92,10 @@ class LokiSnodeAPI {
       );
       // Filter 0.0.0.0 nodes which haven't submitted uptime proofs
       const snodes = result.result.service_node_states.filter(
-        snode => snode.address !== '0.0.0.0'
+        snode => snode.public_ip !== '0.0.0.0'
       );
       this.randomSnodePool = snodes.map(snode => ({
-        address: snode.public_ip,
+        ip: snode.public_ip,
         port: snode.storage_port,
       }));
     } catch (e) {
@@ -162,21 +162,17 @@ class LokiSnodeAPI {
 
   async getSwarmNodes(pubKey) {
     // TODO: Hit multiple random nodes and merge lists?
-    const { address, port } = await this.getRandomSnodeAddress();
+    const { ip, port } = await this.getRandomSnodeAddress();
     try {
-      const result = await rpc(
+      const result = await rpc(`https://${ip}`, port, 'get_snodes_for_pubkey', {
-        `https://${address}`,
+        pubKey,
-        port,
+      });
-        'get_snodes_for_pubkey',
+      const snodes = result.snodes.filter(snode => snode.ip !== '0.0.0.0');
-        {
+      return snodes;
-          pubKey,
-        }
-      );
-      return result.snodes;
     } catch (e) {
       this.randomSnodePool = _.without(
         this.randomSnodePool,
-        _.find(this.randomSnodePool, { address })
+        _.find(this.randomSnodePool, { ip })
       );
       return this.getSwarmNodes(pubKey);
     }