#99 allowing extra attributes in RADIUS-based modules

[Victor Gamov]

Hi guys!

The following patch add "extra attributes" feature in AUTH_RADIUS and URI_RADIUS modules.
New config parameter "radius_extra" for patched modules added.

Currently we use this feature to add really useful attributes into RADIUS request, for example:

modparam("auth_radius", "radius_extra", "Called-Station-Id=$tu;Calling-Station-Id=$fu")

This code based on "extra accounting" feature from ACC module. Many thanks to Ramona!

[Victor Gamov]

Logged In: YES
user_id=1625470
Originator: YES

oops! incorrect patch attached. wait a moment!

[Victor Gamov]

Logged In: YES
user_id=1625470
Originator: YES

right (I hope :-) ) archive attached
File Added: patch-extra_attrs.tar

[Victor Gamov]

Logged In: YES
user_id=1625470
Originator: YES

Hi All !

is this patch tested somebody ?

if any problems with it please email me and I'll try to help or make improovment

Thanks

[Victor Gamov]

[Victor Gamov]

Logged In: YES
user_id=1625470
Originator: YES

RADIUS attributes with type IP-Address (like Framed-IP-Address) processed correctly now
(patch for 1.2.1 still)
File Added: patch-radius_extra_attrs.tar

[Juha Heinanen]

Logged In: YES
user_id=1332122
Originator: NO

i looked at the patches and have a couple of comments:

- in order to cleanly separate the new code, i would like to include a test if extra attributes are present to sterman.c and checks.c before add_extra_2_radius is called.

- uri_radius module includes extra_attrs.h from auth_radius module. i don't know if that is according to openser policy. if there is common code, perhaps it should be included in core.

- documentation diffs should also be included.

-- juha