[Last-Call] Artart last call review of draft-billon-expires-06

Barry Leiba via Datatracker <noreply@ietf.org> Tue, 29 November 2022 15:58 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: last-call@ietf.org
Delivered-To: last-call@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id C8860C1522A2; Tue, 29 Nov 2022 07:58:27 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Barry Leiba via Datatracker <noreply@ietf.org>
To: art@ietf.org
Cc: draft-billon-expires.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 9.1.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <166973750780.46991.17035913353233818933@ietfa.amsl.com>
Reply-To: Barry Leiba <barryleiba@computer.org>
Date: Tue, 29 Nov 2022 07:58:27 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/last-call/J1Zqxjal9xkR5KNbBocNyOfW56U>
Subject: [Last-Call] Artart last call review of draft-billon-expires-06
X-BeenThere: last-call@ietf.org
X-Mailman-Version: 2.1.39
List-Id: IETF Last Calls <last-call.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/last-call>, <mailto:last-call-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/last-call/>
List-Post: <mailto:last-call@ietf.org>
List-Help: <mailto:last-call-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/last-call>, <mailto:last-call-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 29 Nov 2022 15:58:27 -0000

Reviewer: Barry Leiba
Review result: Ready with Nits

Thanks for this; I think it's useful to expand the use of "Expires" beyond just
am X.400 mapping field.

My main comment is that the Introduction is jarringly terse, leading me to
derive the content of Section 6 on my own before I even read further (and,
thus, before I found and read Section 6).  I strongly recommend reorganizing
this by moving the content of Section 6 into Section 1 and re-wording it a bit
to make it a proper introduction.  Something like:

   1. Introduction

   [RFC2156] defines a mapping of header fields between X.400 and
   RFC822/MIME.  One of the mapped fields is the “Expires” header
   field, which provides a date and time at which a message is
   considered to lose its validity.

   This document extends the use of the “Expires” header field to
   Internet email in general, whether the message comes from an X.400
   gateway or elsewhere.

   The date and time of expiration can be used by the mailbox provider
   or the MUA to indicate to the user that certain messages could be
   deleted, in an attempt to unclutter the user's mailbox and spare
   storage resources.

   Netnews articles [RFC5536] have an Expires header with a similar,
   slightly more strict syntax and similar meaning.

Note that I don't think it's necessary to mention RFC 4021 at all.

I would also change the citation to 4021 in Section 2:
OLD
   The header field definition and syntax remain the same as in
   [RFC4021], the time at which a message loses its validity.
NEW
   The header field definition and syntax remain the same as in
   [RFC2156] Sections 2.3.1.2 and 5.3.4: the time at which a
   message loses its validity.

...and then remove the informative reference to 4021.

There's a typo in Section 5:
OLD
   and could allow users to control over the actions to take
NEW
   and could allow users control over the actions to take