Re: [Dime] RFC 3588 - fixed positioned session-ID AVPs

Victor Fajardo <vf0213@gmail.com> Wed, 01 September 2010 13:15 UTC

Return-Path: <vf0213@gmail.com>
X-Original-To: dime@core3.amsl.com
Delivered-To: dime@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 1A9693A6A1B for <dime@core3.amsl.com>; Wed, 1 Sep 2010 06:15:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.598
X-Spam-Level:
X-Spam-Status: No, score=-2.598 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HTML_MESSAGE=0.001]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id yZLQnIpW3M2L for <dime@core3.amsl.com>; Wed, 1 Sep 2010 06:15:29 -0700 (PDT)
Received: from mail-ww0-f44.google.com (mail-ww0-f44.google.com [74.125.82.44]) by core3.amsl.com (Postfix) with ESMTP id 343F03A6A2F for <dime@ietf.org>; Wed, 1 Sep 2010 06:14:14 -0700 (PDT)
Received: by wwb17 with SMTP id 17so1078850wwb.13 for <dime@ietf.org>; Wed, 01 Sep 2010 06:14:44 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type; bh=3PS952v/WxSR37YdvWLaqR27Qe4mnbAisHEDeeWYBRI=; b=u6JkfZOA2t1BK5IDx2S65ytkQ8UgZiipZUw6eg1MVprM+EJCfD0RsR91X7w5l/FEoZ gbqoaW4Hpv60kNfo/1LD7aWNfeR7L03PaNYsrP/0B4pLVfboiNgHoBs1Ey7EOZKsgqLy KfWMOLc23nsHVNJ3yPwM7QSjkDr6MXOHhqy0E=
DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=ovAjvvFjy5IsksI+pvpQIZOpKeCac88iShaQakDk6ESj9SC4kCXTwGOa3O8pjjSHuQ n2EqPYuBPIGrcPX3q+gjlGYQgvmJrpQc1Gm/1obEWVmkmpztD5ZgTv/iznVz8PwAwN56 Kkh4rmAzlzuCTCIIfUYWacNQb53c68s4qeFqA=
MIME-Version: 1.0
Received: by 10.216.51.212 with SMTP id b62mr286845wec.97.1283346761941; Wed, 01 Sep 2010 06:12:41 -0700 (PDT)
Received: by 10.216.58.130 with HTTP; Wed, 1 Sep 2010 06:12:41 -0700 (PDT)
In-Reply-To: <A51D8ACD861B7E41BFC7FE5C64BE96481167B0FB@MTLEXVS01.ulticom.com>
References: <A51D8ACD861B7E41BFC7FE5C64BE96481167B0F1@MTLEXVS01.ulticom.com> <009f01cb4913$e76b5b50$b64211f0$@net> <A51D8ACD861B7E41BFC7FE5C64BE96481167B0F7@MTLEXVS01.ulticom.com> <AANLkTi=GiPLzuAnLqwRe7sPGMJoRE+LTzBFdZgdCffnZ@mail.gmail.com> <A51D8ACD861B7E41BFC7FE5C64BE96481167B0F9@MTLEXVS01.ulticom.com> <AANLkTiku7LqpiaRmDoB8DPndLv6JNKz_NRR0VK7sAT1E@mail.gmail.com> <A51D8ACD861B7E41BFC7FE5C64BE96481167B0FA@MTLEXVS01.ulticom.com> <AANLkTinAXtekvdU7Z83bFrZkT818VSA2vXaLc69bCTre@mail.gmail.com> <A51D8ACD861B7E41BFC7FE5C64BE96481167B0FB@MTLEXVS01.ulticom.com>
Date: Wed, 01 Sep 2010 09:12:41 -0400
Message-ID: <AANLkTinVm7Cfh-Bzc=zLxaWg9h2BTqoEc8qs2DGcB1Mf@mail.gmail.com>
From: Victor Fajardo <vf0213@gmail.com>
To: David Lehmann <dlehmann@ulticom.com>
Content-Type: multipart/alternative; boundary="0016e6dd8a18de83d7048f327314"
Cc: dime@ietf.org
Subject: Re: [Dime] RFC 3588 - fixed positioned session-ID AVPs
X-BeenThere: dime@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: Diameter Maintanence and Extentions Working Group <dime.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/dime>, <mailto:dime-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/dime>
List-Post: <mailto:dime@ietf.org>
List-Help: <mailto:dime-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dime>, <mailto:dime-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 01 Sep 2010 13:15:32 -0000

Its good to have some clarity, though the statement below probably needs
some re-org since SHOULD is followed with a MUST and theres a
conditional 'may' in between. Anyway, the statement maybe redundant
since each BNF already tells you where the session-id can be found ..

my 2 cents.

On Wed, Sep 1, 2010 at 8:55 AM, David Lehmann <dlehmann@ulticom.com> wrote:

