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

[BUG] Call Center Module creates wrong "To" Header after migration from 3.1.10 to 3.4.1 #3176

Closed
kosniaz opened this issue Sep 4, 2023 · 8 comments
Assignees
Milestone

Comments

@kosniaz
Copy link

kosniaz commented Sep 4, 2023

OpenSIPS version you are running

version: opensips 3.4.1 (x86_64/linux)
flags: STATS: On, DISABLE_NAGLE, USE_MCAST, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, HP_MALLOC, DBG_MALLOC, FAST_LOCK-ADAPTIVE_WAIT
ADAPTIVE_WAIT_LOOPS=1024, MAX_RECV_BUFFER_SIZE 262144, MAX_LISTEN 16, MAX_URI_SIZE 1024, BUF_SIZE 65535
poll method support: poll, epoll, sigio_rt, select.
git revision: 3a7a2bdfb
main.c compiled on  with gcc 8

Describe the bug
We have implemented a call center that receives calls in a specific URI "phonebanking@hostname"and then handles them with the cc_handle_call method. This works well in Opensips 3.1.10. However, upon migrating to 3.4, the call leg (towards the agent) created by b2b_logic fails to generate a proper INVITE: while the agent URI is correct in the SIP METHOD header, the <To> header is set to the intial destination URI: "phonebanking@hostname".

The two invite messages: the first with opensips-proxy v3.1.10 and the second with v3.4.1

opensips_3_1_b2b_invite

opensips_3_4_b2b_invite

To Reproduce

Start opensips 3.1.10 and 3.4.1 with the provided configurations. Make sure to replace the DB URI with your mysql DB, and to create the "phonebanking-dev" flow, with agents a1,a2,a3.

Expected behavior

Incoming calls should be bridged with available agents. Instead they are rejected.

@kosniaz kosniaz changed the title [BUG] Call Center leg wrong To Header after migration from 3.1.16 to 3.4.1 [BUG] Call Center Module creates wrong "To" Header after migration from 3.1.16 to 3.4.1 Sep 4, 2023
@kosniaz
Copy link
Author

kosniaz commented Sep 4, 2023

Uploading the configuration files:

  1. Opensips 3.4.1 (the one that doens't work) : https://pastebin.com/raw/R091AXGq
  2. Opensips 3.1.14 (current version, stable): https://pastebin.com/raw/i9LZ8wW6

@kosniaz kosniaz changed the title [BUG] Call Center Module creates wrong "To" Header after migration from 3.1.16 to 3.4.1 [BUG] Call Center Module creates wrong "To" Header after migration from 3.1.10 to 3.4.1 Sep 4, 2023
@bogdan-iancu
Copy link
Member

@kosniaz, could you provide me (off ticket) a pcap showing the 3.4 call (incoming call into CC and the call to the agent) ?

@bogdan-iancu bogdan-iancu self-assigned this Sep 19, 2023
@bogdan-iancu bogdan-iancu added this to the 3.4.2 milestone Sep 19, 2023
@kosniaz
Copy link
Author

kosniaz commented Sep 19, 2023

Thank you for your comment. I just sent you an email to your @opensips.org address, titled [BUG] Call Center Module creates wrong "To" Header after migration from 3.1.10 to 3.4.1

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

Any updates here? No progress has been made in the last 15 days, marking as stale. Will close this issue if no further updates are made in the next 30 days.

@github-actions github-actions bot added the stale label Oct 5, 2023
@bogdan-iancu
Copy link
Member

@kosniaz , as per my email reply, could you re-run the scenario for 3.4 but with log_level 4 and send me the logs for the calling (and the pcap).
Thanks

@stale stale bot removed the stale label Oct 5, 2023
@github-actions
Copy link

Any updates here? No progress has been made in the last 15 days, marking as stale. Will close this issue if no further updates are made in the next 30 days.

@github-actions github-actions bot added the stale label Oct 21, 2023
@c0louri
Copy link

c0louri commented Nov 14, 2023

@bogdan-iancu, we are terribly sorry for the delaying in providing you with the files you requested. We have sent you an email to your @opensips.org address titled "OpenSIPS issue 3176 - Logs and PCAP file you requested" .

@stale stale bot removed the stale label Nov 14, 2023
Copy link

Any updates here? No progress has been made in the last 15 days, marking as stale. Will close this issue if no further updates are made in the next 30 days.

@github-actions github-actions bot added the stale label Nov 30, 2023
bogdan-iancu added a commit that referenced this issue Dec 4, 2023
Keep the original idea (<=3.1) to have TO and RURI set to the same URI. Starting 3.2 this was accidentally changed, TO hdr being preserved from the UAS entity.
Fixes #3176

(cherry picked from commit d3e3ad1)
bogdan-iancu added a commit that referenced this issue Dec 4, 2023
Keep the original idea (<=3.1) to have TO and RURI set to the same URI. Starting 3.2 this was accidentally changed, TO hdr being preserved from the UAS entity.
Fixes #3176

(cherry picked from commit d3e3ad1)
bogdan-iancu added a commit that referenced this issue Dec 4, 2023
Keep the original idea (<=3.1) to have TO and RURI set to the same URI. Starting 3.2 this was accidentally changed, TO hdr being preserved from the UAS entity.
Fixes #3176

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

No branches or pull requests

3 participants