rtcweb-wg / rtcweb-transport

draft-ietf-rtcweb-transport
2 stars 2 forks source link

Temp/Permanent IPv6 issue #22

Closed alvestrand closed 7 years ago

alvestrand commented 8 years ago

From Magnus Westerlund in Last Call:

. Section 3.3:

When a client gathers all IPv6 addresses on a host, and both temporary addresses and permanent addresses of the same scope are present, the client SHOULD discard the permanent addresses before exposing addresses to the application or using them in ICE. This is consistent with the default policy described in [RFC6724].

If some of the temporary IPv6 addresses, but not all, are marked deprecated, the client SHOULD discard the deprecated addresses.

I haven't noticed this before, but if you have both permanent and temporary addresses, can all the temporary be marked deprecated? If that can occur, does this specification need to say something in this case which should be used, a deprecated temporary or a permanent one?

alvestrand commented 7 years ago

Adding "non-deprecated" in front of "temporary" - that seems reasonable. @gloinul OK with you?

alvestrand commented 7 years ago

From Spencer Dawkins in IESG review:

In this text

If some of the temporary IPv6 addresses, but not all, are marked deprecated, the client SHOULD discard the deprecated addresses.

I would find some explanation of why this is a SHOULD to be helpful ("if some of the addresses are deprecated, and you could discard them because you still have addresses that are not marked deprecated, why would you not discard the deprecated addresses?").

alvestrand commented 7 years ago

From Suresh Krishnan in IESG review:

In Section 3.3. Usage of temporary IPv6 addresses, the draft states

" The IPv6 default address selection specification [RFC6724] specifies that temporary addresses [RFC4941] are to be preferred over permanent addresses."

While this is technically true, this is only the seventh rule in an ordered list of eight rules. e.g. A valid permanent address would be preferred over a deprecated temporary address based on Rule 3.

If a host had only a valid permanent address and a deprecated temporary address, the mechanism specified in the draft would result in no addresses being made available, whereas the permanent address would have been an acceptable choice using RFC6724.

So, it is not entirely clear to me what the draft is trying to accomplish. It says

" However, this rule is not completely obvious in the ICE scope. This is therefore clarified as follows:"

What rule is this talking about when it says "this rule"?

Similarly, it also says

"When a client gathers all IPv6 addresses on a host..."

but it is not clear what "client" the draft is referring to.

I think if you can clarify what you want to achieve with the address selection we can work together on some replacement text.