query.go 14.9 KB
Newer Older
Juan Batiz-Benet's avatar
Juan Batiz-Benet committed
1 2 3
package dht

import (
Jeromy's avatar
Jeromy committed
4
	"context"
Steven Allen's avatar
Steven Allen committed
5
	"errors"
Adin Schmahmann's avatar
Adin Schmahmann committed
6
	"fmt"
7
	"math"
8
	"sync"
Aarsh Shah's avatar
Aarsh Shah committed
9
	"time"
10

11 12
	"github.com/libp2p/go-libp2p-core/network"
	"github.com/libp2p/go-libp2p-core/peer"
13
	pstore "github.com/libp2p/go-libp2p-core/peerstore"
Adin Schmahmann's avatar
Adin Schmahmann committed
14
	"github.com/libp2p/go-libp2p-core/routing"
15

16
	"github.com/google/uuid"
Adin Schmahmann's avatar
Adin Schmahmann committed
17
	"github.com/libp2p/go-libp2p-kad-dht/qpeerset"
18
	kb "github.com/libp2p/go-libp2p-kbucket"
Juan Batiz-Benet's avatar
Juan Batiz-Benet committed
19 20
)

Steven Allen's avatar
Steven Allen committed
21 22 23
// ErrNoPeersQueried is returned when we failed to connect to any peers.
var ErrNoPeersQueried = errors.New("failed to query any peers")

Adin Schmahmann's avatar
Adin Schmahmann committed
24
type queryFn func(context.Context, peer.ID) ([]*peer.AddrInfo, error)
Adin Schmahmann's avatar
Adin Schmahmann committed
25
type stopFn func() bool
26

Aarsh Shah's avatar
Aarsh Shah committed
27
// query represents a single DHT query.
Adin Schmahmann's avatar
Adin Schmahmann committed
28
type query struct {
29 30 31 32 33 34
	// unique identifier for the lookup instance
	id uuid.UUID

	// target key for the lookup
	key string

35 36
	// the query context.
	ctx context.Context
Adin Schmahmann's avatar
Adin Schmahmann committed
37

Adin Schmahmann's avatar
Adin Schmahmann committed
38
	dht *IpfsDHT
Jeromy's avatar
Jeromy committed
39

Adin Schmahmann's avatar
Adin Schmahmann committed
40 41 42
	// seedPeers is the set of peers that seed the query
	seedPeers []peer.ID

43 44 45
	// peerTimes contains the duration of each successful query to a peer
	peerTimes map[peer.ID]time.Duration

Adin Schmahmann's avatar
Adin Schmahmann committed
46 47 48 49 50 51
	// queryPeers is the set of peers known by this query and their respective states.
	queryPeers *qpeerset.QueryPeerset

	// terminated is set when the first worker thread encounters the termination condition.
	// Its role is to make sure that once termination is determined, it is sticky.
	terminated bool
52

53 54 55
	// waitGroup ensures lookup does not end until all query goroutines complete.
	waitGroup sync.WaitGroup

56 57 58 59 60
	// the function that will be used to query a single peer.
	queryFn queryFn

	// stopFn is used to determine if we should stop the WHOLE disjoint query.
	stopFn stopFn
61 62
}

Adin Schmahmann's avatar
Adin Schmahmann committed
63
type lookupWithFollowupResult struct {
Aarsh Shah's avatar
Aarsh Shah committed
64 65
	peers []peer.ID            // the top K not unreachable peers at the end of the query
	state []qpeerset.PeerState // the peer states at the end of the query
Adin Schmahmann's avatar
Adin Schmahmann committed
66

Adin Schmahmann's avatar
Adin Schmahmann committed
67 68 69 70 71 72 73 74 75 76 77 78
	// indicates that neither the lookup nor the followup has been prematurely terminated by an external condition such
	// as context cancellation or the stop function being called.
	completed bool
}

