[babel] Shepherd's review of draft-ietf-babel-hmac-02
Donald Eastlake <d3e3e3@gmail.com> Wed, 26 December 2018 04:32 UTC
Return-Path: <d3e3e3@gmail.com>
X-Original-To: babel@ietfa.amsl.com
Delivered-To: babel@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EAA94130F41; Tue, 25 Dec 2018 20:32:18 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.749
X-Spam-Level:
X-Spam-Status: No, score=-1.749 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 HfYdT3k34EIr; Tue, 25 Dec 2018 20:32:17 -0800 (PST)
Received: from mail-it1-x12d.google.com (mail-it1-x12d.google.com [IPv6:2607:f8b0:4864:20::12d]) (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 3719F130F4C; Tue, 25 Dec 2018 20:32:15 -0800 (PST)
Received: by mail-it1-x12d.google.com with SMTP id g76so19737323itg.2; Tue, 25 Dec 2018 20:32:15 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to:cc; bh=0ZFCl0RKSmuOaTgEU+x0ul1gkOanA7fxHJo2fvGRoL4=; b=qbownFSnFOv5szfTU9hnb+B4dihLcWKV6j256vThtF49wrv4V/UU7ikBALO+jH+U0B uuPO+7J3MbZsoaygwEAIZYFDUTU28yGUv1wmVXoD/bc8Bj/FttnDDEzgoKLfGpxgtBOh 8rcwYYbMKrB+0+sFeYD6KNtnEEBysatNuIqE2xzdhBoTGRMGsCdDnkvlKmRDtJSQF2aD dtxFcXRfWhz550v2oj0ajQKd+aEPKBTEzHQjMVU5xSSMul9LzITsmC0b4tRAKwlErviu E0435Tj0zOowrwtuWc7wHjCDBs0Xmf2oWQFCHr2xoByFwH72MCsm7uEwAMxB7WaFDoX7 n/hw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to:cc; bh=0ZFCl0RKSmuOaTgEU+x0ul1gkOanA7fxHJo2fvGRoL4=; b=a2HfHEhekbUqPazLxvYVQO7uxxXJcTYVPx4BV9vWpNtWzsm4rjGUTusBJzWpeKBmcQ DpzXyjfseVSvSKubT/zwNovS7LArszh+XWRvP3eQARc6ycBR4zMQpjLPNDyH2QNEkDU4 Q1J2lClnH7AI/ELHrOZEw8oacW8DvmIwzjaZ5eIa9ywQu/Hljz1hOIBaxns336GQRvjB mV1x+mWHqM5DYjhwQg4cdY3ggXiJgyv4lDqkbSsFpyEeGNbCBQ/uGeolktwQO69/pBVn TqLHfklLDr+th58/TV3WcpLIVnqHsyx5uKAEjfCAPE3rlaz5PmuU6AiHGEOLrdEgPT6T EDlw==
X-Gm-Message-State: AA+aEWbbzp5jye9HRdbEYdALGxrFpm6SYjfYdk5CzjzzJVtTuhBlMVzl QmfbjV9IR6ylFf0VdOMlGMZ4iMWlAxQqL+Z8wqkytbt6
X-Google-Smtp-Source: AFSGD/UskcbcRWC7abmL1f+eLRhExmqdpwXVaoi0ft5d22RfzxlD+pnTipkjfqfjDjuBfy3VxExtmNyKHww8shAp6FU=
X-Received: by 2002:a24:89:: with SMTP id 131mr11105529ita.105.1545798734315; Tue, 25 Dec 2018 20:32:14 -0800 (PST)
MIME-Version: 1.0
From: Donald Eastlake <d3e3e3@gmail.com>
Date: Tue, 25 Dec 2018 23:32:03 -0500
Message-ID: <CAF4+nEGhxKF0ChmLyJzYy9QimhitCvjGGiw7U3stXP3uDkyd=Q@mail.gmail.com>
To: Babel at IETF <babel@ietf.org>
Cc: babel-chairs <babel-chairs@ietf.org>, draft-ietf-babel-hmac@ietf.org
Content-Type: multipart/alternative; boundary="0000000000007407a2057de550a2"
Archived-At: <https://mailarchive.ietf.org/arch/msg/babel/qnkQJ4NZwy8etciKg3Dy3j7jzbU>
Subject: [babel] Shepherd's review of draft-ietf-babel-hmac-02
X-BeenThere: babel@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "A list for discussion of the Babel Routing Protocol." <babel.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/babel>, <mailto:babel-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/babel/>
List-Post: <mailto:babel@ietf.org>
List-Help: <mailto:babel-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/babel>, <mailto:babel-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 26 Dec 2018 04:32:19 -0000
Section 1.2, Bottom of page 3. "robust random number generator" -> "robust random number [RFC4086] generator". Could also add the RFC 4086 reference at other occurrences of "random" but reference should be included at least once. Section 1.2, Top of page 4, third bullet item. This is a little confusing to read. I assume that the point about the packet being accepted by node A is that it's a valid packet (has a good HMAC TLV). But I think the average reading will be wondering "What is this node A and why isn't it mentioned again later?" Not sure if there is a reasonable wording change that improves this... Section 5.2: The diagram is odd. I would recommend sliding the Index field and preceding vertical bar character to the left so it starts aligned with the Type. Also, the PC is fixed length, which is not shown in the figure - I think you really have to at least mention the length of the PC in this section. Perhaps best to change the figure to something like the following: 0 1 2 3 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Type | Length | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | PC | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Index... +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+- or +-----------+ | Type | 1 byte +-----------+ | Length | 1 byte +-----------+-----------+-----------+-----------+ | PC | 4 bytes +-----------+-----------+-----------+-----------+ | Index... variable +-----------+-----------+-----------+-----... While it is reasonable, for implementation convenience, that there is a maximum size for Index, perhaps for cryptographic strength, there should be a minimum length, maybe 12 bytes? Thinking some more about PC, seems like you either (1) need to say that circular arithmetic is used [rfc1982] (bad idea in my opinion), or (2) say what happens when PC hits all 1's (presumably adopt a new Index, probably a better idea). It might be useful to also state that PC is an "unsigned integer" or the like. Section 7. Should say IANA is "requested" rather than "instructed". (Or, after the number are allocated, should be re-worded to indicated that the IANA actions have been completed.) Appendix B.2. Heading should end with "-hmac-01", not "-hman-00". Thanks, Donald =============================== Donald E. Eastlake 3rd +1-508-333-2270 (cell) 1424 Pro Shop Court, Davenport, FL 33896 USA d3e3e3@gmail.com
- [babel] Shepherd's review of draft-ietf-babel-hma… Donald Eastlake
- Re: [babel] Shepherd's review of draft-ietf-babel… Juliusz Chroboczek
- Re: [babel] Shepherd's review of draft-ietf-babel… Dave Taht
- Re: [babel] Shepherd's review of draft-ietf-babel… Juliusz Chroboczek
- Re: [babel] Shepherd's review of draft-ietf-babel… Donald Eastlake
- Re: [babel] Shepherd's review of draft-ietf-babel… Gabriel Kerneis