summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Barach <dave@barachs.net>2019-10-17 15:53:08 -0400
committerFlorin Coras <florin.coras@gmail.com>2019-10-17 20:59:14 +0000
commit9929561a5d7388e655eadfaf458d27b96f00abd5 (patch)
treef36fd860a00659025097458e08acbe71ab552f50
parent19cb04df063a85ab8e416a9f44fd47ff89857901 (diff)
http_static: pass http request type to builtin handlers
Makes it easy to tell GETs from POSTs, etc., in case one wants to register the same handler for multiple request types. Return interface stats as a json array, makes .js code a lot simpler. Type: refactor Signed-off-by: Dave Barach <dave@barachs.net> Change-Id: Iadae7b909cf3e8ea179fe93620737fdf68f888ac
-rw-r--r--src/plugins/builtinurl/builtins.c89
-rw-r--r--src/plugins/http_static/static_server.c4
2 files changed, 60 insertions, 33 deletions
diff --git a/src/plugins/builtinurl/builtins.c b/src/plugins/builtinurl/builtins.c
index 471fe58e8c7..6b2b57bce7f 100644
--- a/src/plugins/builtinurl/builtins.c
+++ b/src/plugins/builtinurl/builtins.c
@@ -19,7 +19,8 @@
#include <vpp/app/version.h>
int
-handle_get_version (u8 * request, http_session_t * hs)
+handle_get_version (http_builtin_method_type_t reqtype,
+ u8 * request, http_session_t * hs)
{
u8 *s = 0;
@@ -63,63 +64,87 @@ trim_path_from_request (u8 * s, char *path)
}
int
-handle_get_interface_stats (u8 * request, http_session_t * hs)
+handle_get_interface_stats (http_builtin_method_type_t reqtype,
+ u8 * request, http_session_t * hs)
{
u8 *s = 0, *stats = 0;
- u32 sw_if_index;
uword *p;
+ u32 *sw_if_indices = 0;
+ vnet_hw_interface_t *hi;
vnet_sw_interface_t *si;
+ char *q = "\"";
+ int i;
+ int need_comma = 0;
u8 *format_vnet_sw_interface_cntrs (u8 * s, vnet_interface_main_t * im,
vnet_sw_interface_t * si, int json);
-
vnet_main_t *vnm = vnet_get_main ();
+ vnet_interface_main_t *im = &vnm->interface_main;
- trim_path_from_request (request, "interface_stats.json");
+ /* Get stats for a single interface via http POST */
+ if (reqtype == HTTP_BUILTIN_METHOD_POST)
+ {
+ trim_path_from_request (request, "interface_stats.json");
- /* get data */
+ /* Find the sw_if_index */
+ p = hash_get (im->hw_interface_by_name, request);
+ if (!p)
+ {
+ s = format (s, "{\"interface_stats\": {[\n");
+ s = format (s, " \"name\": \"%s\",", request);
+ s = format (s, " \"error\": \"%s\"", "UnknownInterface");
+ s = format (s, "]}\n");
+ goto out;
+ }
- p = hash_get (vnm->interface_main.hw_interface_by_name, request);
- if (!p)
+ vec_add1 (sw_if_indices, p[0]);
+ }
+ else /* default, HTTP_BUILTIN_METHOD_GET */
{
- s = format (s, "{\"interface_stats\": {");
- s = format (s, " \"name\": \"%s\",", request);
- s = format (s, " \"stats\": \"%s\"", "ERRORUnknownInterface");
- s = format (s, "}}\r\n");
- goto out;
+ /* *INDENT-OFF* */
+ pool_foreach (hi, im->hw_interfaces,
+ ({
+ vec_add1 (sw_if_indices, hi->sw_if_index);
+ }));
+ /* *INDENT-ON* */
}
- sw_if_index = p[0];
- si = vnet_get_sw_interface (vnm, sw_if_index);
+ s = format (s, "{%sinterface_stats%s: [\n", q, q);
- stats = format_vnet_sw_interface_cntrs (stats, &vnm->interface_main, si,
- 1 /* want json */ );
-
- /* No active counters */
- if (stats == 0)
+ for (i = 0; i < vec_len (sw_if_indices); i++)
{
- s = format (s, "{\"interface_stats\": {");
- s = format (s, " \"name\": \"%s\"", request);
- s = format (s, "}}\r\n");
- goto out;
+ si = vnet_get_sw_interface (vnm, sw_if_indices[i]);
+ if (need_comma)
+ s = format (s, ",\n");
+
+ need_comma = 1;
+
+ s = format (s, "{%sname%s: %s%U%s, ", q, q, q,
+ format_vnet_sw_if_index_name, vnm, sw_if_indices[i], q);
+
+ stats = format_vnet_sw_interface_cntrs (stats, &vnm->interface_main, si,
+ 1 /* want json */ );
+ if (vec_len (stats))
+ s = format (s, "%v}", stats);
+ else
+ s = format (s, "%snone%s: %strue%s}", q, q, q, q);
+ vec_reset_length (stats);
}
- /* Build answer */
- s = format (s, "{\"interface_stats\": {");
- s = format (s, "\"name\": \"%s\",\n", request);
- s = format (s, "%s", stats);
- s = format (s, "}}\n");
- vec_free (stats);
+ s = format (s, "]}\n");
out:
hs->data = s;
hs->data_offset = 0;
hs->cache_pool_index = ~0;
hs->free_data = 1;
+ vec_free (sw_if_indices);
+ vec_free (stats);
return 0;
}
int
-handle_get_interface_list (u8 * request, http_session_t * hs)
+handle_get_interface_list (http_builtin_method_type_t reqtype,
+ u8 * request, http_session_t * hs)
{
u8 *s = 0;
int i;
@@ -168,6 +193,8 @@ builtinurl_handler_init (builtinurl_main_t * bm)
bm->register_handler (handle_get_interface_list, "interface_list.json",
HTTP_BUILTIN_METHOD_GET);
bm->register_handler (handle_get_interface_stats,
+ "interface_stats.json", HTTP_BUILTIN_METHOD_GET);
+ bm->register_handler (handle_get_interface_stats,
"interface_stats.json", HTTP_BUILTIN_METHOD_POST);
}
diff --git a/src/plugins/http_static/static_server.c b/src/plugins/http_static/static_server.c
index 16136d2d4ed..5dfa347dc03 100644
--- a/src/plugins/http_static/static_server.c
+++ b/src/plugins/http_static/static_server.c
@@ -676,10 +676,10 @@ find_end:
if (p)
{
int rv;
- int (*fp) (u8 *, http_session_t *);
+ int (*fp) (http_builtin_method_type_t, u8 *, http_session_t *);
fp = (void *) p[0];
hs->path = path;
- rv = (*fp) (request, hs);
+ rv = (*fp) (request_type, request, hs);
if (rv)
{
clib_warning ("builtin handler %llx hit on %s '%s' but failed!",