unistack-org/micro
4
0
Fork 3
Code Issues 17 Pull Requests 1 Actions Packages Projects Releases 258 Wiki Activity

Merge pull request #886 from micro/tunnel-error

Browse Source 
Don't error where the connection is not unicast
This commit is contained in:
Asim Aslam 2019-10-25 15:48:09 +01:00 committed by GitHub
commit e85863d6cc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
tunnel/session.go
View File

@ -110,7 +110,7 @@ func (s *session) newMessage(typ string) *message {
func (s *session) waitFor(msgType string, timeout time.Duration) (*message, error) {
now := time.Now()
after := func() time.Duration {
after := func(timeout time.Duration) time.Duration {
d := time.Since(now)
// dial timeout minus time since
wait := timeout - d
@ -133,7 +133,7 @@ func (s *session) waitFor(msgType string, timeout time.Duration) (*message, erro
}
// got the message
return msg, nil
case <-time.After(after()):
case <-time.After(after(timeout)):
return nil, ErrDialTimeout
case <-s.closed:
return nil, io.EOF
@ -188,9 +188,6 @@ func (s *session) Discover() error {
// wait for announce
_, err = s.waitFor("announce", dialTimeout)
if err != nil {
return err
}
// if its multicast just go ahead because this is best effort
if s.mode != Unicast {