// runLookupWithFollowup executes the lookup on the target using the given query function and stopping when either the
// context is cancelled or the stop function returns true. Note: if the stop function is not sticky, i.e. it does not
// return true every time after the first time it returns true, it is not guaranteed to cause a stop to occur just
// because it momentarily returns true.
//
// After the lookup is complete the query function is run (unless stopped) against all of the top K peers from the
// lookup that have not already been successfully queried.
Aarsh Shah's avatar
Aarsh Shah committed
79
func (dht *IpfsDHT) runLookupWithFollowup(ctx context.Context, target string, queryFn queryFn, stopFn stopFn) (*lookupWithFollowupResult, error) {
Adin Schmahmann's avatar
Adin Schmahmann committed
80
	// run the query
Aarsh Shah's avatar
Aarsh Shah committed
81
	lookupRes, err := dht.runQuery(ctx, target, queryFn, stopFn)
Adin Schmahmann's avatar
Adin Schmahmann committed
82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108
	if err != nil {
		return nil, err
	}

	// query all of the top K peers we've either Heard about or have outstanding queries we're Waiting on.
	// This ensures that all of the top K results have been queried which adds to resiliency against churn for query
	// functions that carry state (e.g. FindProviders and GetValue) as well as establish connections that are needed
	// by stateless query functions (e.g. GetClosestPeers and therefore Provide and PutValue)
	queryPeers := make([]peer.ID, 0, len(lookupRes.peers))
	for i, p := range lookupRes.peers {
		if state := lookupRes.state[i]; state == qpeerset.PeerHeard || state == qpeerset.PeerWaiting {
			queryPeers = append(queryPeers, p)
		}
	}

	if len(queryPeers) == 0 {
		return lookupRes, nil
	}

	// return if the lookup has been externally stopped
	if ctx.Err() != nil || stopFn() {
		lookupRes.completed = false
		return lookupRes, nil
	}

	doneCh := make(chan struct{}, len(queryPeers))
	followUpCtx, cancelFollowUp := context.WithCancel(ctx)
109
	defer cancelFollowUp()
Adin Schmahmann's avatar
Adin Schmahmann committed
110 111 112 113 114 115 116 117 118
	for _, p := range queryPeers {
		qp := p
		go func() {
			_, _ = queryFn(followUpCtx, qp)
			doneCh <- struct{}{}
		}()
	}

	// wait for all queries to complete before returning, aborting ongoing queries if we've been externally stopped
119
	followupsCompleted := 0
Adin Schmahmann's avatar
Adin Schmahmann committed
120 121 122 123
processFollowUp:
	for i := 0; i < len(queryPeers); i++ {
		select {
		case <-doneCh:
124
			followupsCompleted++
Adin Schmahmann's avatar
Adin Schmahmann committed
125 126 127 128 129 130 131 132 133
			if stopFn() {
				cancelFollowUp()
				if i < len(queryPeers)-1 {
					lookupRes.completed = false
				}
				break processFollowUp
			}
		case <-ctx.Done():
			lookupRes.completed = false
134
			cancelFollowUp()
Adin Schmahmann's avatar
Adin Schmahmann committed
135 136 137 138
			break processFollowUp
		}
	}

139 140 141 142 143 144
	if !lookupRes.completed {
		for i := followupsCompleted; i < len(queryPeers); i++ {
			<-doneCh
		}
	}

Adin Schmahmann's avatar
Adin Schmahmann committed
145 146 147
	return lookupRes, nil
}

Aarsh Shah's avatar
Aarsh Shah committed
148
func (dht *IpfsDHT) runQuery(ctx context.Context, target string, queryFn queryFn, stopFn stopFn) (*lookupWithFollowupResult, error) {
Max Inden's avatar
Max Inden committed
149
	// pick the K closest peers to the key in our Routing table.
Adin Schmahmann's avatar
Adin Schmahmann committed
150 151
	targetKadID := kb.ConvertKey(target)
	seedPeers := dht.routingTable.NearestPeers(targetKadID, dht.bucketSize)
152 153 154 155 156 157 158
	if len(seedPeers) == 0 {
		routing.PublishQueryEvent(ctx, &routing.QueryEvent{
			Type:  routing.QueryError,
			Extra: kb.ErrLookupFailure.Error(),
		})
		return nil, kb.ErrLookupFailure
	}
Adin Schmahmann's avatar
Adin Schmahmann committed
159

Aarsh Shah's avatar
Aarsh Shah committed
160
	q := &query{
161 162
		id:         uuid.New(),
		key:        target,
163
		ctx:        ctx,
Aarsh Shah's avatar
Aarsh Shah committed
164 165 166
		dht:        dht,
		queryPeers: qpeerset.NewQueryPeerset(target),
		seedPeers:  seedPeers,
167
		peerTimes:  make(map[peer.ID]time.Duration),
Aarsh Shah's avatar
Aarsh Shah committed
168 169 170
		terminated: false,
		queryFn:    queryFn,
		stopFn:     stopFn,
171 172
	}

Aarsh Shah's avatar
Aarsh Shah committed
173
	// run the query
174 175 176 177 178
	q.run()

	if ctx.Err() == nil {
		q.recordValuablePeers()
	}
179

Aarsh Shah's avatar
Aarsh Shah committed
180
	res := q.constructLookupResult(targetKadID)
Adin Schmahmann's avatar
Adin Schmahmann committed
181
	return res, nil
182 183
}

