Re: [dhcwg] I-D Action: draft-ietf-dhc-relay-port-01.txt

["Bernie Volz (volz)" <[e-mail]>]

Hi:

I don't think the DHCPv6 Relay handing is correct.

The only relay that needs to add this is the relay that is using a non-standard port. Other relays in the chain only need to add this if they too are using a non-standard port.

Thus the text:

   Also when a DHCPv6 relay agent
   detects that the downstream relay agent relays uses a non-DHCP UDP port in
   the packet, it MUST record the port number in the Downstream UDP
   Source Port field of this option.  If this option is included to
   indicate only the local non-DHCP UDP port usage and there is no
   downstream relay agent's non-DHCP UDP port usage, the field
   Downstream UDP Source Port MUST be set to zero.

Is not appropriate. I'm not even sure how a relay agent could detect this (other than by looking in the received Relayed packet for the option).

The concept is that a relay that Relays a packet to a server (i.e., generates a Relay-Forw) should ONLY add this option to the generated Relay-Forw if IT is using a non-standard port. Otherwise no option is added (just don't add the option; not an option with 0 value).

When a server generates Relay-Reply packets from the Relay-Forw chain, it copies this option from the Relay-Forw into the Relay-Reply. And, if the "outermost" Relay-Reply has this option, it sets its destination port number to that port for the packet to be sent.

The most simple case is:

Client -> Relay at port 2000 -> Server

Relay message send to server is then:

Relay-Forw
    Option: UDP Port = 2000
    Option: Relay Message:
           <Client request message>

The server will then generate a Relay-Reply:

Relay-Reply
    Option: UDP Port = 2000
    Option: Relay Message:
           <Client response message>

And send the packet to the relay at port 2000.

The relay will then send the client message to the client.

Note: The above only includes the basic items needed to demonstrate this.


A more complex case:

Client -> Relay at port 1000 -> Relay at standard port -> Relay at port 2000 -> Server

Thus the Relay-Forw received at a server might look like:

Relay-Forw
    Option: UDP Port = 1000
    Option : Relay Message:
         Message: Relay-Forw
               Option: Relay Message:
                      Message: Relay-Forw
                             Option: UDP Port = 2000
                             Option: Relay Message:
                                    <Client request message>

The server then generates:

Relay-Reply
    Option: UDP Port = 1000
    Option : Relay Message: 
         Message: Relay-Reply
               Option: Relay Message:
                      Message: Relay-Reply
                             Option: UDP Port = 2000
                             Option: Relay Message:
                                    <Client response message>
      
And sends this to the relay at port 1000.

The relay will then process this by removing the outer most relay message hence end up with:

Relay-Reply
       Option: Relay Message:
              Message: Relay-Reply
                    Option: UDP Port = 2000
                    Option: Relay Message:
                         <Client response message>

It will then send this to the next relay at the STANDARD Port (no UDP Port in outermost message):

That relay will then process this by removing the outer most relay message and hence end up with:

Relay-Reply
           Option: UDP Port = 2000
           Option: Relay Message:
                 <Client response message>

It will then send this to the next relay at port 2000.

That final relay will send the client message to the client (at the standard port).


You also need to clean up the text as you are using "Relay Source Port Option" and "Downstream UDP Source Port"? I think having a bit more consistent naming would be useful. I think because this is where the "response" should be sent, perhaps best to the "Relay Reply Port Option" or perhaps even "Relay-Reply Port Option" (as in Relay-Reply message). But I'm less concerned about the exact name, just that more consistence exists. I also think that "port" or something simple in the option field definitions would be sufficient rather than using a long name. i.e.:

      Option-Code:  OPTION_RELAY_REPLY_PORT. 16 bits value, to be assigned by IANA.

      Option-Len:  16 bits value to be set to 2.

      Port:  16 bits value.  To be set by to the port to which
      the Reply-Reply should be sent to (by the upstream
      relay-agent or server).

Another issue is that you should give the option a name (OPTION_...) - see http://www.iana.org/assignments/dhcpv6-parameters/dhcpv6-parameters.xhtml#dhcpv6-parameters-2 as all options have such a name and this is not something IANA does -- you need to have it in your draft. (This is less important for v4, though several drafts have been doing this using too - see http://www.iana.org/assignments/bootp-dhcp-parameters/bootp-dhcp-parameters.xhtml#options).

- Bernie

-----Original Message-----
From: dhcwg [mailto:[e-mail]] On Behalf Of Naiming Shen (naiming)
Sent: Friday, January 06, 2017 5:50 PM
To: dhcwg <[e-mail]>
Cc: Enke Chen (enkechen) <[e-mail]>
Subject: Re: [dhcwg] I-D Action: draft-ietf-dhc-relay-port-01.txt


Hi,

We have updated the dhc-relay-port draft.

The main update is on the DHCPv6 Relay Agent Source Port option,
with the change to the definition of UDP port field from previously
of ‘local' UDP source port to 'downstream relay' UDP source port,
this is for the cascaded DHCPv6 relay case.

And added more requirements on server side and relay agent
handling of this DHCPv6 option.

Please review and comment.

thanks.
- Naiming

> On Jan 6, 2017, at 2:38 PM, [e-mail] wrote:
> 
> 
> A New Internet-Draft is available from the on-line Internet-Drafts directories.
> This draft is a work item of the Dynamic Host Configuration of the IETF.
> 
>        Title           : Generalized Source UDP Port of DHCP Relay
>        Authors         : Naiming Shen
>                          Enke Chen
> 	Filename        : draft-ietf-dhc-relay-port-01.txt
> 	Pages           : 8
> 	Date            : 2017-01-06
> 
> Abstract:
>   This document extends the DHCP and DHCPv6 protocols for the UDP
>   transport from relay agent to server and allows the port to be any
>   valid number on the DHCP relay system.
> 
> 
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-ietf-dhc-relay-port/
> 
> There's also a htmlized version available at:
> https://tools.ietf.org/html/draft-ietf-dhc-relay-port-01
> 
> A diff from the previous version is available at:
> https://www.ietf.org/rfcdiff?url2=draft-ietf-dhc-relay-port-01
> 
> 
> Please note that it may take a couple of minutes from the time of submission
> until the htmlized version and diff are available at tools.ietf.org.
> 
> Internet-Drafts are also available by anonymous FTP at:
> ftp://ftp.ietf.org/internet-drafts/
> 
> _______________________________________________
> I-D-Announce mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/i-d-announce
> Internet-Draft directories: http://www.ietf.org/shadow.html
> or ftp://ftp.ietf.org/ietf/1shadow-sites.txt

_______________________________________________
dhcwg mailing list
[e-mail]
https://www.ietf.org/mailman/listinfo/dhcwg