Skip to content

Commit 3f1dadc

Browse files
committed
fix: correct status code separator logic in JSON display
1 parent d28faa5 commit 3f1dadc

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/ngx_http_vhost_traffic_status_display_json.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,16 +106,18 @@ ngx_http_vhost_traffic_status_display_set_server_node(
106106
buf = ngx_sprintf(buf, NGX_HTTP_VHOST_TRAFFIC_STATUS_JSON_FMT_SERVER_STATUS_CODE_START);
107107

108108
ngx_uint_t *status_codes = (ngx_uint_t *) ctx->measure_status_codes->elts;
109+
ngx_uint_t inserted = 0;
109110
for (ngx_uint_t i = 0; i < ctx->measure_status_codes->nelts; i++) {
110111
if (vtsn->stat_status_code_counter[i] == 0 && ctx->measure_all_status_codes) {
111112
continue;
112113
}
113114
status_code = &status_codes[i];
114-
if (i > 0) {
115+
if (inserted > 0) {
115116
buf = ngx_sprintf(buf, NGX_HTTP_VHOST_TRAFFIC_STATUS_JSON_FMT_SERVER_STATUS_SEPARATOR);
116117
}
117118
buf = ngx_sprintf(buf, NGX_HTTP_VHOST_TRAFFIC_STATUS_JSON_FMT_SERVER_STATUS_CODE,
118119
*status_code, vtsn->stat_status_code_counter[i]);
120+
inserted++;
119121
}
120122

121123
buf = ngx_sprintf(buf, NGX_HTTP_VHOST_TRAFFIC_STATUS_JSON_FMT_SERVER_STATUS_CODE_END);

0 commit comments

Comments
 (0)