16 Jun
2024
16 Jun
'24
8:06 p.m.
On Jun 15, 2024, at 5:50 PM, Keith Halewood
<Keith.Halewood(a)pitbulluk.org> wrote:
Hi,
I think there’s a bug in in decnet/nsp.py, line 1173
A phase 3 accept is setting its state to self.cc which doesn’t seem to exist. If I change
that to self.run, like the state setting for phase 2 accept, I can then send a message
after accepting the connection without ‘WrongState’ being thrown.
This is pydecnet 1.1-648.
Keith
Could you give me more details?
self.cc definitely exists, that's the "cc" method. And it definitely is
supposed to be used for Phase III and up.
The point of that state is that it waits for an ack from the other end (the sending end),
retransmitting the connect confirm message if needed. You'll get
"WrongState" if you try to send a message immediately after connect confirm,
because the connection isn't ready yet to send data at that point.
I suppose I could queue the data message in the CC state rather than rejecting it.
paul