Re: isInternalSipURI method
- From: Nitzan Nissim <
- Subject: Re: isInternalSipURI method
- Date: Thu, 27 Feb 2014 16:39:10 +0200
I am also in favor of SIPURI.isInternal()
From: "Binod PG"
To: Eric Cheung
Date: 27/02/2014 03:44 PM
Subject: Re: isInternalSipURI method
I prefer SIPURI.isInternal as well.
I will let Keith explain why inInternal should not be in SIPURI. I might be
misrepresenting his concerns.
------ Original message ------
From: Eric Cheung
Date: 27/02/2014 19:05
Subject:Re: isInternalSipURI method
I vote for SipURI.isInternal().
I am not sure why you wrote this method does not represent the content
of the SipURI. Doesn't the value returned depend on the host, port, and
maybe maddr fields of the SipURI?
On 2/27/14 5:54 AM, Binod wrote:
> In the AR proposal, Keith has a suggestion to add a method to check
> a SIP URI is addressed to the container. After yesterday's discussion,
> we came down
> to two options for this API. Would like to know EG's preference on one
> way or other.
> 1. SipFactory.isInternalSipURI(SipURI uri) : Note that SipFactory is a
> factory class and
> all other the methods are createXXX. This method does not do a factory
> 2. SipURI.isInternal() : Note that unlike other methods in the SipURI
> this method does not
> represent the content of the SipURI.