Project

General

Profile

Actions

Bug #1154

closed

Prijimani vlastnich udalosti

Added by Michal Kostěnec over 11 years ago. Updated about 11 years ago.

Status:
Closed
Priority:
High
Assignee:
Tomáš Plesník
Category:
-
Target version:
Start date:
08/19/2013
Due date:
% Done:

0%

Estimated time:

Description

Klient prijima vlastni udalosti, i kdyz v registraci ma nastaveno receive_own_events = 'f'. Chybu zpusobuje radoby escapovany '%' v teto casti kodu Warden.pm:

my ($domain) = $cn =~ /([^\.]+\.[^\.]+)$/;
$domain = '\%' . $domain;
$sth->execute($last_id, $requested_type, $domain, $used_limit);

Do konference jsem to reportoval jiz 13.6.2012, 'Bug v prijimani udalosti'.

Pls zkontrolujte take automaticke testy, zda tam take neni problem

Actions #1

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

  • Status changed from New to In Progress
Actions #2

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

Kdyz jsem se na to dival, tak jsem zjistil, ze ve funkci nastal problem po propojeni tabulek 'clients' a 'events'. Proto server klientovi zasilala chybovou hlasku:

Server sent error message:: DBD::mysql::st execute failed: Unknown column 'hostname' in 'where clause' at /opt/warden-server/lib/Warden.pm line 340.

Chyba by mela byt opravena v revizi 323031f3.

Actions #3

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

  • Status changed from In Progress to Resolved
Actions #4

Updated by Pavel Kácha about 11 years ago

  • Status changed from Resolved to Feedback

Kostěji, proti jaké verzi jsi zkoušel? Tomáši, v jaké verzi jsi opravoval? Domluvte se pls a ověřte, že je opraveno.

Actions #5

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

Pavel Kácha wrote:

Kostěji, proti jaké verzi jsi zkoušel? Tomáši, v jaké verzi jsi opravoval? Domluvte se pls a ověřte, že je opraveno.

Ja jsme tuto vec opravoval ve verzi serveru 2.2. Ve verzi 2.1 jsem to nezkousel, jelikoz jsem nepredpokladal, ze bychom to jeste nejak opravovali. Muzu to ale klidne ozkouset a pripadne dodatecne opravit bez noveho releasu.

Actions #6

Updated by Michal Kostěnec about 11 years ago

Perfektni, ve verzi 2.2 se to zda byt ok;-)
Kazdopadne mi nejde do hlavy, proc chyba o delce jednoho znaku stale zustava na Vasem hlavnim serveru (at uz se jedna o jakoukoliv verzi) a tak nefunguje jedna ze zakladnich vlastnosti systemu.

Actions #7

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

Michal Kostěnec wrote:

Perfektni, ve verzi 2.2 se to zda byt ok;-)

Tak to je super.

Kazdopadne mi nejde do hlavy, proc chyba o delce jednoho znaku stale zustava na Vasem hlavnim serveru (at uz se jedna o jakoukoliv verzi) a tak nefunguje jedna ze zakladnich vlastnosti systemu.

OKa, podivam se na to, chybu z kodu odstranim, opravu otestuji a melo by byt vse v poradku. Je pak ale potreba rozmyslet, zdali se bude vydavat opravny plnohodnotny balicek, nebo alespon do balicku zabaleny patch.

Actions #8

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

  • Status changed from Feedback to Resolved

Chybu popsanou Kostejem se mi na produkcnim warden serveru (wardne.cesnet.cz) podarilo zreplikovat, opravit a opravu uspesne otestovat, takze vse uz by ted melo byt v poradku. Ted jeste jak se zachovat k teto opravne v ramci vydani balicku/balicku s patchem.

Actions #9

Updated by Pavel Kácha about 11 years ago

Michal:

Kazdopadne mi nejde do hlavy, proc chyba o delce jednoho znaku stale zustava na Vasem hlavnim serveru (at uz se jedna o jakoukoliv verzi) a tak nefunguje jedna ze zakladnich vlastnosti systemu.

Protože se vám to s Bodíkem pokud vím tehdy nedařilo zreplikovat.

Tom:
Který je to commit?

Které chování teď nefunguje, server posílá vše bez ohledu na nastavení? Pak se oprava dotkne devíti klientů.

Actions #10

Updated by Michal Kostěnec about 11 years ago

Pavel Kácha wrote:

Michal:

Kazdopadne mi nejde do hlavy, proc chyba o delce jednoho znaku stale zustava na Vasem hlavnim serveru (at uz se jedna o jakoukoliv verzi) a tak nefunguje jedna ze zakladnich vlastnosti systemu.

Protože se vám to s Bodíkem pokud vím tehdy nedařilo zreplikovat.

Nemyslim si;-) Osobne prijimaciho klienta nepouzivam.

Tom:
Který je to commit?

Které chování teď nefunguje, server posílá vše bez ohledu na nastavení? Pak se oprava dotkne devíti klientů.

Ano, posila vse.

Actions #11

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

Michal Kostěnec wrote:

Pavel Kácha wrote:

Michal:

Kazdopadne mi nejde do hlavy, proc chyba o delce jednoho znaku stale zustava na Vasem hlavnim serveru (at uz se jedna o jakoukoliv verzi) a tak nefunguje jedna ze zakladnich vlastnosti systemu.

Protože se vám to s Bodíkem pokud vím tehdy nedařilo zreplikovat.

Nemyslim si;-) Osobne prijimaciho klienta nepouzivam.

Tom:
Který je to commit?

Jak jsem psal, chyba je opravena na serveru warden.cesnet.cz kde bezi warden server v2.1. Cekam na rozhodnuti jak to dopadne s balickem. Opravu neni problem nekam commitnout.

Které chování teď nefunguje, server posílá vše bez ohledu na nastavení? Pak se oprava dotkne devíti klientů.

Ano, posila vse.

Po vcerejsi oprave uz server respektuje nastaveni received_own_events (dale ROE) a data, ktera byla nareportovana vlastni organizaci, klientovi, jez ma ROE nastavene na false neposle. Pokud ma klient ROE nastaveno na true, tyto data mu chodit budou.

Ve warden serveru v.2.2 toto chovani uz bylo opraveno drive.

Actions #12

Updated by Pavel Kácha about 11 years ago

Bylo by rozumné to commitnout jako vyvětvení 2.1 (tag warden-server-2.1), ale žádný balíček 2.1 errata už vydávat nebudeme, zbytečná práce navíc.

Actions #13

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

Pavel Kácha wrote:

Bylo by rozumné to commitnout jako vyvětvení 2.1 (tag warden-server-2.1), ale žádný balíček 2.1 errata už vydávat nebudeme, zbytečná práce navíc.

OKa, tim padem z tagu warden-server-2.1 udelat branch, opravit, commitnout a zatagovat jako warden-server-2.1-patch1?

Actions #14

Updated by Pavel Kácha about 11 years ago

Souhlas.

Actions #15

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

Hotovo, zalozil jsem novou branch pro dalsi pripadne opravy warden-server-2.1-patch (warden-server-2.1 uz nesla, jelikoz se shoduje s nazvem tagu final release), opravil jsem, commitnul, zatagoval jako warden-server-2.1-patch1 a pushnul na server. Cislo commitu teto opravy je a88c42c2.

Actions #16

Updated by Pavel Kácha about 11 years ago

  • Status changed from Resolved to Closed
  • Target version set to 2.1
Actions

Also available in: Atom PDF