Aarsh Shah's avatar
Aarsh Shah committed
184
func (q *query) recordPeerIsValuable(p peer.ID) {
185 186 187 188
	if !q.dht.routingTable.UpdateLastUsefulAt(p, time.Now()) {
		// not in routing table
		return
	}
Aarsh Shah's avatar
Aarsh Shah committed
189
}
190 191

func (q *query) recordValuablePeers() {
192 193 194 195 196 197 198 199 200 201 202 203 204 205
	// Valuable peers algorithm:
	// Label the seed peer that responded to a query in the shortest amount of time as the "most valuable peer" (MVP)
	// Each seed peer that responded to a query within some range (i.e. 2x) of the MVP's time is a valuable peer
	// Mark the MVP and all the other valuable peers as valuable
	mvpDuration := time.Duration(math.MaxInt64)
	for _, p := range q.seedPeers {
		if queryTime, ok := q.peerTimes[p]; ok && queryTime < mvpDuration {
			mvpDuration = queryTime
		}
	}

	for _, p := range q.seedPeers {
		if queryTime, ok := q.peerTimes[p]; ok && queryTime < mvpDuration*2 {
			q.recordPeerIsValuable(p)
206 207 208 209
		}
	}
}

Adin Schmahmann's avatar
Adin Schmahmann committed
210
// constructLookupResult takes the query information and uses it to construct the lookup result
Aarsh Shah's avatar
Aarsh Shah committed
211 212
func (q *query) constructLookupResult(target kb.ID) *lookupWithFollowupResult {
	// determine if the query terminated early
Adin Schmahmann's avatar
Adin Schmahmann committed
213
	completed := true
Aarsh Shah's avatar
Aarsh Shah committed
214

215 216 217 218
	// Lookup and starvation are both valid ways for a lookup to complete. (Starvation does not imply failure.)
	// Lookup termination (as defined in isLookupTermination) is not possible in small networks.
	// Starvation is a successful query termination in small networks.
	if !(q.isLookupTermination() || q.isStarvationTermination()) {
Aarsh Shah's avatar
Aarsh Shah committed
219
		completed = false
Adin Schmahmann's avatar
Adin Schmahmann committed
220
	}
221

Aarsh Shah's avatar
Aarsh Shah committed
222
	// extract the top K not unreachable peers
Adin Schmahmann's avatar
Adin Schmahmann committed
223 224
	var peers []peer.ID
	peerState := make(map[peer.ID]qpeerset.PeerState)
225
	qp := q.queryPeers.GetClosestNInStates(q.dht.bucketSize, qpeerset.PeerHeard, qpeerset.PeerWaiting, qpeerset.PeerQueried)
Aarsh Shah's avatar
Aarsh Shah committed
226 227 228 229
	for _, p := range qp {
		state := q.queryPeers.GetState(p)
		peerState[p] = state
		peers = append(peers, p)
230 231
	}

Adin Schmahmann's avatar
Adin Schmahmann committed
232 233
	// get the top K overall peers
	sortedPeers := kb.SortClosestPeers(peers, target)
Aarsh Shah's avatar
Aarsh Shah committed
234 235
	if len(sortedPeers) > q.dht.bucketSize {
		sortedPeers = sortedPeers[:q.dht.bucketSize]
236 237
	}

Aarsh Shah's avatar
Aarsh Shah committed
238
	// return the top K not unreachable peers as well as their states at the end of the query
Adin Schmahmann's avatar
Adin Schmahmann committed
239 240 241 242 243 244 245 246 247
	res := &lookupWithFollowupResult{
		peers:     sortedPeers,
		state:     make([]qpeerset.PeerState, len(sortedPeers)),
		completed: completed,
	}

	for i, p := range sortedPeers {
		res.state[i] = peerState[p]
	}
248 249

	return res
Adin Schmahmann's avatar
Adin Schmahmann committed
250
}
251

Adin Schmahmann's avatar
Adin Schmahmann committed
252
type queryUpdate struct {
253 254 255 256 257
	cause       peer.ID
	queried     []peer.ID
	heard       []peer.ID
	unreachable []peer.ID

258
	queryDuration time.Duration
Adin Schmahmann's avatar
Adin Schmahmann committed
259
}
260

