#99 allowing extra attributes in RADIUS-based modules

ver 1.2.x
open
nobody
modules (140)
5
2007-06-28
2007-06-28
Victor Gamov
No

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!

Discussion

  • Victor Gamov
    Victor Gamov
    2007-06-28

    Logged In: YES
    user_id=1625470
    Originator: YES

    oops! incorrect patch attached. wait a moment!

     
  • Victor Gamov
    Victor Gamov
    2007-06-28

    Logged In: YES
    user_id=1625470
    Originator: YES

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

     
  • Victor Gamov
    Victor Gamov
    2007-08-14

    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
    2007-09-22

    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
    Juha Heinanen
    2008-04-19

    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