View Issue Details

IDProjectCategoryView StatusLast Update
0001819GNUnettransport servicepublic2011-10-31 12:00
Reporteramatus Assigned ToMatthias Wachs  
PriorityurgentSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product VersionGit master 
Summary0001819: Peers which cannot accept incoming connections also can't establish outgoing sessions.
DescriptionAssume Alice cannot accept incoming connections, when she connects to Bob she sends a HELLO, Bob tries to validate all addresses in the HELLO but fails and does not store Alice's peer ID or public key. When Alice then sends CORE_SET_KEY and CORE_PING to Bob he just ignores them.
Additional InformationBob's statistics after Alice is started and stopped twice:
 core # Delayed connecting due to lack of public key : 10
 transport # bytes TCP was asked to transmit : 1488
 transport # keepalives sent : 4
 transport # TCP sessions active : 0
 transport # network-level TCP disconnect events : 4
 core # PING messages received : 8
 transport # bytes received via TCP : 6240
 core # session keys received : 8
!hostlist hostlist requests processed : 4
 core # SET_KEY messages deferred (need public key) : 3

 transport # bytes transmitted via TCP : 1310
 transport # PONGs unicast via reliable transport : 2
 transport # bytes currently in TCP buffers : 0
 transport # PING messages received : 2
 transport # SET QUOTA messages received : 2
 transport # peers connected : 1
 transport # bytes in message queue for other peers : 32
 core # peers connected (transport) : 1
 core # neighbour entries allocated : 1
 transport # TCP WELCOME messages received : 2
 transport # peers disconnected due to timeout : 1
 transport # bytes discarded by TCP (disconnect) : 426
 transport # PING without HELLO messages sent : 2
 core # discarded lower priority CORE_SEND requests : 0
 core # discarded CORE_SEND requests : 0
TagsNo tags attached.

Activities

Matthias Wachs

2011-10-04 11:58

reporter   ~0004679

fixed in 17161:

when HELLO message arrives, peer's public key is added to peerinfo even no addresses could be validated

Matthias Wachs

2011-10-04 11:58

reporter   ~0004680

Next step:

change testcases

Christian Grothoff

2011-10-05 22:09

manager   ~0004691

Status?

Matthias Wachs

2011-10-06 13:49

reporter   ~0004693

Status:

- checked test cfg files:
p1 behind nat, p2 reachable
- checked test behaviour
p1 connects to p2
p2 sends to p1

- checked peerinfo status:
without fix 17161
p1 has 2 HELLOS, p2 has own HELLO
with 17161
p1 && p2 have both hellos

Issue History

Date Modified Username Field Change
2011-10-04 04:26 amatus New Issue
2011-10-04 04:26 amatus Status new => assigned
2011-10-04 04:26 amatus Assigned To => Christian Grothoff
2011-10-04 10:54 Christian Grothoff Assigned To Christian Grothoff => Matthias Wachs
2011-10-04 10:54 Christian Grothoff Priority high => urgent
2011-10-04 11:22 Matthias Wachs Status assigned => acknowledged
2011-10-04 11:58 Matthias Wachs Note Added: 0004679
2011-10-04 11:58 Matthias Wachs Note Added: 0004680
2011-10-04 13:41 Christian Grothoff Status acknowledged => assigned
2011-10-04 13:45 Christian Grothoff Target Version => 0.9.0pre4
2011-10-05 22:09 Christian Grothoff Note Added: 0004691
2011-10-06 13:49 Matthias Wachs Note Added: 0004693
2011-10-06 13:49 Matthias Wachs Status assigned => resolved
2011-10-06 13:49 Matthias Wachs Resolution open => fixed
2011-10-31 12:00 Christian Grothoff Status resolved => closed