Skip to main content

Re: Public draft: comments about ForkingContext and @SipApplicationKey

  • From: binod pg < >
  • To:
  • Subject: Re: Public draft: comments about ForkingContext and @SipApplicationKey
  • Date: Tue, 11 Feb 2014 17:29:51 +0530
  • Organization: Oracle Corporation

I see what you are saying and it is a good thinking.

However, that would make @SipApplicationKey more complicated. For example,
then, the container will need to support more than one ApplicationKey methods.

May be this is something we should explore for a future release.

thanks,
Binod.

On 2/11/2014 4:48 PM, Nitzan Nissim wrote:
Looks good.

What do you think about using Method annotations and predicate together
with a @SipApplicationKey for a SipServletRequest parameterized method?
I do see how the selection logic an application would use with Predicate
might be reused both for selecting requests that will need SAS targeting
and in the handling method of the servlet....


- Nitzan






From:   Binod 
< >
To:     
,
Date:   11/02/2014 12:14 PM
Subject:        Re: Public draft: comments about ForkingContext and
             @SipApplicationKey



On Tuesday 11 February 2014 03:09 PM, Nitzan Nissim wrote:
I can't think of a scenario where application would want to reuse the key
generating methods with more parameters. If they had to do it, its
probably
possible using other method that reuse the shared logic.
This seems to add more complication and ambiguity. The text can also
sounds
like it is possible to have multiple such methods with different
signatures
and additional/different parameters, but that can't be allowed.
Thats fine. I will change it to just one parameter.

Another thing that might be confusing is when it refers to "...same type
of
request...". The meaning is that there are 2 types, HTTP and SIP, right?
So
there can be max of 2 methods annotated with @SipApplicationKey?
I think this needs to be made more clear, not to be mistaken with SIP
requests of different methods. (Maybe it is an idea though, if this
annotation could be used in convergence with the method specific
annotations....)
Would the following work?

<snip>
A method this annotation is attached to MUST be a public and static method,
MUST return a String and MUST accept the incoming request as the single
argument.

If the method annotated with @SipApplicationKey is not of this signature,
the container MUST fail deployment of the application.

If a given SIP application (SAR/WAR) contains more than one
@SipApplicationKey annotated
method with SipServletRequest as parameter, the container MUST fail the
deployment.

Similarly, if a given SIP application (SAR/WAR) contains more than one
@SipApplicationKey
annotated method with HttpServletRequest as paramater, the container
MUST fail the
deployment.

</snip>

thanks,
Binod.

Thanks,
Nitzan

_______________________________________________
Nitzan Nissim
SIP Container Architect, WebSphere SIP Infrastructure
IBM Software Group, AIM
Israel Software Lab
+972 (54) 6976107
+972 (8) 9482326







From:            binod 
pg< >
To:              
,
Date:            11/02/2014 06:03 AM
Subject:                 Re: Public draft: comments about ForkingContext and
              @SipApplicationKey



Hi Nitzan.

On 2/10/2014 10:17 PM, Nitzan Nissim wrote:
- Javadoc for ForkingContext
SipServletRequest createRequest(String method)

It is not clear what would happen if the session owning this context is
a
proxy session. IllegalStateException?
Yes, it should throw IllegalStateException. Added it in my source.

- 23.2.5 @SipApplicationKey Annotation:
"....accept the incoming request as the first argument...". I might have
missed something, but what would it mean to have other arguments there?
How
would the container assign them when calling the method?
When container calls them, they will be null or equivalent. Applications
may invoke the same method
with more arguments.

Do you think, it will be useful? If it is unnecessary complexity, we can
just move it back to
single method parameter.

- Binod.

- Nitzan








Public draft: comments about ForkingContext and @SipApplicationKey

Nitzan Nissim 02/10/2014

Re: Public draft: comments about ForkingContext and @SipApplicationKey

binod pg 02/11/2014

Re: Public draft: comments about ForkingContext and @SipApplicationKey

Nitzan Nissim 02/11/2014

Re: Public draft: comments about ForkingContext and @SipApplicationKey

Binod 02/11/2014

Re: Public draft: comments about ForkingContext and @SipApplicationKey

Nitzan Nissim 02/11/2014

Re: Public draft: comments about ForkingContext and @SipApplicationKey

binod pg 02/11/2014
 
 
Close
loading
Please Confirm
Close