On Wed, 11 Mar 2020, Andrew Cagney wrote:

- drop "STATE_PARENT_I2: "

It sounds like bad idea to rush this change. An identifier without spaces is
easy grep.

This is an internal variable, it doesn't belong in user visible logs.
It should be removed.

I agree with both of you.

The text paul Proposes, namely:
 sent IKE_AUTH request
is more than sufficient

I pulled the first part of the change as I discovered tests running:
  ipsec status | grep STATE_
that's wrong at so many levels.

It is, but it was easy. But those can be changed to "ipsec briefstatus"
which also just displays the states, without depending on the STATE_
string. But it will change the output slightly so it will require fixing
up a bunch of tests.

Paul
_______________________________________________
Swan-dev mailing list
[email protected]
https://lists.libreswan.org/mailman/listinfo/swan-dev

Reply via email to