Skip to content

Commit

Permalink
respect contexts while reading messages in dht
Browse files Browse the repository at this point in the history
License: MIT
Signed-off-by: Jeromy <why@ipfs.io>
  • Loading branch information
whyrusleeping committed Jun 9, 2016
1 parent 0e81124 commit 73463cd
Showing 1 changed file with 15 additions and 1 deletion.
16 changes: 15 additions & 1 deletion routing/dht/dht_net.go
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ func (ms *messageSender) SendRequest(ctx context.Context, pmes *pb.Message) (*pb
log.Event(ctx, "dhtSentMessage", ms.dht.self, ms.p, pmes)

mes := new(pb.Message)
if err := ms.r.ReadMsg(mes); err != nil {
if err := ms.ctxReadMsg(ctx, mes); err != nil {
ms.s.Close()
ms.s = nil
return nil, err
Expand All @@ -227,3 +227,17 @@ func (ms *messageSender) SendRequest(ctx context.Context, pmes *pb.Message) (*pb

return mes, nil
}

func (ms *messageSender) ctxReadMsg(ctx context.Context, mes *pb.Message) error {
errc := make(chan error, 1)
go func() {
errc <- ms.r.ReadMsg(mes)
}()

select {
case err := <-errc:
return err
case <-ctx.Done():
return ctx.Err()
}
}

0 comments on commit 73463cd

Please sign in to comment.