RE: Gen-ART LC Review of draft-ietf-dhc-relay-id-suboption-11

RAMAKRISHNADTV <RAMAKRISHNADTV@infosys.com> Fri, 21 December 2012 12:48 UTC

Return-Path: <RAMAKRISHNADTV@infosys.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C19BA21F84FB; Fri, 21 Dec 2012 04:48:38 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id TwqIW5rQDMAu; Fri, 21 Dec 2012 04:48:37 -0800 (PST)
Received: from KECGATE08.infosys.com (kecgate08.infosysconsulting.com [122.98.10.33]) by ietfa.amsl.com (Postfix) with ESMTP id EA2D421F84F5; Fri, 21 Dec 2012 04:48:36 -0800 (PST)
X-TM-IMSS-Message-ID: <5051633600276dbe@infosys.com>
Received: from blrkechub01.ad.infosys.com ([10.66.236.41]) by infosys.com ([122.98.10.33]) with ESMTP (TREND IMSS SMTP Service 7.1) id 5051633600276dbe ; Fri, 21 Dec 2012 18:23:03 +0530
Received: from BLRKECHUB07.ad.infosys.com (10.66.236.117) by blrkechub01.ad.infosys.com (10.66.236.41) with Microsoft SMTP Server (TLS) id 8.2.176.0; Fri, 21 Dec 2012 18:18:12 +0530
Received: from BLRKECMBX23.ad.infosys.com ([fe80::d9f9:2028:fd8:1e35]) by BLRKECHUB07.ad.infosys.com ([::1]) with mapi id 14.02.0318.004; Fri, 21 Dec 2012 18:18:12 +0530
From: RAMAKRISHNADTV <RAMAKRISHNADTV@infosys.com>
To: Ben Campbell <ben@nostrum.com>, "draft-ietf-dhc-relay-id-suboption.all@tools.ietf.org" <draft-ietf-dhc-relay-id-suboption.all@tools.ietf.org>
Subject: RE: Gen-ART LC Review of draft-ietf-dhc-relay-id-suboption-11
Thread-Topic: Gen-ART LC Review of draft-ietf-dhc-relay-id-suboption-11
Thread-Index: AQHN3i35q1rCmEQ/mUyYBxRcqJiFtpgi7Klw
Date: Fri, 21 Dec 2012 12:48:13 +0000
Message-ID: <F2B120E98374B2448745C1117BDA1854238F281F@BLRKECMBX23.ad.infosys.com>
References: <BE996F07-CFB7-47F5-8B17-FA651C294FA3@nostrum.com>
In-Reply-To: <BE996F07-CFB7-47F5-8B17-FA651C294FA3@nostrum.com>
Accept-Language: en-US, en-IN
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.66.236.147]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Mailman-Approved-At: Fri, 21 Dec 2012 09:20:04 -0800
Cc: "gen-art@ietf.org Review Team" <gen-art@ietf.org>, "ietf@ietf.org List" <ietf@ietf.org>, Bharat Joshi <bharat_joshi@infosys.com>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 21 Dec 2012 12:48:38 -0000

Hi Ben,

Thank you for your review comments. 
Please find my responses inline below.

Regards,
Ramakrishna DTV.

