Re: [lms] radius patch
No nietylko :) tam bylo INT (xx) NOT NULL DEFAULT ''.... i sie wywalał bo brakowalo wartości wiec wstawilem tam '0'
!DSPAM:49b849db253921703090906!
W dniu 12 marca 2009 00:30 użytkownik Przemysław Bryniak hacus@tlen.pl napisał:
No nietylko :) tam bylo INT (xx) NOT NULL DEFAULT ''.... i sie wywalał bo brakowalo wartości wiec wstawilem tam '0'
Robercie masz kolejny błąd przynajmniej u mnie po zaaplikowaniu patcha w tabeli nas pole ktore powinno sie nazywac nasname nazywa sie inet_ntoa(no.ipaddr) ;)
!DSPAM:49bbb156157468045311664!
Robercie masz kolejny błąd przynajmniej u mnie po zaaplikowaniu patcha w tabeli nas pole ktore powinno sie nazywac nasname nazywa sie inet_ntoa(no.ipaddr) ;)
dobrze Was miec :) u siebie modyfikowalem baze z palca i widze teraz ze pare spraw mi potem umknelo w patchu. w poniedzialek wszystko poprawie i wystawie nowego patcha mam nadzieje juz bez niespodzianek
Robert CyberM
Druga sprawa grupy radiusa powinny byc zachowane, wtedy byc moze udalo by sie zrobic widok radcheck i inne bo w authorize_check, powinno sie zwraacac tylko username i password, reszta powinna byc trzymana w groupach, kolejna sprawa zapytanie zwraca User-Password, wystarczy samo Password
trzecia sprawa twoje zapytanie z authorize_check_query zwraca mi bład, jak zadam je z palca bazie: SELECT id, lower(name) as UserName , 'User-Password' as Attribute , passwd as Value, '==' as op FROM nodes WHERE name = 'GACEK-PC' UNION SELECT id, lower(name) as UserName , 'Simultaneous-Use' as Attribute, '1' as Value, ':=' as op FROM nodes WHERE name = 'GACEK-PC' UNION SELECT 0 AS id, 'GACEK-PC' AS UserName, 'Max-Octets' AS Attribute, CONCAT(ROUND(COALESCE(x.dlimit, y.dlimit))) AS Value, ':=' AS op FROM (SELECT n.id, MIN(n.name) AS name, SUM(t.dlimit/o.cnt) AS dlimit FROM nodeassignments na JOIN assignments a ON (na.assignmentid = a.id) JOIN tariffs t ON (a.tariffid = t.id) JOIN nodes n ON (na.nodeid = n.id) JOIN (SELECT assignmentid, COUNT(*) AS cnt FROM nodeassignments GROUP BY assignmentid) o ON (o.assignmentid = na.assignmentid)WHERE (a.datefrom <= unix_timestamp() OR a.datefrom = 0) AND (a.dateto > nix_timestamp() OR a.dateto = 0) AND a.suspended = 0 AND n.name = 'GACEK-PC' GROUP BY n.id) x RIGHT JOIN (SELECT SUM(t.dlimit)/o.cnt AS dlimit FROM assignments a JOIN tariffs t ON (a.tariffid = t.id) JOIN nodes n ON (a.customerid = n.ownerid) JOIN (SELECT COUNT(*) AS cnt, ownerid FROM nodes WHERE NOT EXISTS(SELECT 1 FROM nodeassignments, assignments a WHERE assignmentid = a.id AND nodeid = nodes.id AND a.suspended = 0 AND (a.dateto > unix_timestamp() OR a.dateto = 0)) GROUP BY ownerid) o ON (o.ownerid = n.ownerid) WHERE (a.datefrom <= unix_timestamp() OR a.datefrom = 0) AND (a.dateto > unix_timestamp() OR a.dateto = 0) AND a.suspended = 0 AND t.dlimit != '0' AND NOT EXISTS (SELECT 1 FROM nodeassignments WHERE assignmentid = a.id) AND n.name = 'GACEK-PC' GROUP BY n.id) y ON (1=1);
ERROR 1064 (42000): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' AND n.name = 'GACEK-PC' GROUP BY n.id) y ON (1=1)' at line 1
Pozdrawiam :)
O to wtym chodzi, bez beta testerow programisci byli by nikim ;)
!DSPAM:49bbbb41163611614647931!
Dnia sobota, 14 marca 2009 15:12, Michał Gacek napisał:
Druga sprawa grupy radiusa powinny byc zachowane, wtedy byc moze udalo by sie zrobic widok radcheck i inne bo w authorize_check, powinno sie zwraacac tylko username i password, reszta powinna byc trzymana w groupach,
to jest jednak dosc dyskusyjne i zalezne od przyzwyczajen, jedni sporo informacji trzymaja w grupach inni wogole nie uzywaja grup
kolejna sprawa zapytanie zwraca User-Password, wystarczy samo Password
a widzisz nie prawda nie zawsze AUTH-TYPE=local :-)
trzecia sprawa twoje zapytanie z authorize_check_query zwraca mi bład,
tutaj #SOA1 a testowane bylo ze sto tysiecy razy (na mysql), oczywiscie zerkne dla pewnosci ale nie przypominam sobie zebym cos zmienial a orginalny plik pracuje produkcyjnie (passwd do bazy sie tylko zmienil ;-) )
O to wtym chodzi, bez beta testerow programisci byli by nikim ;)
do rangi programisty narazie nawet nie staram sie aspirowac he he ale faktem jest ze jak sie robi samemu to czasem w wersji finalnej wychodza jakies drobiazgi ktorych w wersji roboczej nie bylo (przeciez dziala). jak kilka osob zerknie na kod to latwiej znalesc jakies uchybienia czy drobne (badz nie) niedorobki. wiem juz na pewno ze nastepnym razem dokladniej wszystko przetestuje zanim zdecyduje sie upublicznic
Robert CyberM
W dniu 14 marca 2009 16:05 użytkownik Robert CyberM cyberm@sarocom.net napisał:
a widzisz nie prawda nie zawsze AUTH-TYPE=local :-)
no gdy uzywa sie mysql prawie zawsze no chyba ze radius pozniej zapytuje sie innego radiusa ;), o ile dobrze rozumiem ;)
tutaj #SOA1 a testowane bylo ze sto tysiecy razy (na mysql), oczywiscie zerkne dla pewnosci ale nie przypominam sobie zebym cos zmienial a orginalny plik pracuje produkcyjnie (passwd do bazy sie tylko zmienil ;-) )
no puscilem je z palca przez klienta mysql i takie cos mi wywalalo chcialem zobaczyc co dokladnie wyciagasz z bazy dla radiusa, bo nie czuje sie na silach aby przeczytac to zapytanie ze zrozumieniem ;) dodam ze nie mam nodeassignments, bo lacze przydzielam per klient nie per ip... jezeli ma to jakies znaczenie, a nie powinno ;)
Pozdrawiam
!DSPAM:49bbe5c9190312079114581!
uczestnicy (3)
-
Michał Gacek -
Przemysław Bryniak -
Robert CyberM