[OpenSIPS-Devel] [ opensips-Patches-2046495 ] Adding Acct-Seesion-ID to Access-Request (auth_radius)

[SourceForge.net]

Patches item #2046495, was opened at 2008-08-11 17:10
Message generated for change (Settings changed) made by bogdan_iancu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=2046495&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: trunk
>Status: Closed
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Dror Wald (dror_wald)
Assigned to: Bogdan-Andrei Iancu (bogdan_iancu)
Summary: Adding Acct-Seesion-ID to Access-Request (auth_radius)

Initial Comment:
This patch to auth_radius module adds Acct-Session-Id to Access-Request According to RFC2866: 
"
5.5.  Acct-Session-Id
   Description
      This attribute is a unique Accounting ID to make it easy to match
      start and stop records in a log file.  The start and stop records
      for a given session MUST have the same Acct-Session-Id.  An
      Accounting-Request packet MUST have an Acct-Session-Id.  An
      Access-Request packet MAY have an Acct-Session-Id; if it does,
      then the NAS MUST use the same Acct-Session-Id in the Accounting-
      Request packets for that session.
"

The purpose of this patch is to create a relationship between an auth and acct request, based on SIP CALL-ID value (Acct-Session-Id attribute). 





----------------------------------------------------------------------

>Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2008-08-30 14:54

Message:
Logged In: YES 
user_id=1275325
Originator: NO

Hi Dror,

I applied your patch on the trunk, but with some small changes: CALLID is
mandatory in SIP, so, instead of setting "none", an error is generated. And
there is no need to allocate separate pkg memory for the value - anthow, in
the original version, that was a mem leak.

Thanks and regards,
Bogdan

----------------------------------------------------------------------

Comment By: Dror Wald (dror_wald)
Date: 2008-08-13 14:54

Message:
Logged In: YES 
user_id=2176865
Originator: YES

File Added: sterman.c.diff

----------------------------------------------------------------------

Comment By: Dror Wald (dror_wald)
Date: 2008-08-13 14:53

Message:
Logged In: YES 
user_id=2176865
Originator: YES

File Added: authrad_mod.c.diff

----------------------------------------------------------------------

Comment By: Dror Wald (dror_wald)
Date: 2008-08-11 17:11

Message:
Logged In: YES 
user_id=2176865
Originator: YES

File Added: authrad_mod.c.diff

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=2046495&group_id=232389