Re: [Sframe] Intdir telechat review of draft-ietf-sframe-enc-07

Richard Barnes <rlb@ipv.sx> Wed, 03 April 2024 12:41 UTC

Return-Path: <rlb@ipv.sx>
X-Original-To: sframe@ietfa.amsl.com
Delivered-To: sframe@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 72812C14F684 for <sframe@ietfa.amsl.com>; Wed, 3 Apr 2024 05:41:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.896
X-Spam-Level:
X-Spam-Status: No, score=-1.896 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=ipv-sx.20230601.gappssmtp.com
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id LOXeJV4QkZZR for <sframe@ietfa.amsl.com>; Wed, 3 Apr 2024 05:41:42 -0700 (PDT)
Received: from mail-il1-x12d.google.com (mail-il1-x12d.google.com [IPv6:2607:f8b0:4864:20::12d]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C9189C151063 for <sframe@ietf.org>; Wed, 3 Apr 2024 05:41:42 -0700 (PDT)
Received: by mail-il1-x12d.google.com with SMTP id e9e14a558f8ab-369f0e324adso2258325ab.3 for <sframe@ietf.org>; Wed, 03 Apr 2024 05:41:42 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ipv-sx.20230601.gappssmtp.com; s=20230601; t=1712148101; x=1712752901; darn=ietf.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=1m6oHKya8ZmAVeqYMFGF+NZhL7FPsK+bHKruMm2Vd2M=; b=wqyhvywAJbxoUrvDeEUtTicyydMLFUA8fQCogi6Z7OowkBKiPN5AlA7yzuerNxM0Jr X66JKbYbT36C0Yv/Hx49i/UI+P62msu+iZESkaVp8+doIZA9WIVn81P89Vu8Hm+DVwBP tieBwEN3aL2fY7kXTAEcVzTHgv9OYXwlO5Kly8SKbxMjBVGly72icGHIapwehK3x2T4x XWMMWzS68u5GQanJLO1DtAsJaPrzK724aPo8Te57qsw9VJJ/sTxilePZxuDWn+eQ9WvZ upTa82wHZcSFB6UzIb3ZDcqqqSaSZvm1sa3KiOohENQO1f5/OGPcD9l5S1uTtOVWH3B7 QLJA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712148101; x=1712752901; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=1m6oHKya8ZmAVeqYMFGF+NZhL7FPsK+bHKruMm2Vd2M=; b=tlzfkzNSl5+u8/Drlq6xCmMG0X57vb3sb6U0wWscU9oMnLou/sSSqM850Jdujim97U H+bsBQ1EgtW9dLk2lNHUKOl4N3WotrHpvcVK5AoMqV+LolIs8GftT2xjZ7SAu7uXX+9Z tKCke68MBRE2Tfs4SRR7EtWAiaTdV3R3feuwYpwoTOdg+X5e3a7OZgrbsJFajxsjYUqp NxrxTaV/mg61InJ8aRagrtb3PmAT6vbXrF6VxqAZIziiWPzij4z5kqOzjzqpbNxpeWKX B1MFoA5WtSxsuP6YVE4KKVj4lrwxVqp7z1SMpfxQNiczVMG1RCUWfhLYFJ6jsJ3E8pE7 okjA==
X-Forwarded-Encrypted: i=1; AJvYcCVdc8hwDDk680FRKsTY8VrQnP1ITlAk4AqbQIEFl2GckdpJfaENkq5UPnJ3JicBUbTRmfvlctccgOX91IAx+fw=
X-Gm-Message-State: AOJu0YzohugGvGSg2qeUsHFFz0YH/SIL0X5kGf5ozWTN78xbjZW2+GpB VucBZ5+5qT0oqxe904gV56A+uFY5KCX6fh1z3EGKrOPJA9+bD7ZbCj8FgUtrnNt+BMNP2d6DE27 QvuFwg9tdvnLf1lrMIc86kcszyLgkt0vcfGOR0Q==
X-Google-Smtp-Source: AGHT+IELdPlK8vOLOs3/lpQEV3UBYGZxr1/aWrBXvj5qLIWn8KGUm96nmsssPQQO1n/sU9+qCOWG5/tcho+9Rx9MtL0=
X-Received: by 2002:a05:6e02:17cb:b0:366:b889:4a90 with SMTP id z11-20020a056e0217cb00b00366b8894a90mr20261819ilu.4.1712148101640; Wed, 03 Apr 2024 05:41:41 -0700 (PDT)
MIME-Version: 1.0
References: <171172625624.49272.18301693085107891676@ietfa.amsl.com>
In-Reply-To: <171172625624.49272.18301693085107891676@ietfa.amsl.com>
From: Richard Barnes <rlb@ipv.sx>
Date: Wed, 03 Apr 2024 08:41:30 -0400
Message-ID: <CAL02cgS6OibVg+dB=pznERprwpJcPfLuwF_FU9obkkqXZGH+Fg@mail.gmail.com>
To: Suresh Krishnan <sureshk@cisco.com>
Cc: int-dir@ietf.org, draft-ietf-sframe-enc.all@ietf.org, last-call@ietf.org, sframe@ietf.org
Content-Type: multipart/alternative; boundary="00000000000066d1b60615308b37"
Archived-At: <https://mailarchive.ietf.org/arch/msg/sframe/xWHmzUaoEtyPbXzi06CLk0lTyRo>
Subject: Re: [Sframe] Intdir telechat review of draft-ietf-sframe-enc-07
X-BeenThere: sframe@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Secure Media Frames <sframe.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sframe>, <mailto:sframe-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sframe/>
List-Post: <mailto:sframe@ietf.org>
List-Help: <mailto:sframe-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sframe>, <mailto:sframe-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 03 Apr 2024 12:41:43 -0000

Hi Suresh,

Thanks for the review.  I filed a PR:
https://github.com/sframe-wg/sframe/pull/195

--Richard

On Fri, Mar 29, 2024 at 11:30 AM Suresh Krishnan via Datatracker <
noreply@ietf.org> wrote:

> Reviewer: Suresh Krishnan
> Review result: Ready with Nits
>
> I found the document well written and easy to understand. I did have two
> minor
> comments to bring up:
>
> * The draft seems to have a few (looks like Python) code blocks embedded
> in it.
> Consider enclosing them within <CODE BEGINS> and <CODE ENDS> markers to
> identify them as code components for licensing purposes.
>
> * The draft seems to refer to an obsoleted version of the SDP spec RFC4566
> which was obsoleted by RFC8866. Unless there is something specific in the
> old
> SDP needed for this draft, it probably makes sense to refer to the newer
> RFC.
>
>
>