261
func (q *query) run() {
Adin Schmahmann's avatar
Adin Schmahmann committed
262 263
	pathCtx, cancelPath := context.WithCancel(q.ctx)
	defer cancelPath()
264

Adin Schmahmann's avatar
Adin Schmahmann committed
265
	alpha := q.dht.alpha
266

Adin Schmahmann's avatar
Adin Schmahmann committed
267
	ch := make(chan *queryUpdate, alpha)
268
	ch <- &queryUpdate{cause: q.dht.self, heard: q.seedPeers}
269

270 271
	// return only once all outstanding queries have completed.
	defer q.waitGroup.Wait()
Adin Schmahmann's avatar
Adin Schmahmann committed
272
	for {
273
		var cause peer.ID
Adin Schmahmann's avatar
Adin Schmahmann committed
274 275
		select {
		case update := <-ch:
276 277
			q.updateState(pathCtx, update)
			cause = update.cause
Adin Schmahmann's avatar
Adin Schmahmann committed
278
		case <-pathCtx.Done():
279
			q.terminate(pathCtx, cancelPath, LookupCancelled)
Adin Schmahmann's avatar
Adin Schmahmann committed
280 281
		}

282 283 284 285
		// calculate the maximum number of queries we could be spawning.
		// Note: NumWaiting will be updated in spawnQuery
		maxNumQueriesToSpawn := alpha - q.queryPeers.NumWaiting()

Adin Schmahmann's avatar
Adin Schmahmann committed
286
		// termination is triggered on end-of-lookup conditions or starvation of unused peers
287 288 289
		// it also returns the peers we should query next for a maximum of `maxNumQueriesToSpawn` peers.
		ready, reason, qPeers := q.isReadyToTerminate(pathCtx, maxNumQueriesToSpawn)
		if ready {
290 291
			q.terminate(pathCtx, cancelPath, reason)
		}
Adin Schmahmann's avatar
Adin Schmahmann committed
292

293 294
		if q.terminated {
			return
Adin Schmahmann's avatar
Adin Schmahmann committed
295 296
		}

297
		// try spawning the queries, if there are no available peers to query then we won't spawn them
298 299
		for _, p := range qPeers {
			q.spawnQuery(pathCtx, cause, p, ch)
Adin Schmahmann's avatar
Adin Schmahmann committed
300 301 302 303
		}
	}
}

304
// spawnQuery starts one query, if an available heard peer is found
305
func (q *query) spawnQuery(ctx context.Context, cause peer.ID, queryPeer peer.ID, ch chan<- *queryUpdate) {
Alan Shaw's avatar
Alan Shaw committed
306 307 308 309 310 311 312
	PublishLookupEvent(ctx,
		NewLookupEvent(
			q.dht.self,
			q.id,
			q.key,
			NewLookupUpdateEvent(
				cause,
313 314 315 316 317
				q.queryPeers.GetReferrer(queryPeer),
				nil,                  // heard
				[]peer.ID{queryPeer}, // waiting
				nil,                  // queried
				nil,                  // unreachable
Alan Shaw's avatar
Alan Shaw committed
318 319 320 321 322
			),
			nil,
			nil,
		),
	)
323
	q.queryPeers.SetState(queryPeer, qpeerset.PeerWaiting)
Alan Shaw's avatar
Alan Shaw committed
324
	q.waitGroup.Add(1)
325
	go q.queryPeer(ctx, ch, queryPeer)
Adin Schmahmann's avatar
Adin Schmahmann committed
326 327
}

328
func (q *query) isReadyToTerminate(ctx context.Context, nPeersToQuery int) (bool, LookupTerminationReason, []peer.ID) {
Adin Schmahmann's avatar
Adin Schmahmann committed
329 330
	// give the application logic a chance to terminate
	if q.stopFn() {
331
		return true, LookupStopped, nil
Adin Schmahmann's avatar
Adin Schmahmann committed
332 333
	}
	if q.isStarvationTermination() {
334
		return true, LookupStarvation, nil
Adin Schmahmann's avatar
Adin Schmahmann committed
335 336
	}
	if q.isLookupTermination() {
337
		return true, LookupCompleted, nil
Adin Schmahmann's avatar
Adin Schmahmann committed
338
	}
339 340 341 342 343 344 345 346 347 348 349 350 351 352

	// The peers we query next should be ones that we have only Heard about.
	var peersToQuery []peer.ID
	peers := q.queryPeers.GetClosestInStates(qpeerset.PeerHeard)
	count := 0
	for _, p := range peers {
		peersToQuery = append(peersToQuery, p)
		count++
		if count == nPeersToQuery {
			break
		}
	}

	return false, -1, peersToQuery
Adin Schmahmann's avatar
Adin Schmahmann committed
353 354 355 356 357
}

