diff options
Diffstat (limited to 'res')
-rw-r--r-- | res/ari/ari_model_validators.c | 22 | ||||
-rw-r--r-- | res/ari/ari_model_validators.h | 2 | ||||
-rw-r--r-- | res/ari/resource_asterisk.c | 27 | ||||
-rw-r--r-- | res/ari/resource_asterisk.h | 26 | ||||
-rw-r--r-- | res/res_ari_asterisk.c | 99 |
5 files changed, 164 insertions, 12 deletions
diff --git a/res/ari/ari_model_validators.c b/res/ari/ari_model_validators.c index 26e9b74b7..74611750e 100644 --- a/res/ari/ari_model_validators.c +++ b/res/ari/ari_model_validators.c @@ -366,29 +366,29 @@ int ast_ari_validate_log_channel(struct ast_json *json) { int res = 1; struct ast_json_iter *iter; + int has_channel = 0; int has_configuration = 0; - int has_name = 0; int has_status = 0; int has_type = 0; for (iter = ast_json_object_iter(json); iter; iter = ast_json_object_iter_next(json, iter)) { - if (strcmp("configuration", ast_json_object_iter_key(iter)) == 0) { + if (strcmp("channel", ast_json_object_iter_key(iter)) == 0) { int prop_is_valid; - has_configuration = 1; + has_channel = 1; prop_is_valid = ast_ari_validate_string( ast_json_object_iter_value(iter)); if (!prop_is_valid) { - ast_log(LOG_ERROR, "ARI LogChannel field configuration failed validation\n"); + ast_log(LOG_ERROR, "ARI LogChannel field channel failed validation\n"); res = 0; } } else - if (strcmp("name", ast_json_object_iter_key(iter)) == 0) { + if (strcmp("configuration", ast_json_object_iter_key(iter)) == 0) { int prop_is_valid; - has_name = 1; + has_configuration = 1; prop_is_valid = ast_ari_validate_string( ast_json_object_iter_value(iter)); if (!prop_is_valid) { - ast_log(LOG_ERROR, "ARI LogChannel field name failed validation\n"); + ast_log(LOG_ERROR, "ARI LogChannel field configuration failed validation\n"); res = 0; } } else @@ -420,13 +420,13 @@ int ast_ari_validate_log_channel(struct ast_json *json) } } - if (!has_configuration) { - ast_log(LOG_ERROR, "ARI LogChannel missing required field configuration\n"); + if (!has_channel) { + ast_log(LOG_ERROR, "ARI LogChannel missing required field channel\n"); res = 0; } - if (!has_name) { - ast_log(LOG_ERROR, "ARI LogChannel missing required field name\n"); + if (!has_configuration) { + ast_log(LOG_ERROR, "ARI LogChannel missing required field configuration\n"); res = 0; } diff --git a/res/ari/ari_model_validators.h b/res/ari/ari_model_validators.h index b181506d2..1803f57c9 100644 --- a/res/ari/ari_model_validators.h +++ b/res/ari/ari_model_validators.h @@ -1302,8 +1302,8 @@ ari_validator ast_ari_validate_application_fn(void); * - attribute: string (required) * - value: string (required) * LogChannel + * - channel: string (required) * - configuration: string (required) - * - name: string (required) * - status: string (required) * - type: string (required) * Module diff --git a/res/ari/resource_asterisk.c b/res/ari/resource_asterisk.c index d2e77f504..070a1aa3d 100644 --- a/res/ari/resource_asterisk.c +++ b/res/ari/resource_asterisk.c @@ -628,6 +628,33 @@ void ast_ari_asterisk_reload_module(struct ast_variable *headers, ast_ari_response_no_content(response); } +void ast_ari_asterisk_add_log(struct ast_variable *headers, + struct ast_ari_asterisk_add_log_args *args, + struct ast_ari_response *response) +{ + int res; + + ast_assert(response != NULL); + + res = ast_logger_create_channel(args->log_channel_name, args->configuration); + + if (res == AST_LOGGER_DECLINE) { + ast_ari_response_error(response, 400, "Bad Request", + "Configuration levels are required"); + return; + } else if (res == AST_LOGGER_FAILURE) { + ast_ari_response_error(response, 409, "Conflict", + "Log channel already exists"); + return; + } else if (res == AST_LOGGER_ALLOC_ERROR) { + ast_ari_response_error(response, 500, "Internal Server Error", + "Allocation failed"); + return; + } + + ast_ari_response_no_content(response); +} + void ast_ari_asterisk_rotate_log(struct ast_variable *headers, struct ast_ari_asterisk_rotate_log_args *args, struct ast_ari_response *response) diff --git a/res/ari/resource_asterisk.h b/res/ari/resource_asterisk.h index 00f463cee..5f84d0761 100644 --- a/res/ari/resource_asterisk.h +++ b/res/ari/resource_asterisk.h @@ -194,6 +194,32 @@ struct ast_ari_asterisk_reload_module_args { * \param[out] response HTTP response */ void ast_ari_asterisk_reload_module(struct ast_variable *headers, struct ast_ari_asterisk_reload_module_args *args, struct ast_ari_response *response); +/*! Argument struct for ast_ari_asterisk_add_log() */ +struct ast_ari_asterisk_add_log_args { + /*! The log channel to add */ + const char *log_channel_name; + /*! levels of the log channel */ + const char *configuration; +}; +/*! + * \brief Body parsing function for /asterisk/logging/{logChannelName}. + * \param body The JSON body from which to parse parameters. + * \param[out] args The args structure to parse into. + * \retval zero on success + * \retval non-zero on failure + */ +int ast_ari_asterisk_add_log_parse_body( + struct ast_json *body, + struct ast_ari_asterisk_add_log_args *args); + +/*! + * \brief Adds a log channel. + * + * \param headers HTTP headers + * \param args Swagger parameters + * \param[out] response HTTP response + */ +void ast_ari_asterisk_add_log(struct ast_variable *headers, struct ast_ari_asterisk_add_log_args *args, struct ast_ari_response *response); /*! Argument struct for ast_ari_asterisk_delete_log() */ struct ast_ari_asterisk_delete_log_args { /*! Log channels name */ diff --git a/res/res_ari_asterisk.c b/res/res_ari_asterisk.c index 320800a5d..08ec95166 100644 --- a/res/res_ari_asterisk.c +++ b/res/res_ari_asterisk.c @@ -721,6 +721,104 @@ static void ast_ari_asterisk_reload_module_cb( fin: __attribute__((unused)) return; } +int ast_ari_asterisk_add_log_parse_body( + struct ast_json *body, + struct ast_ari_asterisk_add_log_args *args) +{ + struct ast_json *field; + /* Parse query parameters out of it */ + field = ast_json_object_get(body, "configuration"); + if (field) { + args->configuration = ast_json_string_get(field); + } + return 0; +} + +/*! + * \brief Parameter parsing callback for /asterisk/logging/{logChannelName}. + * \param get_params GET parameters in the HTTP request. + * \param path_vars Path variables extracted from the request. + * \param headers HTTP headers. + * \param[out] response Response to the HTTP request. + */ +static void ast_ari_asterisk_add_log_cb( + struct ast_tcptls_session_instance *ser, + struct ast_variable *get_params, struct ast_variable *path_vars, + struct ast_variable *headers, struct ast_ari_response *response) +{ + struct ast_ari_asterisk_add_log_args args = {}; + struct ast_variable *i; + RAII_VAR(struct ast_json *, body, NULL, ast_json_unref); +#if defined(AST_DEVMODE) + int is_valid; + int code; +#endif /* AST_DEVMODE */ + + for (i = get_params; i; i = i->next) { + if (strcmp(i->name, "configuration") == 0) { + args.configuration = (i->value); + } else + {} + } + for (i = path_vars; i; i = i->next) { + if (strcmp(i->name, "logChannelName") == 0) { + args.log_channel_name = (i->value); + } else + {} + } + /* Look for a JSON request entity */ + body = ast_http_get_json(ser, headers); + if (!body) { + switch (errno) { + case EFBIG: + ast_ari_response_error(response, 413, "Request Entity Too Large", "Request body too large"); + goto fin; + case ENOMEM: + ast_ari_response_error(response, 500, "Internal Server Error", "Error processing request"); + goto fin; + case EIO: + ast_ari_response_error(response, 400, "Bad Request", "Error parsing request body"); + goto fin; + } + } + if (ast_ari_asterisk_add_log_parse_body(body, &args)) { + ast_ari_response_alloc_failed(response); + goto fin; + } + ast_ari_asterisk_add_log(headers, &args, response); +#if defined(AST_DEVMODE) + code = response->response_code; + + switch (code) { + case 0: /* Implementation is still a stub, or the code wasn't set */ + is_valid = response->message == NULL; + break; + case 500: /* Internal Server Error */ + case 501: /* Not Implemented */ + case 400: /* Bad request body */ + case 409: /* Log channel could not be created. */ + is_valid = 1; + break; + default: + if (200 <= code && code <= 299) { + is_valid = ast_ari_validate_void( + response->message); + } else { + ast_log(LOG_ERROR, "Invalid error response %d for /asterisk/logging/{logChannelName}\n", code); + is_valid = 0; + } + } + + if (!is_valid) { + ast_log(LOG_ERROR, "Response validation failed for /asterisk/logging/{logChannelName}\n"); + ast_ari_response_error(response, 500, + "Internal Server Error", "Response validation failed"); + } +#endif /* AST_DEVMODE */ + +fin: __attribute__((unused)) + return; +} /*! * \brief Parameter parsing callback for /asterisk/logging/{logChannelName}. * \param get_params GET parameters in the HTTP request. @@ -1120,6 +1218,7 @@ static struct stasis_rest_handlers asterisk_logging_logChannelName = { .path_segment = "logChannelName", .is_wildcard = 1, .callbacks = { + [AST_HTTP_POST] = ast_ari_asterisk_add_log_cb, [AST_HTTP_DELETE] = ast_ari_asterisk_delete_log_cb, }, .num_children = 1, |