dect/asterisk
Archived
14
0
Fork 0
Code Releases Activity

Always relying on the responses when crossing NAT's are not a good

Browse source 
solution, it breaks communication.
Rizzo - you need to implement a configuration option for this 
code. It's good, but maybe should be off by default.


git-svn-id: http://svn.digium.com/svn/asterisk/trunk@89285 f38db490-d61c-443f-a65b-d21fe96a405b
This commit is contained in:
oej 2007-11-15 12:21:57 +00:00
parent 6a4d1a57fd
commit 529e51ce39
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
channels/chan_sip.c
View file

@ -13810,7 +13810,13 @@ static void handle_response(struct sip_pvt *p, int resp, char *rest, struct sip_
gettag(req, "To", tag, sizeof(tag));
ast_string_field_set(p, theirtag, tag);
}
check_via_response(p, req);
/* This needs to be configurable on a channel/peer/user level,
not mandatory for all communication. Sadly enough, NAT implementations
are not so stable so we can always rely on these headers.
Temporarily disabled, while waiting for fix.
Fix assigned to Rizzo :-)
*/
/* check_via_response(p, req); */
if (p->relatedpeer && p->method == SIP_OPTIONS) {
/* We don't really care what the response is, just that it replied back.
Well, as long as it's not a 100 response... since we might