Project

General

Profile

Actions

Task #818

closed

Task #813: 10: Server: Upravy serverovych funkci v Warden.pm

Pri spatne zadanem typu polozek target_port, attack_scale, priority, timeout servet potichu zmenu hodnotu na undef a do DB se ulozi NULL

Added by Tomáš Plesník almost 12 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomáš Plesník
Category:
-
Target version:
Start date:
01/25/2013
Due date:
% Done:

0%

Estimated time:

Description

Tak je to nove udelano v warden-server-2.1

Actions #1

Updated by Pavel Kácha almost 12 years ago

Možná by bylo rozumné původní zprávu zalogovat, pokud ji měníme - abychom při případném debugování věděli, o co jsme přišli.

Actions #2

Updated by Tomáš Plesník almost 12 years ago

Toz mozem, uz i ve 2.1?

Actions #3

Updated by Pavel Kácha almost 12 years ago

Pokud jsi to udělal i v 2.1 (to tiché NULLování), tak zalogování tam určitě přidej.

Actions #4

Updated by Tomáš Plesník almost 12 years ago

OK, pridam to tam, ale nemyslim si, ze bychom kvuli tomuto museli vydavat novou BETU. Staci to jen nasadit na warden-dev a komitnout do GITu. Do finalniho balicku se to pak dostane pri jeho finalnim buildu.

Actions #5

Updated by Pavel Kácha almost 12 years ago

Souhlasím. Jedním logovacím řádkem už se snad další chyby zatáhnout nedají (slavná poslední slova). Všiml jsem si, že jste s Kostějem domlouvali jeho testování - jakou betu tedy testoval? Tu, kde už byla inicializace prázdného validačního pole a kontrola a měnění na undef, nebo tu předtím?

Actions #6

Updated by Tomáš Plesník almost 12 years ago

  • Status changed from New to Resolved

Hotovo, doplneno do obou verzi - revize bd6cf11e (2.1) a eaf4b57b (2.2). Kostej mi akorat v sobotu oznamil, ze jde testovat, tak at s tim pocitam a moc nelaboruji, zadna jina domluva neprobehla a testoval verzi 2.1 jiz s pridanou funkcionalitou vypnuti hashe %VALID_STRINGS pomoci zakomentovani v warden-server.conf a zmenou chybnych polozek udalosti na undef pokud nejsou cele cislo.

Actions #7

Updated by Tomáš Plesník over 11 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF