[Asterisk-code-review] res/ari/resource_channels.c: Added hangup reason code for channels (...asterisk[13])
sungtae kim
asteriskteam at digium.com
Wed May 15 15:50:44 CDT 2019
sungtae kim has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/11382
Change subject: res/ari/resource_channels.c: Added hangup reason code for channels
......................................................................
res/ari/resource_channels.c: Added hangup reason code for channels
Currently, DELETE /ari/channels/<channelID> supports only few hangup reasons.
It's good enough for simple use, but when it needs to set the detail reason,
it comes challenges.
Added reason_code query parameter for that.
ASTERISK-28385
Change-Id: I1cf1d991ffd759d0591b347445a55f416ddc3ff2
---
M res/ari/resource_channels.c
M res/ari/resource_channels.h
M res/res_ari_channels.c
M rest-api/api-docs/channels.json
4 files changed, 78 insertions(+), 15 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/82/11382/1
diff --git a/res/ari/resource_channels.c b/res/ari/resource_channels.c
index bdb3534..d20d0c4 100644
--- a/res/ari/resource_channels.c
+++ b/res/ari/resource_channels.c
@@ -55,6 +55,40 @@
#include <limits.h>
+/*! \brief Return the corresponded hangup code of the given reason */
+static int convert_reason_to_hangup_code(const char* reason)
+{
+ if (!strcmp(reason, "normal")) {
+ return AST_CAUSE_NORMAL;
+ } else if (!strcmp(reason, "busy")) {
+ return AST_CAUSE_BUSY;
+ } else if (!strcmp(reason, "congestion")) {
+ return AST_CAUSE_CONGESTION;
+ } else if (!strcmp(reason, "no_answer")) {
+ return AST_CAUSE_NOANSWER;
+ } else if (!strcmp(reason, "timeout")) {
+ return AST_CAUSE_NO_USER_RESPONSE;
+ } else if (!strcmp(reason, "rejected")) {
+ return AST_CAUSE_CALL_REJECTED;
+ } else if (!strcmp(reason, "unallocated")) {
+ return AST_CAUSE_UNALLOCATED;
+ } else if (!strcmp(reason, "normal_unspecified")) {
+ return AST_CAUSE_NORMAL_UNSPECIFIED;
+ } else if (!strcmp(reason, "number_incomplete")) {
+ return AST_CAUSE_INVALID_NUMBER_FORMAT;
+ } else if (!strcmp(reason, "codec_mismatch")) {
+ return AST_CAUSE_BEARERCAPABILITY_NOTAVAIL;
+ } else if (!strcmp(reason, "interworking")) {
+ return AST_CAUSE_INTERWORKING;
+ } else if (!strcmp(reason, "failure")) {
+ return AST_CAUSE_FAILURE;
+ } else if(!strcmp(reason, "answered_elsewhere")) {
+ return AST_CAUSE_ANSWERED_ELSEWHERE;
+ }
+
+ return -1;
+}
+
/*!
* \brief Finds the control object for a channel, filling the response with an
* error, if appropriate.
@@ -782,21 +816,34 @@
return;
}
- if (ast_strlen_zero(args->reason) || !strcmp(args->reason, "normal")) {
- cause = AST_CAUSE_NORMAL;
- } else if (!strcmp(args->reason, "busy")) {
- cause = AST_CAUSE_BUSY;
- } else if (!strcmp(args->reason, "congestion")) {
- cause = AST_CAUSE_CONGESTION;
- } else if (!strcmp(args->reason, "no_answer")) {
- cause = AST_CAUSE_NOANSWER;
- } else {
- ast_ari_response_error(
- response, 400, "Invalid Reason",
- "Invalid reason for hangup provided");
+ if (!ast_strlen_zero(args->reason) && !ast_strlen_zero(args->reason_code)) {
+ ast_ari_response_error(response, 400, "Bad Request",
+ "The reason and reason_code can't both be specified");
return;
}
+ if (!ast_strlen_zero(args->reason_code)) {
+ /* reason_code allows any hangup code */
+ if (sscanf(args->reason_code, "%30d", &cause) != 1) {
+ ast_ari_response_error(
+ response, 400, "Invalid Reason Code",
+ "Invalid reason for hangup reason code provided");
+ return;
+ }
+ } else if (!ast_strlen_zero(args->reason)) {
+ /* reason allows only listed hangup reason */
+ cause = convert_reason_to_hangup_code(args->reason);
+ if (cause == -1) {
+ ast_ari_response_error(
+ response, 400, "Invalid Reason",
+ "Invalid reason for hangup reason provided");
+ return;
+ }
+ } else {
+ /* not specified. set default hangup */
+ cause = AST_CAUSE_NORMAL;
+ }
+
ast_channel_hangupcause_set(chan, cause);
ast_softhangup(chan, AST_SOFTHANGUP_EXPLICIT);
diff --git a/res/ari/resource_channels.h b/res/ari/resource_channels.h
index 3077297..adecb3d 100644
--- a/res/ari/resource_channels.h
+++ b/res/ari/resource_channels.h
@@ -171,7 +171,9 @@
struct ast_ari_channels_hangup_args {
/*! Channel's id */
const char *channel_id;
- /*! Reason for hanging up the channel */
+ /*! The reason code for hanging up the channel for detail use. Mutually exclusive with 'reason'. See detail hangup codes at here. https://wiki.asterisk.org/wiki/display/AST/Hangup+Cause+Mappings */
+ const char *reason_code;
+ /*! Reason for hanging up the channel for simple use. Mutually exclusive with 'reason_code'. */
const char *reason;
};
/*!
diff --git a/res/res_ari_channels.c b/res/res_ari_channels.c
index 5119d64..6b45210 100644
--- a/res/res_ari_channels.c
+++ b/res/res_ari_channels.c
@@ -479,6 +479,10 @@
{
struct ast_json *field;
/* Parse query parameters out of it */
+ field = ast_json_object_get(body, "reason_code");
+ if (field) {
+ args->reason_code = ast_json_string_get(field);
+ }
field = ast_json_object_get(body, "reason");
if (field) {
args->reason = ast_json_string_get(field);
@@ -506,6 +510,9 @@
#endif /* AST_DEVMODE */
for (i = get_params; i; i = i->next) {
+ if (strcmp(i->name, "reason_code") == 0) {
+ args.reason_code = (i->value);
+ } else
if (strcmp(i->name, "reason") == 0) {
args.reason = (i->value);
} else
diff --git a/rest-api/api-docs/channels.json b/rest-api/api-docs/channels.json
index fdeb0b8..446cdd8 100644
--- a/rest-api/api-docs/channels.json
+++ b/rest-api/api-docs/channels.json
@@ -324,13 +324,20 @@
"dataType": "string"
},
{
+ "name": "reason_code",
+ "description": "The reason code for hanging up the channel for detail use. Mutually exclusive with 'reason'. See detail hangup codes at here. https://wiki.asterisk.org/wiki/display/AST/Hangup+Cause+Mappings",
+ "paramType": "query",
+ "required": false,
+ "allowMultiple": false,
+ "dataType": "string"
+ },
+ {
"name": "reason",
- "description": "Reason for hanging up the channel",
+ "description": "Reason for hanging up the channel for simple use. Mutually exclusive with 'reason_code'.",
"paramType": "query",
"required": false,
"allowMultiple": false,
"dataType": "string",
- "defalutValue": "normal",
"allowableValues": {
"valueType": "LIST",
"values": [
--
To view, visit https://gerrit.asterisk.org/c/asterisk/+/11382
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I1cf1d991ffd759d0591b347445a55f416ddc3ff2
Gerrit-Change-Number: 11382
Gerrit-PatchSet: 1
Gerrit-Owner: sungtae kim <pchero21 at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190515/8acf205c/attachment-0001.html>
More information about the asterisk-code-review
mailing list