Subject: [AAA-WG]: Issue with draft-ietf-aaa-diameter-nasreq-15.txt

From: "Javier Gonzalez Gallego" <[email-masked]>
Date: Wed, 7 Jul 2004 15:43:48 +0100

Description of issue: Oversight of one AVP in the AAA message
Submitter name: Javier Gonzalez
Date first submitted: 07.07.2004
Reference: 
Document:  draft-ietf-aaa-diameter-nasreq-15.txt
Comment type: 
Priority: ['S' Must fix | '1' Should fix | '2' May fix ]
Section: 3.2

Rationale/Explanation of issues: 

In the AAA message, the *[Failed-AVP] is missing. This is needed to report
some of the permanent failures as inicated by RFC 3588.
To follow the pattern of other Answer messages (RAA), it should be inserted
after 
                      [ Error-Message ]
                      [ Error-Reporting-Host ]

Requested change: 

Before:
 <AA-Answer> ::= < Diameter Header: 265, PXY >
                      < Session-Id >
                      { Auth-Application-Id }
                      { Auth-Request-Type }
                      { Result-Code }
                      { Origin-Host }
                      { Origin-Realm }
                      [ User-Name ]
                      [ Service-Type ]
                    * [ Class ]
                    * [ Configuration-Token ]
                      [ Acct-Interim-Interval ]
                      [ Error-Message ]
                      [ Error-Reporting-Host ]
                      [ Idle-Timeout ]
                      [ Authorization-Lifetime ]
....
(Skipped text)

After:
 <AA-Answer> ::= < Diameter Header: 265, PXY >
                      < Session-Id >
                      { Auth-Application-Id }
                      { Auth-Request-Type }
                      { Result-Code }
                      { Origin-Host }
                      { Origin-Realm }
                      [ User-Name ]
                      [ Service-Type ]
                    * [ Class ]
                    * [ Configuration-Token ]
                      [ Acct-Interim-Interval ]
                      [ Error-Message ]
                      [ Error-Reporting-Host ]
			  * [ Failed-AVP ]
                      [ Idle-Timeout ]
                      [ Authorization-Lifetime ]
....
(Skipped text)