>  OK, BNF rules.    IMHO, this should be noted or clarified in section 8.8.
> e.g.
>
> “When present, the Session-Id SHOULD appear immediately following the
> Diameter Header. Further, the message BNF may mandate that the Session-Id
> MUST be positioned immediately following the message header.  Indeed, all
> messages defined in this RFC require such a positioning. (see Section 3)”
>
>
>
> --
>
> *David Lehmann*
>
> Ulticom, Inc.
>
> 856-787-2952
>
>
>
> *From:* Victor Fajardo [mailto:vf0213@gmail.com]
> *Sent:* Tuesday, August 31, 2010 4:25 PM
>
> *To:* David Lehmann
> *Cc:* Glen Zorn; dime@ietf.org
> *Subject:* Re: [Dime] RFC 3588 - fixed positioned session-ID AVPs
>
>
>
> yes. the BNF sets the positioning/sequencing rules. it's a common
> practice in BNFs to place session-id near the head to optimize msg
> processing .. etc.
>
> On Tue, Aug 31, 2010 at 3:22 PM, David Lehmann <dlehmann@ulticom.com>
> wrote:
>
> So you are stating that the Diameter protocol itself does NOT require all
> session-ID AVPs to follow immediately after the message header, but a
> message’s BNF may require it?
>
>
>
> --
>
> *David Lehmann*
>
> Ulticom, Inc.
>
> 856-787-2952
>
>
>
> *From:* Victor Fajardo [mailto:vf0213@gmail.com]
> *Sent:* Tuesday, August 31, 2010 2:21 PM
>
>
> *To:* David Lehmann
> *Cc:* Glen Zorn; dime@ietf.org
> *Subject:* Re: [Dime] RFC 3588 - fixed positioned session-ID AVPs
>
>
>
>
>
> On Tue, Aug 31, 2010 at 1:54 PM, David Lehmann <dlehmann@ulticom.com>
> wrote:
>
> The existing text in 8.8 is contradicting the BNF.
>
>
>
>
>
> Which BNF though ? There maybe apps beyond 3588 that may not necessarily
> put the session-id after the header in their BNF's. In that regard, the
> existing text is not contradictory but is meant to be generalized specially
> because it describes an AVP and is agnostic to any BNF.
>
>
>
> my two cents,
>
> victor
>
>
>
>
>
>  I am suggesting text that agrees and supports the BNF.
>
>
>
> If you don’t want to modify the text to agree with the BNF, then I suggest
> removing the existing text which contradicts it.
>
>
>
> --
>
> *David Lehmann*
>
> Ulticom, Inc.
>
> 856-787-2952
>
>
>
> *From:* Victor Fajardo [mailto:vf0213@gmail.com]
> *Sent:* Tuesday, August 31, 2010 11:52 AM
> *To:* David Lehmann
> *Cc:* Glen Zorn; dime@ietf.org
> *Subject:* Re: [Dime] RFC 3588 - fixed positioned session-ID AVPs
>
>
>
> Hi David,
>
> On Tue, Aug 31, 2010 at 10:37 AM, David Lehmann <dlehmann@ulticom.com>
> wrote:
>
> Glen,
>
>
>
> That is not what the spec says.  At least, the wording is not clear and,
> IMHO, is misleading.  In which “*some* message” can the session-id AVP be
> in any position?
>
>
>
> It seems to me that the wording in section 8.8 should be:  “When present,
> the Session-Id MUST appear immediately following the Diameter Header (see
> Section 3).”
>
>
>
> As Glen has mentioned, the BNF dictates the positioning of the AVP. If you
> add this text you are adding new rules beyond the BNF.
>
>
>
> regards,
>
> victor
>
>
>
>
>
>
>
>
>
> --
>
> *David Lehmann*
>
> Ulticom, Inc.
>
> 856-787-2952
>
>
>
> *From:* Glen Zorn [mailto:gwz@net-zen.net]
> *Sent:* Tuesday, August 31, 2010 9:54 AM
> *To:* David Lehmann
> *Cc:* dime@ietf.org
> *Subject:* RE: [Dime] RFC 3588 - fixed positioned session-ID AVPs
>
>
>
> David Lehmann [mailto://dlehmann@ulticom.com] <[mailto://dlehmann@ulticom.com%5d> writes:
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> Hello,
>
> Hello.
>
> In RFC 3588 (and 3588bis), messages with session IDs are defined with the session ID AVPs with a fixed position which is immediately following the header. (e.g. section 8.3.1)
>
> Yes.
>
> However, this is in contradiction with section 8.8 which states, “When present, the Session-Id SHOULD appear immediately following the Diameter Header (see Section 3).”
>
> No.
>
> By using “SHOULD”, the spec is stating that the session-ID AVP could be in any position in the message.
>
> No, it is stating that the AVP could be in any position in *some*message.  The syntax of the existing messages in RFC 3588 is defined by the
> associated BNF and in *those* messages the Session-Id AVP *must*immediately follow the Diameter header.
>
>
>
>
>
> Hope this helps.
>
>
>
>  ~gwz
>
>
> _______________________________________________
> DiME mailing list
> DiME@ietf.org
> https://www.ietf.org/mailman/listinfo/dime
>
>
>
>
>
>
>