[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/>