RestComm / sip-servlets

Leading SIP - IMS - WebRTC Application Server
http://www.restcomm.com/
GNU Affero General Public License v3.0
236 stars 181 forks source link

SipApplication annotation question #120

Open RomanDS opened 8 years ago

RomanDS commented 8 years ago

Hello all, we have figured out how to get away from using sip.xml and relay purely on annotations. However, we dont know how to define a uri for the sip servlet.

Here are some details: We needed to add package-info.java file as part of our application: " /**

@javax.servlet.sip.annotation.SipApplication( name="dlgc_sample_demo", sessionTimeout=0, distributable=true, mainServlet="AsyncPlayer" )

package play; "

This allowed the demo to work without a need for sip.xml.

But as we used to define a URI in sip.xml we are not sure how to do that through annotations. Please advise.

Example of sip.xml used before with defined uri as "player". We are trying to mimic this using annotations:

AsyncPlayer request.method INVITE request.to.uri.user player

Log Time

deruelle commented 8 years ago

@RomanDS the spec doesn't define a standard way using annotations to define servlet-mapping AFAIK... Any Chances you can use main-servlet instead and do the selection at the app level ?

RomanDS commented 8 years ago

We thought about it but that means that App needs to have a concept of SIP Servlet dispatcher and that also means that the app will be getting all Invites, correct?

From: Jean Deruelle [mailto:notifications@github.com] Sent: Tuesday, March 8, 2016 9:08 AM To: RestComm/sip-servlets sip-servlets@noreply.github.com Cc: Roman Stobnicki Roman.Stobnicki@dialogic.com Subject: Re: [sip-servlets] SipApplication annotation question (#120)

@RomanDShttps://github.com/RomanDS the spec doesn't define a standard way using annotations to define servlet-mapping AFAIK... Any Chances you can use main-servlet instead and do the selection at the app level ?

— Reply to this email directly or view it on GitHubhttps://github.com/RestComm/sip-servlets/issues/120#issuecomment-193797167.

deruelle commented 8 years ago

that's correct.