Работа поля

[megatron_dam]

Коллеги, доброго времени суток.
Использую CentOS 7 и Asterisk 1.8.32.3 без GUI.
Вход работает, а вот исход отваливается

[virus_net]

Не зачем съезжать. Зайти и если надо, то попробовать поправить собственными ручками.
В chan_sip.c есть функция initreqprep, собственно в ней то и обрабатывается usereqphone и там же добавляют user=phone.
В функции есть коммент:

Код: Выделить всё

                const char *s = p->username;    /* being a string field, cannot be NULL */

                /* Test p->username against allowed characters in AST_DIGIT_ANY
                        If it matches the allowed characters list, then sipuser = ";user=phone"
                        If not, then sipuser = ""
                */

[megatron_dam]

Спасибо за совет.
Просто не особо очевидно как его безболезненно поправить и можно ли это вообще делать(
phone

[xelas]

Может быть надо было исправить не имя пира а fromuser/defaultuser?

[megatron_dam]

Может быть надо было исправить не имя пира а fromuser/defaultuser?

Пир без регистрации, коллега(
Только по IP.

[xelas]

И что?
fromuser прописывается в настройках пира и к регистрации отношения не имеет.

Added after 3 minutes:

А при ближайшем рассмотрении находим еще и такую опцию в конфигурации пира:

usereqphone = yes|no : Indicates whether to add a ";user=phone" to the URI. Default no.

[megatron_dam]

И что?
fromuser прописывается в настройках пира и к регистрации отношения не имеет.

Added after 3 minutes:

А при ближайшем рассмотрении находим еще и такую опцию в конфигурации пира:

usereqphone = yes|no : Indicates whether to add a ";user=phone" to the URI. Default no.

[xelas]

Кроме как поправить руками код -- никак.
Судя по форуму и баг трекеру Астериска эта проблема возникает с определенной периодичностью во всех ветках и только с определенным софтом на конце оператора.
Так что, либо менять оператора, либо одно из двух.
В любом случае, для версии 1.8 вы не найдете решения. EOL.

https://issues.asterisk.org/jira/browse ... 3Dphone%22

https://community.asterisk.org/t/sip-us ... er/44177/3

[virus_net]

в случае, если пир состоит из цифр

С чего вы решили, что в p->username содержится название пира ? Мне вот так не кажется. Сделайте print содержимого этой переменной и тогда точно поймете что там.

можно ли это вообще делать(

Ну а почему нет то ? Если есть исходный код, то его можно править

как его безболезненно поправить

делаете виртуальную машину, на ней все обкатываете, а затем переносите на боевую систему. Либо ищите человека, который за денежку поправит за вас.

Так что, либо менять оператора, либо одно из двух.

+1

[megatron_dam]

Нашел вот это, коллеги!
Копи-паст оригинала.

[xelas]

Символы - и + означают, что строка удаляется и добавляется.
Между 11 и 1.8 версиями очень много времени прошло. И данный кусок кода может не соответствовать тому, что вы увидите в реальности.

[awsswa]

что мешает перейти на 11 ветку и патч накатить ?

[virus_net]

xelas прав и патч для 11-ой в прямую не подойдет для 1.8, т.к. там немного не так.
awsswa, зачем переходить, если легко можно 1.8 допилить.
megatron_dam, вы выложили результат diff

Для 1.8 будет:

Код: Выделить всё

[work:/usr/ports/net/asterisk/work/asterisk-1.8.23.1/channels/] diff -uN chan_sip.c.orig chan_sip.c

Код: Выделить всё

--- chan_sip.c.orig     2013-08-27 23:23:25.000000000 +0400
+++ chan_sip.c      2016-07-31 09:46:52.000000000 +0300
@@ -12527,9 +12527,9 @@

        ourport = (p->fromdomainport) ? p->fromdomainport : ast_sockaddr_port(&p->ourip);
        if (!sip_standard_port(p->socket.type, ourport)) {
-               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s&#58;%d>;tag=%s", n, tmp_l, d, ourport, p->tag&#41;;
+               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s&#58;%d%s>;tag=%s", n, tmp_l, d, ourport, urioptions, p->tag&#41;;
        &#125; else &#123;
-               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s>;tag=%s", n, tmp_l, d, p->tag&#41;;
+               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s%s>;tag=%s", n, tmp_l, d, urioptions, p->tag&#41;;
        &#125;

        if &#40;!ast_strlen_zero&#40;explicit_uri&#41;&#41; &#123;

Всего то надо добавить %s и urioptions.

Ну или вообще, кардинальный подход:

Код: Выделить всё

--- chan_sip.c.orig     2013-08-27 23:23:25.000000000 +0400
+++ chan_sip.c      2016-07-31 09:46:52.000000000 +0300
@@ -12527,9 +12527,9 @@

        ourport = (p->fromdomainport) ? p->fromdomainport : ast_sockaddr_port(&p->ourip);
        if (!sip_standard_port(p->socket.type, ourport)) {
-               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s&#58;%d>;tag=%s", n, tmp_l, d, ourport, p->tag&#41;;
+               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s&#58;%d;user=phone>;tag=%s", n, tmp_l, d, ourport, p->tag&#41;;
        &#125; else &#123;
-               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s>;tag=%s", n, tmp_l, d, p->tag&#41;;
+               snprintf&#40;from, sizeof&#40;from&#41;, "\"%s\" <sip&#58;%s@%s;user=phone>;tag=%s", n, tmp_l, d, p->tag&#41;;
        &#125;

        if &#40;!ast_strlen_zero&#40;explicit_uri&#41;&#41; &#123;

[megatron_dam]

Коллеги, всем большое спасибо.
То ли я что-т не так делаю, то ли еще что.

[virus_net]

Изменил файлик

Ну а asterisk то вы пересобрали после этого ?

[megatron_dam]

Да, как положено я написал об этом...
./configure и т.д.

[virus_net]

Применил патч, пересобрал, перезапустил.
Проверяю:

INVITE sip:123@127.0.0.1 SIP/2.0
Via: SIP/2.0/UDP 127.0.0.1:5060;branch=z9hG4bK3b21679d;rport
Max-Forwards: 70
From: "asterisk" <sip:asterisk@test.mega-net.ru;user=phone>;tag=as341eb14a
To: <sip:123@127.0.0.1>
Contact: <sip:asterisk@127.0.0.1:5060>
Call-ID: 43fd9cca0968754f6ad68f2a228ecae4@test.mega-net.ru
CSeq: 102 INVITE
User-Agent: test.mega-net.ru
Date: Tue, 02 Aug 2016 06:12:50 GMT
Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH
Supported: replaces, timer
Content-Type: application/sdp
Content-Length: 259

Что я делаю не так ? А точнее все же это вы что-то делаете не так.

[megatron_dam]

Спасибо за фидбэк. Попробую еще раз. Что-то "паранормал" какой-то(
А вы какой "вариант" использовали? Тот, что радикальнее?))

Added after 1 hours:

Спасибо за фидбэк. Попробую еще раз. Что-то "паранормал" какой-то(
А вы какой "вариант" использовали? Тот, что радикальнее?))

Всем большое спасибо за посильную помощь, коллеги.
Все завелось. В заголовках инвайтов необходимые поля помечены нужными "тэгами".
Только не понятно, почему сразу не завелось...только после второй пересборки(
Надеюсь, что наши происки будут кому-то еще полезными!

[virus_net]

А вы какой "вариант" использовали?

Я использовал тот вариант, который вы указали в этом своем посте, т.е. радикальный. Если бы вы делали менее радикальный, то и я бы тоже тестил его же. Иначе какой смысл пробовать разные варианты и сравнивать полученные результаты ?

не понятно, почему сразу не завелось...

Это можно осознать только в одном случае: в точности воспроизведя порядок совершенных действий и результатов проверок после этих действий.
Например самое простое для проверки это смотреть изменился ли размер файла модуля chan_sip.so после ваших действий по пересборке или нет. Если не изменился, то вывод сразу очевиден и дальнейшие тесты бессмысленны.

[Oy90]

Патч для 11.23:

Код: Выделить всё

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index c35091f..2362617 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -13921,9 +13921,9 @@ static void initreqprep(struct sip_request *req, struct sip_pvt *p, int sipmetho
        ourport = (p->fromdomainport && (p->fromdomainport != STANDARD_SIP_PORT)) ? p->fromdomainport : ast_sockaddr_port(&p->ourip);
 
        if (!sip_standard_port(p->socket.type, ourport)) {
-               ret = ast_str_set&#40;&from, 0, "<sip&#58;%s@%s&#58;%d>;tag=%s", tmp_l, d, ourport, p->tag&#41;;
+               ret = ast_str_set&#40;&from, 0, "<sip&#58;%s@%s&#58;%d%s>;tag=%s", tmp_l, d, ourport, urioptions, p->tag&#41;;
        &#125; else &#123;
-               ret = ast_str_set&#40;&from, 0, "<sip&#58;%s@%s>;tag=%s", tmp_l, d, p->tag&#41;;
+               ret = ast_str_set&#40;&from, 0, "<sip&#58;%s@%s%s>;tag=%s", tmp_l, d, urioptions, p->tag&#41;;
        &#125;
        if &#40;ret == AST_DYNSTR_BUILD_FAILED&#41; &#123;
                /* We don't have an escape path from here... */