res_pjsip_outbound_registration: Subscribe to network change events

Outbound registration now subscribes to network change events
published by res_stun_monitor and refreshes all registrations
when an event happens.

The 'pjsip send (un)register' CLI commands were updated to accept
'*all' as an argument to operate on all registrations.

The 'PJSIP(Un)Register' AMI commands were also updated to
accept '*all'.

ASTERISK-26808 #close

Change-Id: Iad58a9e0aa5d340477fca200bf293187a6ca5a25
pull/7/head
George Joseph 8 years ago
parent df22d297a6
commit 22242fef5d

@ -88,6 +88,14 @@ app_record
* Added new 'u' option to Record() application which prevents Asterisk from * Added new 'u' option to Record() application which prevents Asterisk from
truncating silence from the end of recorded files. truncating silence from the end of recorded files.
res_pjsip_outbound_registration
------------------
* Outbound registrations are now refreshed when res_stun_monitor detects
a network change event has happened.
The 'pjsip send (un)register' CLI commands were updated to accept '*all'
as an argument to operate on all registrations.
The 'PJSIP(Un)Register' AMI commands were updated to also accept '*all'.
app_voicemail app_voicemail
------------------ ------------------
* The 'Comedian Mail' prompts can now be overriden using the 'vm-login' and * The 'Comedian Mail' prompts can now be overriden using the 'vm-login' and

