Re: [sipcore] I-D Action: draft-ietf-sipcore-sip-push-23.txt

[Paul Kyzivat <[e-mail]>]

On 1/22/19 3:27 AM, Christer Holmberg wrote:
> Hi,
> 
> I was taking a second look in RFC 3261, and the text already says:
> 
> “o  URI uri-parameter components are compared as follows:
> 
>           -  Any uri-parameter appearing in both URIs must match.”
> 
> So, in my opinion that means that 3261 already covers the pn- 
> parameters, and the only thing we need to do in draft-sip-push is to 
> point that out.
> 
> However, draft-sip-push also allows a partial match, and that needs to 
> be normative.
> 
> So, perhaps something like:
> 
>     “When a SIP proxy compares two SIP URIs, the proxy uses the URI
> 
>     comparison rules defined in [RFC3261], meaning that in order
> 
>     for a comparison to be successful the pn-prid, pn-provider and
> 
>     pn-param SIP URI parameters must also match.”
> 
> So, a proxy will always use these rules when comparing SIP URIs, not 
> only when comparing a Contact URI with a Request-URI.
> 
> Pull request with suggested modficiation:
> 
> https://github.com/cdh4u/draft-sip-push/pull/34

I reviewed this change in -24. It resolves my concern.

	Thanks,
	Paul