From 445d8eef7b55f1ad6debdae05cce0ac73112e4af Mon Sep 17 00:00:00 2001
From: Henrique Dias <hacdias@gmail.com>
Date: Thu, 10 Oct 2019 15:15:37 +0100
Subject: [PATCH] fix: remove peer location cache

Remove peer location cache as peers might move location.

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
---
 src/bundles/peer-locations.js | 16 +---------------
 1 file changed, 1 insertion(+), 15 deletions(-)

diff --git a/src/bundles/peer-locations.js b/src/bundles/peer-locations.js
index 759293512..0e4e0d1bd 100644
--- a/src/bundles/peer-locations.js
+++ b/src/bundles/peer-locations.js
@@ -288,13 +288,6 @@ const parseNotes = (peer, bootstrapPeers) => {
 
 class PeerLocationResolver {
   constructor (opts) {
-    this.peerLocCache = getConfiguredCache({
-      name: 'peerLocations',
-      version: 1,
-      maxAge: ms.weeks(1),
-      ...opts.cache
-    })
-
     this.geoipCache = getConfiguredCache({
       name: 'geoipCache',
       version: 1,
@@ -306,12 +299,6 @@ class PeerLocationResolver {
   }
 
   async lookupWithCaches (peerId, addr, getIpfs) {
-    // maybe we have it cached by peerid already
-    let location = await this.peerLocCache.get(peerId)
-    if (location) {
-      return location
-    }
-
     const ipv4Tuple = Multiaddr(addr).stringTuples().find(isNonHomeIPv4)
     if (!ipv4Tuple) {
       throw new Error(`Unable to resolve location for non-IPv4 address ${addr}`)
@@ -320,7 +307,7 @@ class PeerLocationResolver {
     var ipv4Addr = ipv4Tuple[1]
 
     // maybe we have it cached by ipv4 address already, check that.
-    location = await this.geoipCache.get(ipv4Addr)
+    let location = await this.geoipCache.get(ipv4Addr)
     if (location) {
       return location
     }
@@ -334,7 +321,6 @@ class PeerLocationResolver {
         geoip.lookup(ipfs, ipv4Addr, (err, data) => {
           // save it in the caches, and remove the promise.
           this.geoipCache.set(ipv4Addr, data)
-          this.peerLocCache.set(peerId, data)
           delete this.geoipLookupPromises[ipv4Addr]
 
           if (err) return reject(err)