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

Re: Review Requested: draft-ietf-radext-extended-attributes-05.txt



Hi,

> Just for the record, I made no such claim: I stated that the issues
> were resolved to my satisfaction.  Certain elements of one of the
> issues I disputed, but since the owner thereof hasn't bothered to
> respond to my comments in over a month, I unilaterally marked it as
> resolved.  It is possible (likely?) that the owner will now respond,
> helping to delay publication further (inadvertently, no doubt).
>

I hope you weren't thinking of me. Going through my list of comments,
the majority of items has been taken care of, with some exceptions.
Breakdown below:

- TLV encoding and M flag: not resolved. Discussion on the list resolved
in that we should make sure also morons are forced to split attributes
correctly (choice of words property of the original commenter). Proposed
text from Alan DeKok was: "The More Flag MUST NOT be set if the Length
is less than 255." I suggest to include that line.

- TLV encoding rules: not resolved. I suggest adding two words to the
sentence in question: "The next octet is the Ext-Len field, representing
the length of the entire TLV *in octets*, including ..."

- figure 3: resolved

- figure 4, other encodings possible: not resolved. I suggest adding
"Note that this encoding is only one out of several possibilities since
there is no strict order in attribute marshalling."

- figure 4, diagram: mostly resolved, one thing missed: Integer c of
value 0x12345678 is still depicted in the Value field as 0x12 0x34 0x78
0x56 (twist in the last two segments)

- nitbit in intro: resolved

- nitbit in first example: resolved

- security considerations: resolved

***
New issues with this -05 draft:
***

- figure 3: Ext-Type (cont) is listed as "(0x00" while it is actually
"(0x01)".


Greetings,

Stefan Winter
>
> we are requesting that individuals who filed those issues read the
> current document, and send email to the list indicating whether their
> issue is resolved.
>
> I apparently "own" Issue 272 although it was you, I believe, who
> brought it up initially.  I withdraw it, since it's become quite
> apparent that it's not really a problem at all (except, perhaps, for
> those poor souls who actually try to _use_ Extended Attributes).  Oh,
> well…
>
>
>  
> The RADEXT WG Issues list is available here:
> http://www.drizzle.com/~aboba/RADEXT/
> <http://www.drizzle.com/%7Eaboba/RADEXT/>
>  
> The latest version of the Extended Attributes document can be found here:
> http://tools.ietf.org/html/draft-ietf-radext-extended-attributes-05
>  
> The list of previously opened issues is as follows:
>  
>
> *Extended Attributes*
>
> *Issue Number*
>
> 	
>
> *Status*
>
> 	
>
> *Type*
>
> 	
>
> *Description*
>
> 	
>
> *Owner*
>
> Issue 225
>
> 	
>
> Pending
>
> 	
>
> Technical
>
> 	
>
> Review
> <http://www.drizzle.com/%7Eaboba/RADEXT/radissues2.html#Issue%20225>
>
> 	
>
> Alan DeKok <mailto:aland@deployingradius.com>
>
> Issue 250
>
> 	
>
> Pending
>
> 	
>
> Technical
>
> 	
>
> Comments
> <http://www.drizzle.com/%7Eaboba/RADEXT/radissues2.html#Issue%20250>
>
> 	
>
> Greg Weber <mailto:gdweber@gmail.com>
>
> Issue 251
>
> 	
>
> Pending
>
> 	
>
> Technical
>
> 	
>
> Review
> <http://www.drizzle.com/%7Eaboba/RADEXT/radissues2.html#Issue%20251>
>
> 	
>
> Bernard Aboba <mailto:Bernard_Aboba@hotmail.com>
>
> Issue 272
>
> 	
>
> Pending
>
> 	
>
> Technical
>
> 	
>
> No overall type in Extended Attributes
> <http://www.drizzle.com/%7Eaboba/RADEXT/radissues2.html#Issue%20272>
>
> 	
>
> Glen Zorn <mailto:glenzorn@comcast.net>
>
> Issue 278
>
> 	
>
> Pending
>
> 	
>
> Technical
>
> 	
>
> Comments
> <http://www.drizzle.com/%7Eaboba/RADEXT/radissues2.html#Issue%20278>
>
> 	
>
> Alan DeKok <mailto:aland@deployingradius.com>
>
> Issue 279
>
> 	
>
> No Discussion
>
> 	
>
> Technical
>
> 	
>
> Comments
> <http://www.drizzle.com/%7Eaboba/RADEXT/radissues2.html#Issue%20279>
>
> 	
>
> Stefan Winter <mailto:stefan.winter@restena.lu>
>
>  
>  
> > From: glenzorn@comcast.net
> > To: radiusext@ops.ietf.org
> > CC: glenzorn@comcast.net
> > Subject: RE: I-D Action:draft-ietf-radext-extended-attributes-05.txt
> > Date: Sun, 2 Nov 2008 17:21:26 -0800
> >
> > All known issues are resolved to my satisfaction but YMMV; complaints
> to me.
> > ;-)
>


-- 
Stefan WINTER
Ingenieur de Recherche
Fondation RESTENA - Réseau Téléinformatique de l'Education Nationale et de la Recherche
6, rue Richard Coudenhove-Kalergi
L-1359 Luxembourg

Tel: +352 424409 1
Fax: +352 422473


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