Re: [CDNi] AD review of draft-ietf-cdni-uri-signing-19

Barry Leiba <barryleiba@computer.org> Fri, 29 January 2021 14:31 UTC

Return-Path: <barryleiba@gmail.com>
X-Original-To: cdni@ietfa.amsl.com
Delivered-To: cdni@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 38DEF3A0D5D; Fri, 29 Jan 2021 06:31:52 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.401
X-Spam-Level:
X-Spam-Status: No, score=-1.401 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, FREEMAIL_FORGED_FROMDOMAIN=0.249, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=0.249, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
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 05ncIEWAuMxC; Fri, 29 Jan 2021 06:31:51 -0800 (PST)
Received: from mail-lf1-f46.google.com (mail-lf1-f46.google.com [209.85.167.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C8F1B3A0D60; Fri, 29 Jan 2021 06:31:50 -0800 (PST)
Received: by mail-lf1-f46.google.com with SMTP id e2so9122179lfj.13; Fri, 29 Jan 2021 06:31:50 -0800 (PST)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=MnUKkSt8cmeZEp+oZFLrGfTJWbua0TXmrx9OXbH0OnI=; b=egjQldAOQ4/ZNhJO70HqOzTFh+KGEY2Vu8fYoz0aHrRwtvYDujgsHcHmHJWujlEW6U qrFyvTTV005NnAWoEu/YlPox4rj6XaiKmIIvo5lDyh8m3qIyEUdvl8QZIsJ1LUWna1Ep Ub/LjUaYbLmXJJtC1f+wewWZxPb15Q2Zg/sIjgzsx5j9wM0cV+xDxFuatNGa/dGHKa2G 6ERudqeGQCG4W76hKSLQr7r3ClL0xCXnajwPl9bZ0g1362KC8l8PH/G6NtTRnL086qFA gvvAAibdS6QeOulXBYoqiwxMdCbU+Bm7i8sj+12iDiifZboDNk9MDs9j9rnKgiVHl5Vi 4xjg==
X-Gm-Message-State: AOAM533k6cgeNYlo3J44/VCelZXskhZ1SMAcpbOKQmQwggQrQvAPC3kA RQe0Xan75e2Qnf3skjwPGnaxCDE1CbchczX8uLx4zZ5Y
X-Google-Smtp-Source: ABdhPJxSgpgbKzPYcsi/ulETP/3ZGvoQIVh3FRkCce83m9CiiW73DT2dQTs5+5m9wW5VZyuq0LeC6Brej1vw0esY1+4=
X-Received: by 2002:ac2:592a:: with SMTP id v10mr2281583lfi.123.1611930707539; Fri, 29 Jan 2021 06:31:47 -0800 (PST)
MIME-Version: 1.0
References: <CALaySJJkZiRYx16cUYDtD2LU+3qAKMtMB_qY7cwkpQnWMnVMpg@mail.gmail.com> <CABF6JR3fJ8qiCwiv-5br9bepUQAsh4_x3ae8Z0oMA5SBgfa4CQ@mail.gmail.com> <CALaySJ+qHBXVsK-BPu2vySDSP+Kqa_tTLrUr4RvuuCqWZd5Tzg@mail.gmail.com>
In-Reply-To: <CALaySJ+qHBXVsK-BPu2vySDSP+Kqa_tTLrUr4RvuuCqWZd5Tzg@mail.gmail.com>
From: Barry Leiba <barryleiba@computer.org>
Date: Fri, 29 Jan 2021 09:31:36 -0500
Message-ID: <CALaySJJmeSKeX6FVtrEje_HGRNtruz3cZvV4W14gr4xzeGX3sQ@mail.gmail.com>
To: Phil Sorber <sorber@apache.org>
Cc: "draft-ietf-cdni-uri-signing.all@ietf.org" <draft-ietf-cdni-uri-signing.all@ietf.org>, "<cdni@ietf.org>" <cdni@ietf.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/cdni/oblqP3skiKk-EHU5PLf1798npq0>
Subject: Re: [CDNi] AD review of draft-ietf-cdni-uri-signing-19
X-BeenThere: cdni@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "This list is to discuss issues associated with the Interconnection of Content Delivery Networks \(CDNs\)" <cdni.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/cdni>, <mailto:cdni-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/cdni/>
List-Post: <mailto:cdni@ietf.org>
List-Help: <mailto:cdni-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/cdni>, <mailto:cdni-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 29 Jan 2021 14:31:52 -0000

Phil, please go ahead and post version -20 as soon as you can; thanks.

Barry

On Fri, Jan 22, 2021 at 10:09 AM Barry Leiba <barryleiba@computer.org> wrote:
>
> Hi, Phil, and thanks for getting this reply out.
>
> > Let's iterate there and I will submit an updated draft once you are
> > satisfied with the changes.
>
> Makes sense.
>
> > > — Section 2 —
> > >
> > >    HTTP or HTTPS URI [RFC7230] (Section 2.7).
> > >
> > > This is a quirk of the tools and not a fault in the draft, but this
> > > results in the html version having a clickable link to RFC 7230, and then
> > > another clickable link to Section 2.7 of this document (which, of course,
> > > isn’t there).  I think if you change this to “HTTP or HTTPS URI (see
> > > [RFC7230] Section 2.7)” the generated link will work better.
> >
> > I made this change but not sure if it had the intended effect. Please
> > confirm.
>
> It's a small thing, so let's not sweat it, but I can't check it until
> the new draft is posted and HTML-ized.  Let's assume it'll have the
> intended effect, and if it doesn't we'll let the RFC Editor deal with
> it.
>
> > > — Section 2.1.15 —
> > >
> > >    The URI Container (cdniuc) claim takes one of the following forms
> > >
> > > This is the only claim definition that doesn’t say “[optional]”.  Should
> > > it?
> > >
> >
> > I see it as optional in the latest published version of the draft already.
> > Was there another claim you were meaning instead?
>
> No, this was my mis-reading: cdniuc is actually defined in 2.1.11, and
> it is labelled correctly there.  It's not labelled as "[optional]"
> here in 2.1.15, but it shouldn't be.  Never mind.
>
> > > — Section 6.7 —
> > > Thanks for including advice to the experts.
> > >
> > >    Expert Reviewers should be empowered to pass judgements
> > >    as they see fit
> > >
> > > This is very vague and, to me, implies more capriciousness than I think
> > > you really intend.  Can you come up with another way to word this that
> > > gives a bit more guidance?  What sorts of judgments are they likely to make?
> > >
> >
> > This one I agree with you on, but I struggled in adding it. I made some
> > changes, but more guidance is welcome.
>
> Let's call this one resolved also; thanks.
>
> So I think your updated version should be ready to post as -20.  Please do!
>
> Barry