Re: [dhcwg] WGLC on draft-ietf-dhc-relay-port-02

Hi:

I reviewed the 04 version.

This document could use some updates. Below are my comments. I think these should be addressed before the document advances, but otherwise I think it should be mostly ready to move forward.

1. General - I would highly recommend this draft adopt and document the "DHCP" terminology as follows:
	DHCP means DHCPv4 and DHCPv6.
	DHCPv4 means the Dynamic Host Configuration  Protocol for IPv4 as documented in RFC2131.
	DHCPv6 means the Dynamic Host Configuration Protocol for IPv6 as documented in RFC3315.
And make use of these correctly.

The document currently uses DHCP to mean DHCPv4 or both DHCPv4 and DHCPv6.

Similarly, I 'd suggest using IPv4 and IPv6 (or IP when being general and apply to both or either).

2. Abstract - As currently written I don't see why this RFC is needed. Relay source ports in 2131/3315 are meaningless and the abstract "allows any valid number" which is already the case. It needs to be document that this provides the means for the relay to receive packets from a server or (upstream) relay on any port, not just the default port.

3. Introduction (1) - The "to remember inbound" may cause some concern with regards to relay's -- since they actually do not remember anything (they are supposed to be stateless).  I wonder if this text can be cleaned up -- servers need to "remember" while processing a packet; (cascading) relay's need to add this port information to the Relay-Forw packets.

4. Section 4.2 - there are several places where "non-DHCP UDP port" is used. I wonder whether we could just say "port other than 547. I know this is in the terminology, but in this case it is DHCPv6 specific text so why make someone look it up? I guess you could use: "non-DHCP UDP Port (not 547)" or similar if you wanted to keep the reference to that term?

This same comment applies to section 5.1 and 5.2 (though in 5.1 port 67 applies). 

In the end, it may be that the definition isn't that useful and perhaps that would even be better to remove it and just use the explicit port numbers? I think this would help readability. But review and consider.

5. Section 7 (IANA). Please look at a document such as https://tools.ietf.org/html/rfc7839#section-8. To avoid confusion, we have put the URL of the tables in the IANA considerations. This practice should be followed here to that it is clear to IANA exactly which set of tables to update.

6. Section 8 (Security Considerations). This should reference the security considerations of RFC 2131 and 3315.

The text that is there is more of an operational consideration rather than what security issues this new work might expose - which may be nothing new over 2131/3315. But pointing out that firewalls need to be adjusted is a good thing.

7. Section 10 (Document Change Log). Usually an RFC Editor note is added to request that they REMOVE this material in the RFC publication process.

- Bernie

-----Original Message-----
From: dhcwg [mailto:dhcwg-bounces@ietf.org] On Behalf Of Tomek Mrugalski
Sent: Wednesday, April 05, 2017 3:26 PM
To: dhcwg <dhcwg@ietf.org>
Subject: [dhcwg] WGLC on draft-ietf-dhc-relay-port-02 - respond by Apr 26

Hi,
draft-ietf-dhc-relay-port-02 defines a small extension to the DHCPv4 and
DHCPv6 protocols that allows usage of other UDP source ports to be used by relay agents. Authors believe this document is ready. The discussions so far on the mailing list were a bit modest, but the concept is simple, the draft was presented twice (in Buenos Aires and Berlin) and enjoyed a favourable reception in the room. As such, we announce a working group last call on this document. Please review and comment.

This is a very short document. It has around 6 pages of actual text.

Many WG participants may celebrate upcoming Easter and take some days off, therefore this WGLC is a bit longer than typical two weeks.

Title: Generalized UDP Source Port for DHCP Relay
Authors: N. Shen, E. Chen
Filename: draft-ietf-dhc-relay-port-02
Pages: 10 (around 6 of actual technical text)
Date: 2017-02-28
Link: https://tools.ietf.org/html/draft-ietf-dhc-relay-port-02

Responses by April 26th are appreciated.

Thanks,
Bernie and Tomek

_______________________________________________
dhcwg mailing list
dhcwg@ietf.org
https://www.ietf.org/mailman/listinfo/dhcwg

Re: [dhcwg] WGLC on draft-ietf-dhc-relay-port-02 - respond by Apr 26