>-----Original Message-----
>From: Ben Campbell [mailto:ben@nostrum.com]
>Sent: Thursday, December 20, 2012 2:45 AM
>To: draft-ietf-dhc-relay-id-suboption.all@tools.ietf.org
>Cc: gen-art@ietf.org Review Team; ietf@ietf.org List
>Subject: Gen-ART LC Review of draft-ietf-dhc-relay-id-suboption-11
>
>I am the assigned Gen-ART reviewer for this draft. For background on
>Gen-ART, please see the FAQ at
>
><http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>
>Please resolve these comments along with any other Last Call comments
>you may receive.
>
>Document: draft-ietf-dhc-relay-id-suboption-11
>Reviewer: Ben Campbell
>Review Date: 2012-12-19
>IETF LC End Date: 2013-01-07
>
>Summary: This draft is basically ready for publication as a proposed
>standard. However, there is one comment from a prior review that I am
>not sure whether is resolved.
>
>Major issues:
>
>None
>
>Minor issues:
>
>-- In Sean Turner's 2009 review of version 07 of the document [
>http://www.ietf.org/mail-archive/web/gen-art/current/msg04614.html ], he
>made the following comment:
>
>> In the security considerations it says look to RFC 3046 and
>> RFC 4030 for security considerations and then says SHOULD use the
>relay
>> agent authentication option from RFC 4030.  RFC 3046 is targeted at
>> network infrastructures that are "trusted and secure" and RFC 4030
>> allows the relay agent to be part of this trusted and secure network.
>> If an implementation doesn't use the relay agent authentication
>option,
>> then the relay agent can't be part of the "trusted and secure"
>network.

RFC3046 created the relay agent information option.
Relay agent information option exists only in the messages between
relay agents and DHCP servers. RFC3046 is targeted at network infrastructures
that are "trusted and secure" as far as the paths among relay agents and DHCP
servers is concerned. In many deployments, relay agents and DHCP servers 
are under a single administrative control. By careful design and engineering 
of the network, it is possible to ensure that the network infrastructure 
comprising relay agents and DHCP server is trusted and secure. To achieve that,
RFC4030 may be used but is not a MUST. If not, RFC4030 would already be a MUST 
for RFC3046 deployment. But that is not currently the case.

>>  This makes me think that the relay agent authentication option from
>> RFC 4030 ought to be a MUST not a SHOULD?
>
>I can't tell from the resulting conversation if that comment is
>addressed in the current text. Additional text has been added, but the
>SHOULD remains. I'm willing to accept it has been addressed if the
>author's say so--I only mention it to make sure it didn't fall through a
>crack.
>

We have indeed discussed about this comment and addressed it.

The following was a related comment from DHC WG Chair (Ted Lemon) 
(http://www.ietf.org/mail-archive/web/gen-art/current/msg04615.html):

	"This document makes no changes to practice that require more or less security 
	than is provided by existing relay agent options in RFC3046. Thus, the 
	security considerations in RFC3046 should be adequate."

As Ted mentioned, our draft only proposes a new sub-option for relay-agent 
option which was originally created as part of RFC3046. So, the security 
considerations for RFC3046 apply to our draft as well. RFC3046 deployments may
use RFC4030 as explained above. So, we indicated in our draft to refer to 
both RFC3046 and RFC4030. But there are no specific security issues in the 
new relay-id sub-option itself to make RFC4030 a MUST.


>Nits/editorial comments:
>
>-- section 5, last paragraph:
>
>I suggest removing the scare quotes around "stability". If there are
>concerns about whether such stability is real, it would be better to say
>that directly.
>

There is no need for these scare quotes. We will remove them.

>-- informative references:
>
>draft-ietf-dhc-dhcpv4-bulk-leasequery-06 is now 07

We will update this.

>

**************** CAUTION - Disclaimer *****************
This e-mail contains PRIVILEGED AND CONFIDENTIAL INFORMATION intended solely 
for the use of the addressee(s). If you are not the intended recipient, please 
notify the sender by e-mail and delete the original message. Further, you are not 
to copy, disclose, or distribute this e-mail or its contents to any other person and 
any such actions are unlawful. This e-mail may contain viruses. Infosys has taken 
every reasonable precaution to minimize this risk, but is not liable for any damage 
you may sustain as a result of any virus in this e-mail. You should carry out your 
own virus checks before opening the e-mail or attachment. Infosys reserves the 
right to monitor and review the content of all messages sent to or from this e-mail 
address. Messages sent to or from this e-mail address may be stored on the 
Infosys e-mail system.
***INFOSYS******** End of Disclaimer ********INFOSYS***