Re: Issue 46 (was: protocol draft status and movingforward)

 
> Mohan Parthasarathy writes:
> > Why does one have to indicate that they have
> > NO_ADDITIONAL_ADDRESSES ? If i don't send
> > ADDITIONAL_IP* notification then you just have one
> > address to operate with. What is the use of explicitly
> > indicating this ? Also, ADDITIONAL_IP with zero
> > address can do the same i guess i.e ADDITIONAL_IP*
> > replaces the previous list with zero or more
> > addresses.
> 
> Ok, lets add explination about that too:
> 
>     The NO_ADDITIONAL_ADDRESSES notify is needed, as otherwise it
>     would not be possible to distinguish empty DPD message from
>     address update message that does not have any additional
>     addresses. Now we can distinguish them, as DPD is empty
>     INFORMATIONAL exchange and address update with no extra addresses
>     is INFORMATIONAL exchange having only N(NO_ADDITIONAL_ADDRESSES).
>     Both of them can have other additional payloads like N(COOKIE2)
>     etc.  
Hmm...part of the problem is that NO_ADDITIONAL_ADDRESS is really
DELETE_ADDITIONAL_ADDRESS. But we don't want to use the word
DELETE because we provide only replace semantics. It may sound trivial, but
what exact action (delete the addresses contained in the previous ADD_ADDITIONAL_IP*)
needs to be performed is not explained anywhere (not in 05). The defintion in 4.3
contains :

   The NO_ADDITIONAL_ADDRESSES notification can be included in an
   INFORMATIONAL exchange request message to indicate that the exchange
   initiator does not have addresses beyond the one used in the exchange
   (see Section 3.6 for more detailed description).

Section 3.6 only contains "update the peer addresses ...". Perhaps the definition should
be appended with.. "The exchange initiator sends this notification to delete the addresses
sent in the earlier ADD_ADDITIONAL_ADDRESS notification".

-mohan


> -- 
> [email protected]
> _______________________________________________
> Mobike mailing list
> [email protected]
> https://www.machshav.com/mailman/listinfo.cgi/mobike