ПРОЕКТЫ 


  АРХИВ 


Apache-Talk @lexa.ru 

Inet-Admins @info.east.ru 

Filmscanners @halftone.co.uk 

Security-alerts @yandex-team.ru 

nginx-ru @sysoev.ru 


  СТАТЬИ 


  ПЕРСОНАЛЬНОЕ 


  ПРОГРАММЫ 



ПИШИТЕ
ПИСЬМА












     АРХИВ :: nginx-ru
Nginx-ru mailing list archive (nginx-ru@sysoev.ru)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

code verification


  • To: nginx-ru@xxxxxxxxx
  • Subject: code verification
  • From: Alexander Zheglov <tolko.bi.vyzhit@xxxxxxxxx>
  • Date: Fri, 16 Sep 2011 19:50:04 +0400
  • Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=from:content-type:subject:date:message-id:to:mime-version:x-mailer; bh=agj3T3+hMBSLSflNGzS4uBMd9AeToN4pkMO/TmXiQZE=; b=jzIt+pLEYncTHcE/mWei8g0wZGMOyw6D7j9ihB4WhA1wJuN7GQlyHrAgtr19Jgf1QK 2w2JampkaOCoCTtTnT7i4msNDGKd5tLuh5TH/i5FzfvUDFkKwn9S6nyaIU0jNekIOrYT RSYHzQ62hE+PwPGg3X4Wp5DVcmwg8foOxX3eE=

Добрый день!

Я хотел бы попросить уважаемых разработчиков nginx верифицировать работу из моего модуля с переменными. Поскольку nginx изучался  по исходным текстам, я мог опустить какие то существенные моменты и заложить ошибки для дальнейшей нестабильной работы. Если кто-нибудь из экспертов пробежится взглядом по коду (я, естественно, оставил только моменты связанные с переменными), это было бы для меня большой пользой.

Постановка задачи:

Конфигурация модуля dobro определяется в секции http и конкретное подклчение происходит в location.
http {
    dobro_config {
        dobro_amount 10m;
        dobro_timeout 10h;
    };
    server {
        location / {
            dobro on;
	    root /;
        };
     };
};

Требовалось иметь возможность отключать работу модуля используя rewrite_module:
     location / {
            dobro on;
            if ($request_uri ~ /netdobra) { set $dobro_off "true"; }
	    ...
     }

Для упрощения логики считаю, что если переменная dobro_off не определена или пуста, значит работаем как работали.

Соотвественно код. Постарался максимально сократить и дать необходимые пояснения к функциям в комментариях. Хендлер модуля регистрируется в фазе ACCESS.

typedef struct {
    ngx_int_t index;
} ngx_http_dobro_loc_conf_t;

#define DOBRO_NO_INDEX -1

/* в секции preconfiguration переменная создается */
static ngx_int_t ngx_http_dobro_add_variables(ngx_conf_t *cf) {
    ngx_http_variable_t  *var;
    var=ngx_http_add_variable(cf, &ngx_http_dobro_ignore, NGX_HTTP_VAR_CHANGEABLE);
    if (var == NULL) {
        return NGX_ERROR;
    }
    var->get_handler = ngx_http_dobro_get_variable;
    return NGX_OK;
}
/* get handler возвращает пустую переменную, модуль работает всегда, только если явно не отключен */
static ngx_int_t ngx_http_dobro_get_variable(ngx_http_request_t *r, ngx_http_variable_value_t *v, uintptr_t data) {
    *v = ngx_http_variable_null_value;
    return NGX_OK;
}

/* функция, которая обрабатывает конфигурационную строку dobro on; */
/* в момент подключения модуля в конфигурационную структуру location заносится индекс переменной, которая гарантированно создана */
static char * ngx_http_dobro_attach_pool(ngx_conf_t *cf, ngx_command_t *cmd, void *conf) {
    ngx_str_t  *value;
    ngx_http_dobro_loc_conf_t *rlcf = conf;
   ...
    rlcf->index = ngx_http_get_variable_index(cf, &ngx_http_dobro_ignore);
    if (rlcf->index == NGX_ERROR) {
        return NGX_CONF_ERROR;
    }
..}

/* при создании конфигурационной структуры index устанавливается в -1 */
static void * ngx_http_dobro_create_loc_conf(ngx_conf_t *cf) {
    ngx_http_dobro_loc_conf_t  *conf;
    ...
    conf->index = DOBRO_NO_INDEX;
..}

/* индекс переменной наследуется при слиянии локейшнов, если еще не был определен */
static char * ngx_http_dobro_merge_loc_conf(ngx_conf_t *cf, void *parent, void *child) {
    ngx_http_dobro_loc_conf_t  *prev = parent;
    ngx_http_dobro_loc_conf_t  *conf = child;

    if (conf->index == DOBRO_NO_INDEX)
        conf->index = prev->index;
..}

/* в функции-обработчике запроса из конфигурационной структуры берется индекс переменной,
далее проверяется что переменная существует и если ее длина не 0, то сравнивается с "0".
если переменная не равна "0" - не обрабатываем. */
static ngx_int_t ngx_http_dobro_handler(ngx_http_request_t *r) {
    ngx_http_dobro_loc_conf_t * rlcf = ngx_http_get_module_loc_conf(r, ngx_http_dobro_module);
    ngx_http_variable_value_t *v;
...
    if (rlcf->index == DOBRO_NO_INDEX) {
        v = NULL;
    } else {
        v = ngx_http_get_indexed_variable(r, rlcf->index);
    }

    if (v != NULL && v->len > 0) {
        if (ngx_strcmp(v->data, "0") != 0) {
            return NGX_DECLINED;
        }
    }
..}

--
С уважением,
Александр Жеглов



_______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru


 




Copyright © Lexa Software, 1996-2009.