Re: [netmod] Secdir last call review of draft-ietf-netmod-syslog-model-21

"Clyde Wildes (cwildes)" <> Mon, 19 February 2018 21:02 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id B8B28124E15; Mon, 19 Feb 2018 13:02:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -14.512
X-Spam-Status: No, score=-14.512 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: (amavisd-new); dkim=pass (1024-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id JfyTvtP0Cy22; Mon, 19 Feb 2018 13:02:35 -0800 (PST)
Received: from ( []) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by (Postfix) with ESMTPS id E80EC1200B9; Mon, 19 Feb 2018 13:02:34 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple;;; l=4928; q=dns/txt; s=iport; t=1519074155; x=1520283755; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=/yCfbAmTmkbCAKl83ZW8xBGkSK+deaNnrcZfYRHn+J0=; b=ZiiXr73ozCsdv9Z2Gboji8daUev0+e2ELp1nFzwfTWbxqVDI0K8AL8zh 4tOvJEuf5xQ51moGQxWJr9gncord3RjELof+E+fvqZNR9EqOb7WZKUy+S LbJ6/Ek1XexLGbYBJZmEyEw39tYPpwwCc4iDqwGXNIMbfXpKlnNp2YOHb w=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-AV: E=Sophos;i="5.46,536,1511827200"; d="scan'208";a="72968250"
Received: from ([]) by with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Feb 2018 21:02:34 +0000
Received: from ( []) by (8.14.5/8.14.5) with ESMTP id w1JL2Yvl009232 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Mon, 19 Feb 2018 21:02:34 GMT
Received: from ( by ( with Microsoft SMTP Server (TLS) id 15.0.1320.4; Mon, 19 Feb 2018 15:02:26 -0600
Received: from ([]) by ([]) with mapi id 15.00.1320.000; Mon, 19 Feb 2018 15:02:26 -0600
From: "Clyde Wildes (cwildes)" <>
To: Yaron Sheffer <>, "" <>
CC: "" <>, "" <>, "" <>
Thread-Topic: Secdir last call review of draft-ietf-netmod-syslog-model-21
Thread-Index: AQHTqMUZsEq/edddJ0uDZKwhfsdUyqOsnO2A
Date: Mon, 19 Feb 2018 21:02:26 +0000
Message-ID: <>
References: <>
In-Reply-To: <>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: []
Content-Type: text/plain; charset="utf-8"
Content-ID: <>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <>
Subject: Re: [netmod] Secdir last call review of draft-ietf-netmod-syslog-model-21
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: NETMOD WG list <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Mon, 19 Feb 2018 21:02:37 -0000


Thanks for your review. My answers are inline as [clw1].

On 2/18/18, 6:31 AM, "Yaron Sheffer" <> wrote:

    Reviewer: Yaron Sheffer
    Review result: Has Issues
    General Comments
    * The semantics of pattern matching is not clear: "and/or the message text" -
    are there cases where you only match the text but not the facility/severity? *
[clw1] Yes. There are three cases: 1. Match on facility/severity; 2. Match on the regex pattern; 3. Match on both facility/severity and the regex pattern.

    It's very confusing to specify rollover in minutes, but retention in hours.
    People are bound to get this one wrong.

[clw1] I will change the retention to minutes unless others object.

    * Interface selection: the feature
    makes sense, but I think the description is incorrect. "This leaf sets the
    source interface to be used to send messages to the remote syslog server. If
    not set, messages sent to a remote syslog server will contain the IP address of
    the interface the syslog message uses to exit the network element". AFAIK the
    source IP will always correspond to the interface, but this feature allows you
    to select a particular one.

[clw1] You are correct. I will modify the description to make this clearer. How about:

"This leaf sets the source interface to be used to send messages to the remote syslog server. If
not set, messages can be sent on any interface."

    * Usage examples: the second example lists a
    specific IPv6 address, but the Yang snippet shows a domain name. 

[clw1] Thanks for catching this error. I will fix this in the next revision.

    * A generic
    question (I am new to the Yang ecosystem): I understand most implementers will
    use this module from
    - is this the expectation? If so, why not add a link from the RFC into the
    repo, to make it easier for people to find?

[clw1] It is standard practice to include the model in the RFC AFAIK. I have not seen github links published in any other RFCs.
    Security Comments
    * I think almost all writable data nodes here are sensitive, because a network
    attacker's first move is to block any logging on the host, and many of the data
    nodes here can be used for this purpose.

[clw1] I will reword the security section to include all writeable nodes as sensitive.

    * Re: readable data nodes, I'm not
    sure which are sensitive, and the document should give an example or two rather
    than just say "some". Otherwise the security advice is not actionable. One
    example: "remote" sections leak information about other hosts in the network.

[clw1] This text was lifted from another model. I will review the readable nodes and update.

    * Write operations... can have a negative effect on network operations. - I would
    add "and on network security", because logs are often used to detect security

[clw1] I will add this phrase.

    * Also add an advice, similar to the one on "pattern match", that the
    private key used for signing log messages MUST NOT be used for any other
    purpose, and that the implementation of this data node must ensure this
    property (I'm not sure how). The rationale: if the TLS private key is used, for
    example, this could result in a signing oracle for TLS and eventually a MITM

[clw1] I will add this advice.