[Swan-dev] tests failing due to retransmissions

Paul Wouters paul at nohats.ca
Tue Oct 10 19:26:20 UTC 2017


On Tue, 10 Oct 2017, Antony Antony wrote:

>
> Another line that show up in several (20+ output files) IKEv1 tests as diff
> is:
>
> "discarding duplicate packet; already STATE_MAIN_I3"
>
> I wonder how useful is this line, in a typical user case. Why not change to
> a debug line?

I have no problem with making that a debug line. It matterd when any
packet retransmission was 20 seconds, but for this 0.5s I think it is
fine not to tell the user.

> If we remove this line noise in ikev1 could be a bit less.  A similar line
> in ikev2, is is in state.c  I see it 6-10 tests.

It will be a bit less, but anything with packet counters will still
trigger.

Paul


More information about the Swan-dev mailing list