code verification

[Alexander Zheglov]

Добрый день!

Я хотел бы попросить уважаемых разработчиков 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;
        }
    }
..}

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



----------- следущая часть -----------
Вложение в формате HTML было извлечено…
URL: <http://mailman.nginx.org/pipermail/nginx-ru/attachments/20110916/711904dc/attachment-0001.html>