Re: [abfab] Barry Leiba's No Objection on draft-ietf-abfab-aaa-saml-13: (with COMMENT)

Alejandro Pérez Méndez <alex@um.es> Fri, 08 January 2016 09:50 UTC

Return-Path: <alex@um.es>
X-Original-To: abfab@ietfa.amsl.com
Delivered-To: abfab@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EF0FC1ACF07 for <abfab@ietfa.amsl.com>; Fri, 8 Jan 2016 01:50:54 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.911
X-Spam-Level:
X-Spam-Status: No, score=-3.911 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, MIME_8BIT_HEADER=0.3, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ms070bLdcVbd for <abfab@ietfa.amsl.com>; Fri, 8 Jan 2016 01:50:52 -0800 (PST)
Received: from xenon24.um.es (xenon24.um.es [155.54.212.164]) by ietfa.amsl.com (Postfix) with ESMTP id 61AC41ACF04 for <abfab@ietf.org>; Fri, 8 Jan 2016 01:50:52 -0800 (PST)
Received: from localhost (localhost [127.0.0.1]) by xenon24.um.es (Postfix) with ESMTP id D80F085AF for <abfab@ietf.org>; Fri, 8 Jan 2016 10:50:50 +0100 (CET)
X-Virus-Scanned: by antispam in UMU at xenon24.um.es
Received: from xenon24.um.es ([127.0.0.1]) by localhost (xenon24.um.es [127.0.0.1]) (amavisd-new, port 10024) with LMTP id yITPBe2kjRFp for <abfab@ietf.org>; Fri, 8 Jan 2016 10:50:50 +0100 (CET)
Received: from [155.54.204.2] (alex.inf.um.es [155.54.204.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: alex) by xenon24.um.es (Postfix) with ESMTPSA id C75E12AFB for <abfab@ietf.org>; Fri, 8 Jan 2016 10:50:48 +0100 (CET)
To: abfab@ietf.org
References: <20160107014638.22674.62959.idtracker@ietfa.amsl.com>
From: =?UTF-8?Q?Alejandro_P=c3=a9rez_M=c3=a9ndez?= <alex@um.es>
Message-ID: <568F8678.3070108@um.es>
Date: Fri, 8 Jan 2016 10:50:48 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1
MIME-Version: 1.0
In-Reply-To: <20160107014638.22674.62959.idtracker@ietfa.amsl.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/abfab/7U-ILSZobH-e-KVWc6GsTIvd078>
Subject: Re: [abfab] Barry Leiba's No Objection on draft-ietf-abfab-aaa-saml-13: (with COMMENT)
X-BeenThere: abfab@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Application Bridging, Federated Authentication Beyond \(the web\)" <abfab.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/abfab>, <mailto:abfab-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/abfab/>
List-Post: <mailto:abfab@ietf.org>
List-Help: <mailto:abfab-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/abfab>, <mailto:abfab-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 08 Jan 2016 09:50:55 -0000

Hi Barry, Alice,

thanks for the review. Should I generate a -14 version in order to 
address these comments or can this be done in an editing phase without a 
new version?

Regards,
Alejandro

El 07/01/16 a las 02:46, Barry Leiba escribió:
> Barry Leiba has entered the following ballot position for
> draft-ietf-abfab-aaa-saml-13: No Objection
>
> When responding, please keep the subject line intact and reply to all
> email addresses included in the To and CC lines. (Feel free to cut this
> introductory paragraph, however.)
>
>
> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
> for more information about IESG DISCUSS and COMMENT positions.
>
>
> The document, along with other ballot positions, can be found here:
> https://datatracker.ietf.org/doc/draft-ietf-abfab-aaa-saml/
>
>
>
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
>
> Because abfab-arch defines the terms "Client", "Relying Party", and
> "Identity Provider", I think abfab-arch should be a normative reference.
>
> -- Section 3 --
>
>     The RADIUS SAML binding defined in Section 4 of this document uses
>     two attributes to convey SAML assertions and protocol messages
>     respectively [OASIS.saml-core-2.0-os]
>
> Nit: "respectively" is out of place here, and should be removed.  You
> would only use "respectively" if you named the two attributes ("...uses
> two attributes, SAML-Assertion and SAML-Protocol, to convey SAML
> assertions and protocol messages, respectively.").
>
> -- Section 7.3.5 --
>
>     If issued by the Identity Provider, the Relying Party MUST process
>     the <samlp:Response> message and any enclosed assertion elements as
>     described in [OASIS.saml-core-2.0-os]
>
> "If issued" is dangling, and  makes it look like the Relying Party is
> issued by the Identity Provider.
>
> NEW
>     If a <samlp:Response> message is issued by the Identity Provider,
>     the Relying Party MUST process that message and any enclosed
>     assertion elements as described in [OASIS.saml-core-2.0-os]
> END
>
> -- Section 11.2 --
> Thank you; this section is well done.
>
>
> _______________________________________________
> abfab mailing list
> abfab@ietf.org
> https://www.ietf.org/mailman/listinfo/abfab