// From the set of all nodes that are not unreachable,
// if the closest beta nodes are all queried, the lookup can terminate.
func (q *query) isLookupTermination() bool {
358
	peers := q.queryPeers.GetClosestNInStates(q.dht.beta, qpeerset.PeerHeard, qpeerset.PeerWaiting, qpeerset.PeerQueried)
Adin Schmahmann's avatar
Adin Schmahmann committed
359 360 361
	for _, p := range peers {
		if q.queryPeers.GetState(p) != qpeerset.PeerQueried {
			return false
Adin Schmahmann's avatar
Adin Schmahmann committed
362
		}
363
	}
Adin Schmahmann's avatar
Adin Schmahmann committed
364 365 366 367 368
	return true
}

func (q *query) isStarvationTermination() bool {
	return q.queryPeers.NumHeard() == 0 && q.queryPeers.NumWaiting() == 0
369 370
}

371 372 373 374
func (q *query) terminate(ctx context.Context, cancel context.CancelFunc, reason LookupTerminationReason) {
	if q.terminated {
		return
	}
Alan Shaw's avatar
Alan Shaw committed
375 376 377 378 379 380 381 382 383 384 385 386 387

	PublishLookupEvent(ctx,
		NewLookupEvent(
			q.dht.self,
			q.id,
			q.key,
			nil,
			nil,
			NewLookupTerminateEvent(reason),
		),
	)
	cancel() // abort outstanding queries
	q.terminated = true
388 389
}

Adin Schmahmann's avatar
Adin Schmahmann committed
390 391
// queryPeer queries a single peer and reports its findings on the channel.
// queryPeer does not access the query state in queryPeers!
392 393 394
func (q *query) queryPeer(ctx context.Context, ch chan<- *queryUpdate, p peer.ID) {
	defer q.waitGroup.Done()
	dialCtx, queryCtx := ctx, ctx
Adin Schmahmann's avatar
Adin Schmahmann committed
395

396
	startQuery := time.Now()
397
	// dial the peer
Adin Schmahmann's avatar
Adin Schmahmann committed
398
	if err := q.dht.dialPeer(dialCtx, p); err != nil {
Aarsh Shah's avatar
Aarsh Shah committed
399 400 401 402
		// remove the peer if there was a dial failure..but not because of a context cancellation
		if dialCtx.Err() == nil {
			q.dht.peerStoppedDHT(q.dht.ctx, p)
		}
403
		ch <- &queryUpdate{cause: p, unreachable: []peer.ID{p}}
Adin Schmahmann's avatar
Adin Schmahmann committed
404
		return
Adin Schmahmann's avatar
Adin Schmahmann committed
405
	}
406 407

	// send query RPC to the remote peer
Adin Schmahmann's avatar
Adin Schmahmann committed
408 409
	newPeers, err := q.queryFn(queryCtx, p)
	if err != nil {
Aarsh Shah's avatar
Aarsh Shah committed
410 411 412
		if queryCtx.Err() == nil {
			q.dht.peerStoppedDHT(q.dht.ctx, p)
		}
413
		ch <- &queryUpdate{cause: p, unreachable: []peer.ID{p}}
Adin Schmahmann's avatar
Adin Schmahmann committed
414
		return
Juan Batiz-Benet's avatar
Juan Batiz-Benet committed
415
	}
416

417 418
	queryDuration := time.Since(startQuery)

Aarsh Shah's avatar
Aarsh Shah committed
419 420 421
	// query successful, try to add to RT
	q.dht.peerFound(q.dht.ctx, p, true)

Adin Schmahmann's avatar
Adin Schmahmann committed
422 423
	// process new peers
	saw := []peer.ID{}
Adin Schmahmann's avatar
Adin Schmahmann committed
424 425 426 427 428
	for _, next := range newPeers {
		if next.ID == q.dht.self { // don't add self.
			logger.Debugf("PEERS CLOSER -- worker for: %v found self", p)
			continue
		}
429

430 431 432 433
		// add any other know addresses for the candidate peer.
		curInfo := q.dht.peerstore.PeerInfo(next.ID)
		next.Addrs = append(next.Addrs, curInfo.Addrs...)

Adin Schmahmann's avatar
Adin Schmahmann committed
434
		// add their addresses to the dialer's peerstore
435 436 437 438 439
		//
		// add the next peer to the query if matches the query target even if it would otherwise fail the query filter
		// TODO: this behavior is really specific to how FindPeer works and not GetClosestPeers or any other function
		isTarget := string(next.ID) == q.key
		if isTarget || q.dht.queryPeerFilter(q.dht, *next) {
440
			q.dht.maybeAddAddrs(next.ID, next.Addrs, pstore.TempAddrTTL)
441 442
			saw = append(saw, next.ID)
		}
443
	}
Adin Schmahmann's avatar
Adin Schmahmann committed
444

445
	ch <- &queryUpdate{cause: p, heard: saw, queried: []peer.ID{p}, queryDuration: queryDuration}
Adin Schmahmann's avatar
Adin Schmahmann committed
446
}
447

