Document: draft-ietf-ipsec-udp-encaps-08
Reviewer: Scott Brim
Date: March 31, 2004

Good but there are a few nits.  The content is fine, but ...

References need to be updated, e.g. Aboda03 is now RFC3715.  In any case
There are some issues, mostly small.  I don't know if I would even say
it's on the right track.  See Rob Austein's comment about whether
carrying all of this over IKE is architecturally sane.

It still doesn't seem to address the IPv6 differences explicitly.  It
just says there is "no reason why not".  
specific sections in other documents were accurate, but if they have the
name wrong, I'd be suspicious.

"2.2 IKE Header Format for Port 4500" -- I know what this is about, but
the use of Port 4500 is not mentioned before this.  

On page 7:

   2.  If the protocol header after the ESP header is a TCP/UDP header,
       recompute the checksum field in the TCP/UDP header.

   3.  If the protocol header after the ESP header is an UDP header,
       zero the checksum field in the UDP header. If the protocol header
       ...

#3 is actually the details of #2.  Perhaps it should be a sub-bullet.