[IPsec] Fw: New Version Notification for draft-smyslov-ipsecme-ikev2-null-auth-01.txt

[Tero Kivinen <[e-mail]>]

Valery Smyslov writes:
> I've posted new version of the NULL Auth Method draft.
> It addresses comments received from Yaron Sheffer and Paul Wouters.

Ah, I just managed to read the -00 version... Oh well, reading diffs
next...

Anyways I think the document is in quite good shape, I think the
section 2.2 needs to be more specific about how to send the empty ID
payload. I think the idea of sending ID_IPV4_ADDR with 0 bytes of
data is very bad idea. The current text says you can omit data, and
that the type can be anything. The problem is that in most cases the
implementation has code that will parse the ID payload using standard
rules and now if the ID payload has type of ID_IPV4_ADDR and 0 bytes
of data, the parsing will fail.

It would be better to say that if you are sending empty ID payload,
you msut use ID_KEY_ID type which already allows any data, including
empty.

Actually I now noticed you changed the "SHOULD be ignored" to "MUST be
ignored", and I think that is again bad idea. I think logging and
auditing the ID for problem solving purposes is good idea even if it
does not have any meaning for the authentication. I.e. at least then I
can contact helpdesk and say that my NULL authentication connection to
server 1.2.3.4 failed, and I have no idea why, can you help. Oh, my ID
payload had ID_KEY_ID 0324234mkdsff43r5, if that helps you to find it
from your logs... 
-- 
[e-mail]