Status: New
Owner: ----
Labels: Component-JAIN-SIP-Javascript Type-Defect Priority-Medium JAIN-SIP-Javascript-1.0.0.Final

New issue 117 by [email protected]: Conditional braces missing after if in SipRequest.js
http://code.google.com/p/jain-sip/issues/detail?id=117

What steps will reproduce the problem?
1.Create a SIP request of type PUBLISH with Event header in a web client
2.On sending the SIP PUBLISH request from client, error appears in browser console

What is the expected output? What do you see instead?


What version of the product are you using? On what operating system?


Please provide any additional information below.

In the if condition on live 177, in SIPRequest.js file.

--------------------------------------------------
 } else if (this.getMethod()==this.PUBLISH) {
        if (this.getHeader(this.EventHeader) == null)
console.error("SIPRequest:checkHeaders(): "+prefix + this.EventHeader);
            throw "SIPRequest:checkHeaders(): "+prefix + this.EventHeader;
---------------------------------------------------

There should be braces after - if (this.getHeader(this.EventHeader) == null) such as

---------------------------------------------------

} else if (this.getMethod()==this.PUBLISH) {
        if (this.getHeader(this.EventHeader) == null){
console.error("SIPRequest:checkHeaders(): "+prefix + this.EventHeader);
            throw "SIPRequest:checkHeaders(): "+prefix + this.EventHeader;
                        }
-----------------------------------------------------

otherwise the code throws error irrespective of anything;even if the PUBLISH request does have a valid Event header.


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--

--- You received this message because you are subscribed to the Google Groups "mobicents-all-issues-changes" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to