[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Alan's Review of draft-lourdelet-radext-ipv6-access-01.txt






----- Forwarded Message ----
> From: Alan DeKok <aland@deployingradius.com>
> To: Behcet Sarikaya <sarikaya@ieee.org>
> Sent: Friday, August 21, 2009 2:37:58 PM
> Subject: Re: Fw: Review of draft-lourdelet-radext-ipv6-access-01.txt

>   My review is:
> 
> IPv6-Address:
> 
>   I'd suggest giving it a more descriptive name, like "Uplink-IPv6-Address"
> 
> IPv6-Route-Option-Preference:
> 
>   2 octet signed integers aren't nice.  I would suggest using a normal
> 32-bit integer.  This should work, if I understand it correctl.
> 
> IPv6-Route-Option-Lifetime:
> Auth-IPv6-Prefix-Valid-Lifetime:
> 
>   Defining a tag field *and* a 32-bit integer means that you're
> re-defining the meaning of a tag + integer attribute.  See RFC 2868,
> Section 3.1 for the historical definition.
> 
>   It would seem to be OK to have a 24-bit lifetime.  That would allow
> everyone to use this attribute by simply updating their dictionaries,
> rather than writing new code to handle a new format.
> 
> Prefix-Lifetime-Service-Type:
> 
>   This could just be a normal 32-bit integer, rather than a 2-byte integer.
> 
>   Other than that, it looks OK to me.
> 
> >>
> >>  Alan DeKok.





--
to unsubscribe send a message to radiusext-request@ops.ietf.org with
the word 'unsubscribe' in a single line as the message text body.
archive: <http://psg.com/lists/radiusext/>