xinbc / jdiameter

Automatically exported from code.google.com/p/jdiameter
0 stars 0 forks source link

New method to set Origin-State-Id AVP value as CER and DWR #38

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Detailed information about the new feature request

Besides the current method [void CreditControlMessage.setOriginStateId(long 
originStateId)] a new one taking no parameters shall be added, for example 
[void CreditControlMessage.setOriginStateId()]. If this method with no 
parameters is called then the stack will set Origin-State-Id AVP value to the 
value used for CER and DWR.

The current method which sets the CCR Origin-State-Id AVP with the value from 
the parameter originStateId shall be kept as is. This behavior is necessary for 
services implementing relay, proxy, and redirect Diameter agents because they 
forward the CCR Origin-State-Id AVP from the incoming CCR and not the one set 
by the stack in CER/DWR.

And this method with no parameters shall be used optional as the older one.

1. What motivated the request:

Services acting as a Diameter client and sending CCR requests shall use 
optionally Origin-State-Id AVP with the same id as CER/DWR

2. Link to related public forum or mailing list discussion:

https://groups.google.com/d/topic/mobicents-public/Y1C22ZprkRI/discussion

3. Describe functionality and API (if applicable)

Above

Original issue reported on code.google.com by laurenti...@gmail.com on 7 Feb 2013 at 11:38

GoogleCodeExporter commented 9 years ago
Patch solving this issue uploaded

There are two patch files representing the same solution, the first 
diff.diff.patch being taken the Cygwin console "diff" utility and the second 
being taken by "git diff" command

As shown in the attached pictures the Origin-State-Id is the same in CER, DWR, 
and CCR Diameter requests

Original comment by laurenti...@gmail.com on 5 Mar 2013 at 4:30

Attachments:

GoogleCodeExporter commented 9 years ago
By the way, I was using the fallowing version

    <groupId>org.mobicents.servers.jainslee.diameter</groupId>
    <artifactId>mobicents-slee-diameter</artifactId>
    <version>2.8.0-SNAPSHOT</version>

Original repository clone

    git clone https://code.google.com/p/jain-slee.diameter/

Original comment by laurenti...@gmail.com on 5 Mar 2013 at 4:33