res_pjsip_outbound_publish: Add transport for outbound PUBLISH

The first available transport of the appropriate type is used now.
This patch adds new config option 'transport' for outbound-publish.
If transport is set then outbound PUBLISH requests will use this transport.

ASTERISK-25901 #close

Change-Id: Ib389130489b70e36795b0003fa5fd386e2680151
changes/62/2562/2
Alexei Gradinari 9 years ago committed by Mark Michelson
parent 28cefc3e88
commit 4e00e31ef1

@ -283,6 +283,11 @@ chan_pjsip
------------------ ------------------
* Added 'pjsip show channelstats' CLI command. * Added 'pjsip show channelstats' CLI command.
res_pjsip_outbound_publish
------------------
* Added support for setting the transport used on outbound publish
using the transport configuration option.
------------------------------------------------------------------------------ ------------------------------------------------------------------------------
--- Functionality changes from Asterisk 13.7.0 to Asterisk 13.8.0 ------------ --- Functionality changes from Asterisk 13.7.0 to Asterisk 13.8.0 ------------
------------------------------------------------------------------------------ ------------------------------------------------------------------------------

@ -87,6 +87,13 @@
<configOption name="max_auth_attempts" default="5"> <configOption name="max_auth_attempts" default="5">
<synopsis>Maximum number of authentication attempts before stopping the publication.</synopsis> <synopsis>Maximum number of authentication attempts before stopping the publication.</synopsis>
</configOption> </configOption>
<configOption name="transport">
<synopsis>Transport used for outbound publish</synopsis>
<description>
<note><para>A <replaceable>transport</replaceable> configured in
<literal>pjsip.conf</literal>. As with other <literal>res_pjsip</literal> modules, this will use the first available transport of the appropriate type if unconfigured.</para></note>
</description>
</configOption>
<configOption name="type"> <configOption name="type">
<synopsis>Must be of type 'outbound-publish'.</synopsis> <synopsis>Must be of type 'outbound-publish'.</synopsis>
</configOption> </configOption>
@ -117,6 +124,8 @@ struct ast_sip_outbound_publish {
AST_STRING_FIELD(from_uri); AST_STRING_FIELD(from_uri);
/*! \brief URI for the To header */ /*! \brief URI for the To header */
AST_STRING_FIELD(to_uri); AST_STRING_FIELD(to_uri);
/*! \brief Explicit transport to use for publish */
AST_STRING_FIELD(transport);
/*! \brief Outbound proxy to use */ /*! \brief Outbound proxy to use */
AST_STRING_FIELD(outbound_proxy); AST_STRING_FIELD(outbound_proxy);
/*! \brief The event type to publish */ /*! \brief The event type to publish */
@ -150,6 +159,8 @@ struct ast_sip_outbound_publish_client {
unsigned int destroy; unsigned int destroy;
/*! \brief Outbound publish information */ /*! \brief Outbound publish information */
struct ast_sip_outbound_publish *publish; struct ast_sip_outbound_publish *publish;
/*! \brief The name of the transport to be used for the publish */
char *transport_name;
}; };
/*! \brief Outbound publish state information (persists for lifetime of a publish) */ /*! \brief Outbound publish state information (persists for lifetime of a publish) */
@ -331,6 +342,12 @@ static int send_unpublish_task(void *data)
pjsip_tx_data *tdata; pjsip_tx_data *tdata;
if (pjsip_publishc_unpublish(client->client, &tdata) == PJ_SUCCESS) { if (pjsip_publishc_unpublish(client->client, &tdata) == PJ_SUCCESS) {
if (!ast_strlen_zero(client->transport_name)) {
pjsip_tpselector selector = { .type = PJSIP_TPSELECTOR_NONE, };
ast_sip_set_tpselector_from_transport_name(client->transport_name, &selector);
pjsip_tx_data_set_transport(tdata, &selector);
}
pjsip_publishc_send(client->client, tdata); pjsip_publishc_send(client->client, tdata);
} }
@ -566,6 +583,12 @@ static int sip_publish_client_service_queue(void *data)
goto fatal; goto fatal;
} }
if (!ast_strlen_zero(client->transport_name)) {
pjsip_tpselector selector = { .type = PJSIP_TPSELECTOR_NONE, };
ast_sip_set_tpselector_from_transport_name(client->transport_name, &selector);
pjsip_tx_data_set_transport(tdata, &selector);
}
status = pjsip_publishc_send(client->client, tdata); status = pjsip_publishc_send(client->client, tdata);
if (status == PJ_EBUSY) { if (status == PJ_EBUSY) {
/* We attempted to send the message but something else got there first */ /* We attempted to send the message but something else got there first */
@ -647,6 +670,7 @@ static void sip_outbound_publish_client_destroy(void *obj)
ao2_cleanup(client->datastores); ao2_cleanup(client->datastores);
ao2_cleanup(client->publish); ao2_cleanup(client->publish);
ast_free(client->transport_name);
/* if unloading the module and all objects have been unpublished /* if unloading the module and all objects have been unpublished
send the signal to finish unloading */ send the signal to finish unloading */
@ -871,6 +895,11 @@ static void sip_outbound_publish_callback(struct pjsip_publishc_cbparam *param)
if (!ast_sip_create_request_with_auth(&publish->outbound_auths, if (!ast_sip_create_request_with_auth(&publish->outbound_auths,
param->rdata, tsx->last_tx, &tdata)) { param->rdata, tsx->last_tx, &tdata)) {
if (!ast_strlen_zero(client->transport_name)) {
pjsip_tpselector selector = { .type = PJSIP_TPSELECTOR_NONE, };
ast_sip_set_tpselector_from_transport_name(client->transport_name, &selector);
pjsip_tx_data_set_transport(tdata, &selector);
}
pjsip_publishc_send(client->client, tdata); pjsip_publishc_send(client->client, tdata);
} }
client->auth_attempts++; client->auth_attempts++;
@ -1030,6 +1059,7 @@ static struct ast_sip_outbound_publish_state *sip_outbound_publish_state_alloc(
state->client->timer.user_data = state->client; state->client->timer.user_data = state->client;
state->client->timer.cb = sip_outbound_publish_timer_cb; state->client->timer.cb = sip_outbound_publish_timer_cb;
state->client->transport_name = ast_strdup(publish->transport);
state->client->publish = ao2_bump(publish); state->client->publish = ao2_bump(publish);
strcpy(state->id, id); strcpy(state->id, id);
@ -1123,6 +1153,7 @@ static int load_module(void)
ast_sorcery_object_field_register(ast_sip_get_sorcery(), "outbound-publish", "outbound_proxy", "", OPT_STRINGFIELD_T, 0, STRFLDSET(struct ast_sip_outbound_publish, outbound_proxy)); ast_sorcery_object_field_register(ast_sip_get_sorcery(), "outbound-publish", "outbound_proxy", "", OPT_STRINGFIELD_T, 0, STRFLDSET(struct ast_sip_outbound_publish, outbound_proxy));
ast_sorcery_object_field_register(ast_sip_get_sorcery(), "outbound-publish", "expiration", "3600", OPT_UINT_T, 0, FLDSET(struct ast_sip_outbound_publish, expiration)); ast_sorcery_object_field_register(ast_sip_get_sorcery(), "outbound-publish", "expiration", "3600", OPT_UINT_T, 0, FLDSET(struct ast_sip_outbound_publish, expiration));
ast_sorcery_object_field_register(ast_sip_get_sorcery(), "outbound-publish", "max_auth_attempts", "5", OPT_UINT_T, 0, FLDSET(struct ast_sip_outbound_publish, max_auth_attempts)); ast_sorcery_object_field_register(ast_sip_get_sorcery(), "outbound-publish", "max_auth_attempts", "5", OPT_UINT_T, 0, FLDSET(struct ast_sip_outbound_publish, max_auth_attempts));
ast_sorcery_object_field_register(ast_sip_get_sorcery(), "outbound-publish", "transport", "", OPT_STRINGFIELD_T, 0, STRFLDSET(struct ast_sip_outbound_publish, transport));
ast_sorcery_object_field_register_custom(ast_sip_get_sorcery(), "outbound-publish", "outbound_auth", "", outbound_auth_handler, NULL, NULL, 0, 0); ast_sorcery_object_field_register_custom(ast_sip_get_sorcery(), "outbound-publish", "outbound_auth", "", outbound_auth_handler, NULL, NULL, 0, 0);
ast_sorcery_reload_object(ast_sip_get_sorcery(), "outbound-publish"); ast_sorcery_reload_object(ast_sip_get_sorcery(), "outbound-publish");

Loading…
Cancel
Save