@ -180,12 +180,12 @@
<syntax> <syntax>
<xi:include xpointer="xpointer(/docs/manager[@name='Login']/syntax/parameter[@name='ActionID'])" /> <xi:include xpointer="xpointer(/docs/manager[@name='Login']/syntax/parameter[@name='ActionID'])" />
<parameter name="Registration" required="true"> <parameter name="Registration" required="true">
<para>The outbound registration to unregister.</para> <para>The outbound registration to unregister or '*all' to unregister them all.</para>
</parameter> </parameter>
</syntax> </syntax>
<description> <description>
<para> <para>
Unregisters the specified outbound registration and stops future registration attempts. Unregisters the specified (or all) outbound registration(s) and stops future registration attempts.
Call PJSIPRegister to start registration and schedule re-registrations according to configuration. Call PJSIPRegister to start registration and schedule re-registrations according to configuration.
</para> </para>
</description> </description>
@ -197,14 +197,13 @@
<syntax> <syntax>
<xi:include xpointer="xpointer(/docs/manager[@name='Login']/syntax/parameter[@name='ActionID'])" /> <xi:include xpointer="xpointer(/docs/manager[@name='Login']/syntax/parameter[@name='ActionID'])" />
<parameter name="Registration" required="true"> <parameter name="Registration" required="true">
<para>The outbound registration to register.</para> <para>The outbound registration to register or '*all' to register them all.</para>
</parameter> </parameter>
</syntax> </syntax>
<description> <description>
<para> <para>
Unregisters the specified outbound registration then starts registration and schedules re-registrations Unregisters the specified (or all) outbound registration(s) then starts registration and schedules re-registrations
according to configuration. according to configuration.
future registrations.
</para> </para>
</description> </description>
</manager> </manager>
@ -379,6 +378,9 @@ static struct ast_serializer_shutdown_group *shutdown_group;
#define DEFAULT_STATE_BUCKETS 53 #define DEFAULT_STATE_BUCKETS 53
static AO2_GLOBAL_OBJ_STATIC(current_states); static AO2_GLOBAL_OBJ_STATIC(current_states);
/*! subscription id for network change events */
static struct stasis_subscription *network_change_sub;
/*! \brief hashing function for state objects */ /*! \brief hashing function for state objects */
static int registration_state_hash(const void *obj, const int flags) static int registration_state_hash(const void *obj, const int flags)
{ {
@ -1458,6 +1460,26 @@ static int queue_register(struct sip_outbound_registration_state *state)
return 0; return 0;
} }
static void unregister_all(void)
{
struct ao2_container *states;
states = ao2_global_obj_ref(current_states);
if (!states) {
return;
}
/* Clean out all the states and let sorcery handle recreating the registrations */
ao2_callback(states, OBJ_UNLINK | OBJ_NODATA | OBJ_MULTIPLE, NULL, NULL);
ao2_ref(states, -1);
}
static void reregister_all(void)
{
unregister_all();
ast_sorcery_load_object(ast_sip_get_sorcery(), "registration");
}
static char *cli_complete_registration(const char *line, const char *word, static char *cli_complete_registration(const char *line, const char *word,
int pos, int state) int pos, int state)
{ {
@ -1473,6 +1495,10 @@ static char *cli_complete_registration(const char *line, const char *word,
} }
wordlen = strlen(word); wordlen = strlen(word);
if (wordlen == 0 && ++which > state) {
return ast_strdup("*all");
}
registrations = ast_sorcery_retrieve_by_fields(ast_sip_get_sorcery(), "registration", registrations = ast_sorcery_retrieve_by_fields(ast_sip_get_sorcery(), "registration",
AST_RETRIEVE_FLAG_MULTIPLE | AST_RETRIEVE_FLAG_ALL, NULL); AST_RETRIEVE_FLAG_MULTIPLE | AST_RETRIEVE_FLAG_ALL, NULL);
if (!registrations) { if (!registrations) {
@ -1507,8 +1533,9 @@ static char *cli_unregister(struct ast_cli_entry *e, int cmd, struct ast_cli_arg
case CLI_INIT: case CLI_INIT:
e->command = "pjsip send unregister"; e->command = "pjsip send unregister";
e->usage = e->usage =
"Usage: pjsip send unregister <registration>\n" "Usage: pjsip send unregister <registration> | *all\n"
" Unregisters the specified outbound registration and stops future registration attempts.\n"; " Unregisters the specified (or all) outbound registration(s) "
"and stops future registration attempts.\n";
return NULL; return NULL;
case CLI_GENERATE: case CLI_GENERATE:
return cli_complete_registration(a->line, a->word, a->pos, a->n); return cli_complete_registration(a->line, a->word, a->pos, a->n);
@ -1520,6 +1547,12 @@ static char *cli_unregister(struct ast_cli_entry *e, int cmd, struct ast_cli_arg
registration_name = a->argv[3]; registration_name = a->argv[3];
if (strcmp(registration_name, "*all") == 0) {
unregister_all();
ast_cli(a->fd, "Unregister all queued\n");
return CLI_SUCCESS;
}
state = get_state(registration_name); state = get_state(registration_name);
if (!state) { if (!state) {
ast_cli(a->fd, "Unable to retrieve registration %s\n", registration_name); ast_cli(a->fd, "Unable to retrieve registration %s\n", registration_name);
@ -1543,9 +1576,9 @@ static char *cli_register(struct ast_cli_entry *e, int cmd, struct ast_cli_args
case CLI_INIT: case CLI_INIT:
e->command = "pjsip send register"; e->command = "pjsip send register";
e->usage = e->usage =
"Usage: pjsip send register <registration>\n" "Usage: pjsip send register <registration> | *all \n"
" Unregisters the specified outbound " " Unregisters the specified (or all) outbound "
"registration then re-registers and re-schedules it.\n"; "registration(s) then starts registration(s) and schedules re-registrations.\n";
return NULL; return NULL;
case CLI_GENERATE: case CLI_GENERATE:
return cli_complete_registration(a->line, a->word, a->pos, a->n); return cli_complete_registration(a->line, a->word, a->pos, a->n);
@ -1557,6 +1590,12 @@ static char *cli_register(struct ast_cli_entry *e, int cmd, struct ast_cli_args
registration_name = a->argv[3]; registration_name = a->argv[3];
if (strcmp(registration_name, "*all") == 0) {
reregister_all();
ast_cli(a->fd, "Re-register all queued\n");
return CLI_SUCCESS;
}
state = get_state(registration_name); state = get_state(registration_name);
if (!state) { if (!state) {
ast_cli(a->fd, "Unable to retrieve registration %s\n", registration_name); ast_cli(a->fd, "Unable to retrieve registration %s\n", registration_name);
@ -1586,6 +1625,12 @@ static int ami_unregister(struct mansession *s, const struct message *m)
return 0; return 0;
} }
if (strcmp(registration_name, "*all") == 0) {
unregister_all();
astman_send_ack(s, m, "Unregistrations queued.");
return 0;
}
state = get_state(registration_name); state = get_state(registration_name);
if (!state) { if (!state) {
astman_send_error(s, m, "Unable to retrieve registration entry\n"); astman_send_error(s, m, "Unable to retrieve registration entry\n");
@ -1612,6 +1657,12 @@ static int ami_register(struct mansession *s, const struct message *m)
return 0; return 0;
} }
if (strcmp(registration_name, "*all") == 0) {
reregister_all();
astman_send_ack(s, m, "Reregistrations queued.");
return 0;
}
state = get_state(registration_name); state = get_state(registration_name);
if (!state) { if (!state) {
astman_send_error(s, m, "Unable to retrieve registration entry\n"); astman_send_error(s, m, "Unable to retrieve registration entry\n");
@ -1787,10 +1838,6 @@ static int cli_print_body(void *obj, void *arg, int flags)
ast_assert(context->output_buffer != NULL); ast_assert(context->output_buffer != NULL);
if (!state) {
return 0;
}
ast_str_append(&context->output_buffer, 0, " %-s/%-*.*s %-16s %-16s\n", ast_str_append(&context->output_buffer, 0, " %-s/%-*.*s %-16s %-16s\n",
id, id,
(int) (REGISTRATION_URI_FIELD_LEN - strlen(id)), (int) (REGISTRATION_URI_FIELD_LEN - strlen(id)),
@ -1799,8 +1846,8 @@ static int cli_print_body(void *obj, void *arg, int flags)
AST_VECTOR_SIZE(&registration->outbound_auths) AST_VECTOR_SIZE(&registration->outbound_auths)
? AST_VECTOR_GET(&registration->outbound_auths, 0) ? AST_VECTOR_GET(&registration->outbound_auths, 0)
: "n/a", : "n/a",
sip_outbound_registration_status_str(state->client_state->status)); (state ? sip_outbound_registration_status_str(state->client_state->status) : "Unregistered"));
ao2_ref(state, -1); ao2_cleanup(state);
if (context->show_details if (context->show_details
|| (context->show_details_only_level_0 && context->indent_level == 0)) { || (context->show_details_only_level_0 && context->indent_level == 0)) {
@ -1960,10 +2007,23 @@ static const struct ast_sorcery_observer registration_observer = {
.deleted = registration_deleted_observer, .deleted = registration_deleted_observer,
}; };
static void network_change_stasis_cb(void *data, struct stasis_subscription *sub, struct stasis_message *message)
{
/* This callback is only concerned with network change messages from the system topic. */
if (stasis_message_type(message) != ast_network_change_type()) {
return;
}
ast_debug(3, "Received network change event\n");
reregister_all();
}
static int unload_module(void) static int unload_module(void)
{ {
int remaining; int remaining;
network_change_sub = stasis_unsubscribe_and_join(network_change_sub);
ast_manager_unregister("PJSIPShowRegistrationsOutbound"); ast_manager_unregister("PJSIPShowRegistrationsOutbound");
ast_manager_unregister("PJSIPUnregister"); ast_manager_unregister("PJSIPUnregister");
ast_manager_unregister("PJSIPRegister"); ast_manager_unregister("PJSIPRegister");
@ -2101,6 +2161,9 @@ static int load_module(void)
/* Load configuration objects */ /* Load configuration objects */
ast_sorcery_load_object(ast_sip_get_sorcery(), "registration"); ast_sorcery_load_object(ast_sip_get_sorcery(), "registration");
network_change_sub = stasis_subscribe(ast_system_topic(),
network_change_stasis_cb, NULL);
return AST_MODULE_LOAD_SUCCESS; return AST_MODULE_LOAD_SUCCESS;
} }

Loading…
Cancel
Save