[dmarc-ietf] Opsdir last call review of draft-ietf-dmarc-psd-08

Qin Wu via Datatracker <noreply@ietf.org> Wed, 01 April 2020 07:06 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: dmarc@ietf.org
Delivered-To: dmarc@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id AA62A3A0E55; Wed, 1 Apr 2020 00:06:54 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Qin Wu via Datatracker <noreply@ietf.org>
To: <ops-dir@ietf.org>
Cc: draft-ietf-dmarc-psd.all@ietf.org, last-call@ietf.org, dmarc@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.123.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <158572481464.30894.8037097234628362447@ietfa.amsl.com>
Reply-To: Qin Wu <bill.wu@huawei.com>
Date: Wed, 01 Apr 2020 00:06:54 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/dmarc/Hh5rdFObLgDEPCYU161icx-RIj4>
Subject: [dmarc-ietf] Opsdir last call review of draft-ietf-dmarc-psd-08
X-BeenThere: dmarc@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "Domain-based Message Authentication, Reporting, and Compliance \(DMARC\)" <dmarc.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dmarc>, <mailto:dmarc-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dmarc/>
List-Post: <mailto:dmarc@ietf.org>
List-Help: <mailto:dmarc-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dmarc>, <mailto:dmarc-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 01 Apr 2020 07:06:55 -0000

Reviewer: Qin Wu
Review result: Ready

I have reviewed this document as part of the Operational directorate's ongoing
effort to review all IETF documents being processed by the IESG.  These
comments were written with the intent of improving the operational aspects of
the IETF drafts. Comments that are not addressed in last call may be included
in AD reviews during the IESG review.  Document editors and WG chairs should
treat these comments just like any other last call comments.

This document defines DMARC (Domain-based Message Authentication, Reporting,
and Conformance) extension to enable DMARC functionality for Public Suffix
Domains. It allows operators of Public Suffix Domains (PSDs) to express policy
at the level of the PSD that covers all organizational domains that do not
explicitly publish DMARC records.

Major issue:
Not found

Minor issue:
This document provide two registries, one is update of DMARC Tag Registry with
one new element, requested from IANA, the other is DMARC PSD registry
maintained by [psddmarc.org] and following IANA registry style, I see one is
standard registry, the other is non-standard registry, it is not clear to me
non-standard registry should be discussed in this document, which introduce
confusion, if we keep both, I am wondering why section 6 still requests to add
an element to standard registry, which functionality we add is experimental,
which functionality is not.

Second related question, this draft updates informational RFC7489 with
additional components and rules, should the front page of this Experimental RFC
reflect this or not? Nits: Section 1 Somewhere subdommains is used, somewhere
sub-dommains is used, please be consistent. Section 3.5 said: "Specifically,
this is
   not a mechanism to provide feedback addresses (RUA/RUF) when an
   Organizational Domain has declined to do so.
"
Should a reference be added to RUA/RUF, RUA/RUF needs to be expanded or add
abbreviation section in the terminology section.

Section 3.2
 OLD TEXT:
"
If the 'np' tag is absent, the policy specified by the "sp" tag (if the 'sp'
tag is present) or the policy specified by the "p" tag, if the 'sp' tag is not
present, MUST be applied for non-existent subdomains. " NEW TEXT: " If the 'np'
tag is absent, the policy specified by the "sp" tag (if the 'sp' tag is
present) or the policy specified by the "p" tag( if the 'sp' tag is not present
and ‘p’tag is present), MUST be applied for non-existent subdomains. " Section
3.2 Change section 3.2 title from "3.2.  Section 6.3 General Record Format" To
3.2.  Changes in Section 6.3 "General Record Format" Similar changes can be
applicable to other places.