Re: r->connection->number в встроеном перле
umask
umask at yandex.ru
Thu Oct 25 05:24:42 UTC 2012
Добрый день,
Спасибо за патч.
Максим, этот патч попадёт в mainline?
Цель вообще в том, чтобы сделать аналог модуля request_id, но только на встроеном перле (не хочется тянуть сторонний модуль, пересобираться с ним каждый раз и т.д).
24.10.2012, 19:33, "Maxim Dounin" <mdounin at mdounin.ru>:
> Hello!
>
> On Wed, Oct 24, 2012 at 03:06:15PM +0400, umask wrote:
>
>> Это вообще возможно сделать?
>>
>> А то если невозможно, то хотелось бы такой ответ получить.
>
> Сейчас - нет, но вообще добавить элементарно. С прилагающимся
> патчем можно получить через стандартный интерфейс получения
> переменных, r->variable().
>
>> Спасибо.
>>
>> 21.10.2012, 13:45, "umask" <umask at yandex.ru>:
>>> Добрый день,
>>>
>>> а подскажите, как можно во встроеном перле обратиться к значению r->connection->number?
>>>
>>> Мне нужно получить текущее значение счётчика соединений nginx и текущее значение счётчика запросов.
>>>
>>> _______________________________________________
>>> nginx-ru mailing list
>>> nginx-ru at nginx.org
>>> http://mailman.nginx.org/mailman/listinfo/nginx-ru
>> _______________________________________________
>> nginx-ru mailing list
>> nginx-ru at nginx.org
>> http://mailman.nginx.org/mailman/listinfo/nginx-ru
>
> --
> Maxim Dounin
> http://nginx.com/support.html
>
> ,
> # HG changeset patch
> # User Maxim Dounin <mdounin at mdounin.ru>
> # Date 1351092441 -14400
> # Node ID 53b5e562dab09b388ded32c17ffb811fdb03b4a3
> # Parent 0f453c43fe6fb37cd5d50e8850d2043c1873c6ae
> Variables $connection and $connection_requests.
>
> Log module only counterparts are removed as they aren't used often.
>
> diff --git a/src/http/modules/ngx_http_log_module.c b/src/http/modules/ngx_http_log_module.c
> --- a/src/http/modules/ngx_http_log_module.c
> +++ b/src/http/modules/ngx_http_log_module.c
> @@ -78,10 +78,6 @@ static void ngx_http_log_write(ngx_http_
> static ssize_t ngx_http_log_script_write(ngx_http_request_t *r,
> ngx_http_log_script_t *script, u_char **name, u_char *buf, size_t len);
>
> -static u_char *ngx_http_log_connection(ngx_http_request_t *r, u_char *buf,
> - ngx_http_log_op_t *op);
> -static u_char *ngx_http_log_connection_requests(ngx_http_request_t *r,
> - u_char *buf, ngx_http_log_op_t *op);
> static u_char *ngx_http_log_pipe(ngx_http_request_t *r, u_char *buf,
> ngx_http_log_op_t *op);
> static u_char *ngx_http_log_time(ngx_http_request_t *r, u_char *buf,
> @@ -194,9 +190,6 @@ static ngx_str_t ngx_http_combined_fmt
>
> static ngx_http_log_var_t ngx_http_log_vars[] = {
> - { ngx_string("connection"), NGX_ATOMIC_T_LEN, ngx_http_log_connection },
> - { ngx_string("connection_requests"), NGX_INT_T_LEN,
> - ngx_http_log_connection_requests },
> { ngx_string("pipe"), 1, ngx_http_log_pipe },
> { ngx_string("time_local"), sizeof("28/Sep/1970:12:00:00 +0600") - 1,
> ngx_http_log_time },
> @@ -500,22 +493,6 @@ ngx_http_log_copy_long(ngx_http_request_
>
> static u_char *
> -ngx_http_log_connection(ngx_http_request_t *r, u_char *buf,
> - ngx_http_log_op_t *op)
> -{
> - return ngx_sprintf(buf, "%uA", r->connection->number);
> -}
> -
> -
> -static u_char *
> -ngx_http_log_connection_requests(ngx_http_request_t *r, u_char *buf,
> - ngx_http_log_op_t *op)
> -{
> - return ngx_sprintf(buf, "%ui", r->connection->requests);
> -}
> -
> -
> -static u_char *
> ngx_http_log_pipe(ngx_http_request_t *r, u_char *buf, ngx_http_log_op_t *op)
> {
> if (r->pipeline) {
> diff --git a/src/http/ngx_http_variables.c b/src/http/ngx_http_variables.c
> --- a/src/http/ngx_http_variables.c
> +++ b/src/http/ngx_http_variables.c
> @@ -97,6 +97,11 @@ static ngx_int_t ngx_http_variable_sent_
> static ngx_int_t ngx_http_variable_sent_transfer_encoding(ngx_http_request_t *r,
> ngx_http_variable_value_t *v, uintptr_t data);
>
> +static ngx_int_t ngx_http_variable_connection(ngx_http_request_t *r,
> + ngx_http_variable_value_t *v, uintptr_t data);
> +static ngx_int_t ngx_http_variable_connection_requests(ngx_http_request_t *r,
> + ngx_http_variable_value_t *v, uintptr_t data);
> +
> static ngx_int_t ngx_http_variable_nginx_version(ngx_http_request_t *r,
> ngx_http_variable_value_t *v, uintptr_t data);
> static ngx_int_t ngx_http_variable_hostname(ngx_http_request_t *r,
> @@ -265,6 +270,12 @@ static ngx_http_variable_t ngx_http_cor
> offsetof(ngx_http_request_t, limit_rate),
> NGX_HTTP_VAR_CHANGEABLE|NGX_HTTP_VAR_NOCACHEABLE, 0 },
>
> + { ngx_string("connection"), NULL,
> + ngx_http_variable_connection, 0, 0, 0 },
> +
> + { ngx_string("connection_requests"), NULL,
> + ngx_http_variable_connection_requests, 0, 0, 0 },
> +
> { ngx_string("nginx_version"), NULL, ngx_http_variable_nginx_version,
> 0, 0, 0 },
>
> @@ -1822,6 +1833,48 @@ ngx_http_variable_request_body_file(ngx_
>
> static ngx_int_t
> +ngx_http_variable_connection(ngx_http_request_t *r,
> + ngx_http_variable_value_t *v, uintptr_t data)
> +{
> + u_char *p;
> +
> + p = ngx_pnalloc(r->pool, NGX_ATOMIC_T_LEN);
> + if (p == NULL) {
> + return NGX_ERROR;
> + }
> +
> + v->len = ngx_sprintf(p, "%uA", r->connection->number) - p;
> + v->valid = 1;
> + v->no_cacheable = 0;
> + v->not_found = 0;
> + v->data = p;
> +
> + return NGX_OK;
> +}
> +
> +
> +static ngx_int_t
> +ngx_http_variable_connection_requests(ngx_http_request_t *r,
> + ngx_http_variable_value_t *v, uintptr_t data)
> +{
> + u_char *p;
> +
> + p = ngx_pnalloc(r->pool, NGX_INT_T_LEN);
> + if (p == NULL) {
> + return NGX_ERROR;
> + }
> +
> + v->len = ngx_sprintf(p, "%ui", r->connection->requests) - p;
> + v->valid = 1;
> + v->no_cacheable = 0;
> + v->not_found = 0;
> + v->data = p;
> +
> + return NGX_OK;
> +}
> +
> +
> +static ngx_int_t
> ngx_http_variable_nginx_version(ngx_http_request_t *r,
> ngx_http_variable_value_t *v, uintptr_t data)
> {
>
> ,
> _______________________________________________
> nginx-ru mailing list
> nginx-ru at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-ru
Подробная информация о списке рассылки nginx-ru