Skip to content

Commit

Permalink
Remove RFC 3261 18.1.2 sanity check (sent-by host mismatch in Via hea…
Browse files Browse the repository at this point in the history
…der).

This allows JsSIP to interop with non very compliant RFC 3261 servers.
  • Loading branch information
ibc committed Jul 14, 2014
1 parent 91bef59 commit 1f4d36d
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions src/SanityCheck.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ var sanityCheck,
*
* Responses:
* - _rfc3261_8_1_3_3_ Multiple Via headers
* - _rfc3261_18_1_2_ sent-by mismatch
* - _rfc3261_18_3_response_ Body Content-Length
*
* All:
Expand Down Expand Up @@ -111,14 +110,6 @@ function rfc3261_8_1_3_3() {
}
}

function rfc3261_18_1_2() {
var via_host = ua.configuration.via_host;
if(message.via.host !== via_host) {
logger.warn('Via host in the response does not match UA Via host value. Dropping the response');
return false;
}
}

function rfc3261_18_3_response() {
var
len = JsSIP.Utils.str_utf8_length(message.body),
Expand Down Expand Up @@ -177,7 +168,6 @@ requests.push(rfc3261_18_3_request);
requests.push(rfc3261_8_2_2_2);

responses.push(rfc3261_8_1_3_3);
responses.push(rfc3261_18_1_2);
responses.push(rfc3261_18_3_response);

all.push(minimumHeaders);
Expand Down

0 comments on commit 1f4d36d

Please sign in to comment.