448 449 450 451 452 453 454 455 456 457 458 459 460 461 462 463 464 465 466 467 468 469
func (q *query) updateState(ctx context.Context, up *queryUpdate) {
	if q.terminated {
		panic("update should not be invoked after the logical lookup termination")
	}
	PublishLookupEvent(ctx,
		NewLookupEvent(
			q.dht.self,
			q.id,
			q.key,
			nil,
			NewLookupUpdateEvent(
				up.cause,
				up.cause,
				up.heard,       // heard
				nil,            // waiting
				up.queried,     // queried
				up.unreachable, // unreachable
			),
			nil,
		),
	)
	for _, p := range up.heard {
Adin Schmahmann's avatar
Adin Schmahmann committed
470 471 472
		if p == q.dht.self { // don't add self.
			continue
		}
473
		q.queryPeers.TryAdd(p, up.cause)
Adin Schmahmann's avatar
Adin Schmahmann committed
474 475 476 477 478 479 480
	}
	for _, p := range up.queried {
		if p == q.dht.self { // don't add self.
			continue
		}
		if st := q.queryPeers.GetState(p); st == qpeerset.PeerWaiting {
			q.queryPeers.SetState(p, qpeerset.PeerQueried)
481
			q.peerTimes[p] = up.queryDuration
Adin Schmahmann's avatar
Adin Schmahmann committed
482 483 484 485 486 487 488 489
		} else {
			panic(fmt.Errorf("kademlia protocol error: tried to transition to the queried state from state %v", st))
		}
	}
	for _, p := range up.unreachable {
		if p == q.dht.self { // don't add self.
			continue
		}
490

Adin Schmahmann's avatar
Adin Schmahmann committed
491 492 493 494 495
		if st := q.queryPeers.GetState(p); st == qpeerset.PeerWaiting {
			q.queryPeers.SetState(p, qpeerset.PeerUnreachable)
		} else {
			panic(fmt.Errorf("kademlia protocol error: tried to transition to the unreachable state from state %v", st))
		}
496
	}
497
}
Juan Batiz-Benet's avatar
Juan Batiz-Benet committed
498

Adin Schmahmann's avatar
Adin Schmahmann committed
499
func (dht *IpfsDHT) dialPeer(ctx context.Context, p peer.ID) error {
500
	// short-circuit if we're already connected.
Adin Schmahmann's avatar
Adin Schmahmann committed
501
	if dht.host.Network().Connectedness(p) == network.Connected {
502 503 504
		return nil
	}

Matt Joiner's avatar
Matt Joiner committed
505
	logger.Debug("not connected. dialing.")
Adin Schmahmann's avatar
Adin Schmahmann committed
506
	routing.PublishQueryEvent(ctx, &routing.QueryEvent{
507
		Type: routing.DialingPeer,
508 509 510
		ID:   p,
	})

511
	pi := peer.AddrInfo{ID: p}
Adin Schmahmann's avatar
Adin Schmahmann committed
512
	if err := dht.host.Connect(ctx, pi); err != nil {
Matt Joiner's avatar
Matt Joiner committed
513
		logger.Debugf("error connecting: %s", err)
Adin Schmahmann's avatar
Adin Schmahmann committed
514
		routing.PublishQueryEvent(ctx, &routing.QueryEvent{
515
			Type:  routing.QueryError,
516 517 518 519 520 521
			Extra: err.Error(),
			ID:    p,
		})

		return err
	}
Matt Joiner's avatar
Matt Joiner committed
522
	logger.Debugf("connected. dial success.")
523 524
	return nil
}