fix: incorrect error handling of lookups when searching for provider records

aschmahmann · aschmahmann · commit b2b0244ed029 · 2020-04-06T01:33:57.000-04:00
diff --git a/routing.go b/routing.go
@@ -606,7 +606,7 @@ func (dht *IpfsDHT) findProvidersAsyncRoutine(ctx context.Context, key multihash
 		},
 	)
 
-	if err != nil && ctx.Err() == nil {
+	if err == nil && ctx.Err() == nil {
 		dht.refreshRTIfNoShortcut(kb.ConvertKey(string(key)), lookupRes)
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -606,7 +606,7 @@ func (dht *IpfsDHT) findProvidersAsyncRoutine(ctx context.Context, key multihash`
`606`	`606`	`},`
`607`	`607`	`)`
`608`	`608`
`609`		`- if err != nil && ctx.Err() == nil {`
	`609`	`+ if err == nil && ctx.Err() == nil {`
`610`	`610`	`dht.refreshRTIfNoShortcut(kb.ConvertKey(string(key)), lookupRes)`
`611`	`611`	`}`
`612`	`612`	`}`