Bug#454660:
Hello, On 12/07/07 12:34, Julien BLACHE wrote: Rene Mayorga [EMAIL PROTECTED] wrote: Hi, BTW regarding to your previous mail; I'm using postgresql and Radius accounting with 1.2 and there is no problem, I did'nt try using it with Diameter. Reading the code, it handles that fine. The only problem I have is that the main openser package will now depend on the radius libs. to give more details here ... Indeed, the acc can do recording to multiple backends in parallel: syslog, sql server, radius and diameter (this is more or less unmaintained) -- where is done actually, is given by module's parameters. By default, acc module is compiled only with syslog and sql backends support. The reason, as Julien said, radius support will introduce a dependence of libradiusclient-ng which is not available as package in all OS supported by openser. There are discussions to eliminate this dependency by having a new module as driver to communicate with radius server, and the other modules to use a generic interface exported by this module. Same is for SQL backends, there are drivers specific per implementation (now: mysql, postgres, unixodbc) -- the modules that need access to SQL servers do not depend on client libraries. Now it is late to get it in upcoming 1.3, so it is postponed for the following release. Cheers, Daniel I have commit access at pkg-voip svn if you're busy I can add the proper patch there. I'll take care of it. JB. -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#454660:
Daniel-Constantin Mierla [EMAIL PROTECTED] wrote: Hi, Indeed, the acc can do recording to multiple backends in parallel: syslog, sql server, radius and diameter (this is more or less unmaintained) -- where is done actually, is given by module's parameters. By default, acc module is compiled only with syslog and sql backends support. The reason, as Julien said, radius support will introduce a dependence of libradiusclient-ng which is not available as package in all OS supported by openser. Based on this I'll just enable radius and not diameter. I've found and fixed a little bug in the diameter support while I was at it. JB. -- Julien BLACHE [EMAIL PROTECTED] | Debian, because code matters more Debian GNU/Linux Developer| http://www.debian.org Public key available on http://www.jblache.org - KeyID: F5D6 5169 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#454660:
Rene Mayorga [EMAIL PROTECTED] wrote: Hi, BTW regarding to your previous mail; I'm using postgresql and Radius accounting with 1.2 and there is no problem, I did'nt try using it with Diameter. Reading the code, it handles that fine. The only problem I have is that the main openser package will now depend on the radius libs. I have commit access at pkg-voip svn if you're busy I can add the proper patch there. I'll take care of it. JB. -- Julien BLACHE [EMAIL PROTECTED] | Debian, because code matters more Debian GNU/Linux Developer| http://www.debian.org Public key available on http://www.jblache.org - KeyID: F5D6 5169 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#454660:
Daniel-Constantin Mierla [EMAIL PROTECTED] wrote: Hi, In case you enable radius support in debian, then all radius-related modules (auth_radius, avp_radius, ...) can be introduced all together in the main openser package. I'd really like to avoid having the main openser package depend on the radius libs, but adding new packages to work around that just isn't worth it. As you've told in your previous mail, this should be a transient state until the next openser release when the abstraction layer will be added, so I'm pretty much OK with that situation until then. JB. -- Julien BLACHE [EMAIL PROTECTED] | Debian, because code matters more Debian GNU/Linux Developer| http://www.debian.org Public key available on http://www.jblache.org - KeyID: F5D6 5169 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#454660:
Hello, On 12/07/07 13:20, Julien BLACHE wrote: Daniel-Constantin Mierla [EMAIL PROTECTED] wrote: Hi, Indeed, the acc can do recording to multiple backends in parallel: syslog, sql server, radius and diameter (this is more or less unmaintained) -- where is done actually, is given by module's parameters. By default, acc module is compiled only with syslog and sql backends support. The reason, as Julien said, radius support will introduce a dependence of libradiusclient-ng which is not available as package in all OS supported by openser. Based on this I'll just enable radius and not diameter. We preferred to keep the main openser package out of exotic dependencies. However, this might not be exotic in debian as the radiusclient-ng library is in the distro. It is your decision how to go. In case you enable radius support in debian, then all radius-related modules (auth_radius, avp_radius, ...) can be introduced all together in the main openser package. Another option would be to remove acc from main package, and make two new packages: openser-acc-module -- having support for syslog and sql openser-acc-radius module -- having support for syslog, sql and radius The two should conflict each other. As personal preference, will be the second -- I am not using RADIUS that much. I've found and fixed a little bug in the diameter support while I was at it. Thanks for the fix. Cheers, Daniel JB. -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#454660: acc.so module doesn't support radius
Package: openser Version: 1.2.2-3 Severity: normal Hi, The acc.so module is not compiled with radius support here is the error output when you pass radius_* params to acc: (9359) parse error (18,13-14): failed to load module 0(9359) set_mod_param_regex: parameter radius_config not found in module acc Cheers --- System information. --- Architecture: amd64 Kernel: Linux 2.6.23.9 Debian Release: lenny/sid 500 unstableftp.us.debian.org 1 experimentalftp.us.debian.org --- Package information. --- Depends (Version) | Installed =-+-= libc6(= 2.6.1-1) | 2.7-3 adduser | 3.105 -- Rene Mauricio Mayorga | GPG: A209C305 http://rmayorga.org | -- 08B6 58AB A691 DD56 C30B 8D37 8040 19FA A209 C305 signature.asc Description: Esta parte del mensaje está firmada digitalmente
Bug#454660: acc.so module doesn't support radius
Rene Mayorga [EMAIL PROTECTED] wrote: Hi, The acc.so module is not compiled with radius support here is the error output when you pass radius_* params to acc: (9359) parse error (18,13-14): failed to load module 0(9359) set_mod_param_regex: parameter radius_config not found in module acc Indeed, it defaults to SQL accounting. Do you know if it's possible to enable SQL, Radius and Diameter all at the same time ? JB. -- Julien BLACHE - Debian GNU/Linux Developer - [EMAIL PROTECTED] Public key available on http://www.jblache.org - KeyID: F5D6 5169 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#454660:
Rene Mayorga [EMAIL PROTECTED] wrote: Hi, Anyway, I'm not source is this worth an upload, 1.3 is almost out, and the bug doesn't exist there acc.so in 1.3 isn't radius-enabled either in the current state. I'll probably enable the three methods in the next upload. Or were you only reporting the warning it outputs in this case? JB. -- Julien BLACHE - Debian GNU/Linux Developer - [EMAIL PROTECTED] Public key available on http://www.jblache.org - KeyID: F5D6 5169 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#454660:
found 454660 1.2.2-3 notfound 454660 1.3.0~svn20071129-1 tags 454660 patch thanks I'm attaching a simple patch which to fix this. Anyway, I'm not source is this worth an upload, 1.3 is almost out, and the bug doesn't exist there Cheers -- Rene Mauricio Mayorga | GPG: A209C305 http://rmayorga.org | -- 08B6 58AB A691 DD56 C30B 8D37 8040 19FA A209 C305 12_radius_support.dpatch Description: application/shellscript signature.asc Description: Esta parte del mensaje está firmada digitalmente
Bug#454660:
found 454660 1.3.0~svn20071129-1 thanks El jue, 06-12-2007 a las 23:15 +0100, Julien BLACHE escribió: Rene Mayorga [EMAIL PROTECTED] wrote: Hi, Anyway, I'm not source is this worth an upload, 1.3 is almost out, and the bug doesn't exist there acc.so in 1.3 isn't radius-enabled either in the current state. I'll probably enable the three methods in the next upload. Indeed, when I build 1.3 package from experimental I has ENABLE_RADIUS_ACC=true as env var at my work station. I'm market the bug as found at 1.3. Or were you only reporting the warning it outputs in this case? JB. BTW regarding to your previous mail; I'm using postgresql and Radius accounting with 1.2 and there is no problem, I did'nt try using it with Diameter. I have commit access at pkg-voip svn if you're busy I can add the proper patch there. Cheers -- Rene Mauricio Mayorga | GPG: A209C305 http://rmayorga.org | -- 08B6 58AB A691 DD56 C30B 8D37 8040 19FA A209 C305 signature.asc Description: Esta parte del mensaje está firmada digitalmente