[regext] Eric Rescorla's Discuss on draft-ietf-regext-allocation-token-09: (with DISCUSS and COMMENT)

Eric Rescorla <ekr@rtfm.com> Wed, 15 August 2018 14:02 UTC

Return-Path: <ekr@rtfm.com>
X-Original-To: regext@ietf.org
Delivered-To: regext@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id C8921130FA0; Wed, 15 Aug 2018 07:02:29 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Eric Rescorla <ekr@rtfm.com>
To: The IESG <iesg@ietf.org>
Cc: regext-chairs@ietf.org, Patrick Mevzek <patrick+ietf@deepcore.org>, pm@dotandco.com, regext@ietf.org, draft-ietf-regext-allocation-token@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.83.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <153434174981.14384.10609930535615384823.idtracker@ietfa.amsl.com>
Date: Wed, 15 Aug 2018 07:02:29 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/regext/zu5QlASFb2ylJNl1a63OKAiVNKE>
Subject: [regext] Eric Rescorla's Discuss on draft-ietf-regext-allocation-token-09: (with DISCUSS and COMMENT)
X-BeenThere: regext@ietf.org
X-Mailman-Version: 2.1.27
List-Id: Registration Protocols Extensions <regext.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/regext>, <mailto:regext-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/regext/>
List-Post: <mailto:regext@ietf.org>
List-Help: <mailto:regext-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/regext>, <mailto:regext-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 15 Aug 2018 14:02:30 -0000

Eric Rescorla has entered the following ballot position for
draft-ietf-regext-allocation-token-09: Discuss

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-regext-allocation-token/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

Rich version of this review at:
https://mozphab-ietf.devsvcdev.mozaws.net/D3061


These are bearer tokens and therefore I believe transport encryption
needs to be required in S 7, not just listed as should (which isn't
even normative in this context).


----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

S 3.2.4.
>      like [RFC5731], the command MUST contain a child
>      <allocationToken:allocationToken> element for the client to be
>      authorized to transfer and allocate the object.  The authorization
>      associated with the Allocation Token is in addition to and does not
>      replace the authorization mechanism defined for the object's
>      <transfer> request command.  If the Allocation Token is invalid or

I'm having trouble processing this statement. Can you explain in more
detail what the two access control checks are here.


S 7.
>      specifications apply to this specification as well.
>   
>      The mapping acts as a conduit for the passing of Allocation Tokens
>      between a client and a server.  The definition of the Allocation
>      Token is defined outside of this mapping.  The following are security
>      considerations in the definition and use of an Allocation Token:

Do you want to use normative language here?


S 7.
>      3.  An Allocation Token should have a limited life with some form of
>          expiry in the Allocation Token if generated by a trusted 3rd
>          third party, or with a server-side expiry if generated by the
>          server.
>      4.  An Allocation Token should use a strong random value if it is
>          based on an unsigned code.

What is an "unsigned code"?