Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSeq missmatch with UAC module #919

Closed
vinzens opened this issue Jan 9, 2017 · 16 comments
Closed

CSeq missmatch with UAC module #919

vinzens opened this issue Jan 9, 2017 · 16 comments

Comments

@vinzens
Copy link

vinzens commented Jan 9, 2017

While placing a call though kamailio with uac Module the Cseq is not updated if the call gets rejected for any reason.

Call setup is as follow:

Asterisk -> Kamailio with UAC Module -> Starface PBX

The asterisk calls a number which is not valid on Starface. in this case the Starface rejects the call. The ACK from Kamailio to the starface has wrong CSeq and the 403 gets retransmitted.

The call sequence between Asterisk and Kamailio is not the same than on the Call from Kamailio to Starface since the Proxy Auth was handled by UAC Module. Branch module should take care of the different CSeq but it does not work for rejected calls. Wth a normal Call setup (200 OK, Bye...) everything works fine.

In the attachement there is a wireshark trace with only one call (which is the affected one) and a debug log (level 3)

debug_trace_log.zip

@vinzens
Copy link
Author

vinzens commented Jan 9, 2017

The used kamailio is the actual one from master branch. We can reproduce the issue. Please let us know if we should do so with new version.

@miconda
Copy link
Member

miconda commented Jan 9, 2017

What are the parameters you set for the tm module?

@vinzens
Copy link
Author

vinzens commented Jan 10, 2017

I have set no parameter. Everything should be default.

miconda added a commit that referenced this issue Jan 10, 2017
@miconda
Copy link
Member

miconda commented Jan 10, 2017

Can you try with latest master branch?

@greggcz
Copy link

greggcz commented Jan 10, 2017

Can I use dev debian package http://deb.kamailio.org/kamailiodev-nightly or do I have to compile ?

@linuxmaniac
Copy link
Member

linuxmaniac commented Jan 10, 2017

Can I use dev debian package http://deb.kamailio.org/kamailiodev-nightly or do I have to compile ?

@greggcz if you wait for the new package to be built this night then yes

@miconda
Copy link
Member

miconda commented Jan 10, 2017

They are built over the night, so you have to do the testing tomorrow. You can check the date and time of the packages at:

@greggcz
Copy link

greggcz commented Jan 10, 2017

What prefix etc do you use to build debian binaries ? I'd like to build it from source and install in same places as the package does

@greggcz
Copy link

greggcz commented Jan 10, 2017

I compiled from source with no luck. The CSeq is still incorrect

@vinzens
Copy link
Author

vinzens commented Jan 11, 2017

Same with me. Still see wrong cseq in ACK on an 403

debug.log.zip

@miconda
Copy link
Member

miconda commented Jan 11, 2017

Can you get logs and pcap at the same time? I am interested to see if some specific headers are in the ACK. Also, add log_prefix global parameter to print $ci and $hdr(CSeq) -- it will make it easier to correlate the logs and pcap.

@greggcz
Copy link

greggcz commented Jan 11, 2017

Please see #918 I pasted ngrep log from the transaction

@miconda
Copy link
Member

miconda commented Jan 11, 2017

@greggcz - I saw the ngrep output there, but there are no logs corresponding to them. You can also do the same: retest and grab both pcap with sip singaling and syslog messages with debug=3 in kamailio.cfg (and log_prefix to print $ci and $hdr(CSeq)) and attach to your issue.

miconda added a commit that referenced this issue Jan 11, 2017
@miconda
Copy link
Member

miconda commented Jan 11, 2017

A new patch was pushed. Try again with latest master branch.

@vinzens
Copy link
Author

vinzens commented Jan 12, 2017

The fix is working in my case!

thanks a lot for your help

@miconda
Copy link
Member

miconda commented Jan 12, 2017

Thanks for testing. I am closing this one, the other issue is a slightly different scenario I am working on.

@miconda miconda closed this as completed Jan 12, 2017
miconda added a commit that referenced this issue Jan 17, 2017
…ACK and CANCEL

- related to GH #870, #918, #919

(cherry picked from commit 526ec68)
miconda added a commit that referenced this issue Jan 17, 2017
- related to GH #918, #919

(cherry picked from commit b75d4ca)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants