Skip Menu |

This queue is for tickets about the Net-Radius CPAN distribution.

Report information
The Basics
Id: 13998
Status: resolved
Worked: 30 min
Priority: 0/
Queue: Net-Radius

People
Owner: luismunoz [...] cpan.org
Requestors: j7 [...] id.ru
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 1.44
Fixed in: (no value)



Subject: patch to support multiple subattributes in one VSA
There is a simple patch to support multiple attr/value pairs in one VSA value. (At least one known client does send such a messages - accounting in BroadSoft.) It's ok according to RFC 2825 5.26 final clause. Also, it could be nice to make these modules more robust, like checking for errors in readline subroutine of Dictionary.pm and so on
304c304,308 < my ($vid, $vtype, $vlength) = unpack "N C C", $value; --- > my ($vid) = unpack "N", $value; > substr ($value, 0, 4) = ""; > > while (length($value)) { > my ($vtype, $vlength) = unpack "C C", $value; 316,317c320,321 < ($vid, $vtype) = unpack "N N", $value; < $vvalue = unpack "xxxx xxxx a${\($length-10)}", $value; --- > ($vtype) = unpack "N", $value; > $vvalue = unpack "xxxx a${\($length-10)}", $value; 319c323 < $vvalue = unpack "xxxx x x a${\($vlength-2)}", $value; --- > $vvalue = unpack "x x a${\($vlength-2)}", $value; 338a343,346 > > substr($value, 0, $vlength) = ""; > } >
On Wed Aug 03 08:11:47 2005, guest wrote: Show quoted text
> There is a simple patch to support multiple attr/value pairs in one > VSA value. (At least one known client does send such a messages - > accounting in BroadSoft.) It's ok according to RFC 2825 5.26 final > clause.
The patch was added. A new release is pending regression testing at my site. Can you forward one or more packet captures of multi-pair VSAs? I want to write a suite of packet parsing tests to include in the distribution, and need some exotic samples for this. Show quoted text
> Also, it could be nice to make these modules more robust, like > checking for errors in readline subroutine of Dictionary.pm and so on
A number of changes in this direction have been going in the recent versions (>1.5). Regards -lem
Subject: Re: [rt.cpan.org #13998] patch to support multiple subattributes in one VSA
Date: Fri, 12 Jan 2007 22:48:18 +0300
To: via RT <bug-Net-Radius [...] rt.cpan.org>
From: Mtv Europe <j7 [...] id.ru>
Hello Luis! Tuesday, January 09, 2007, 12:56:24 AM, you wrote: Show quoted text
>> There is a simple patch to support multiple attr/value pairs in one >> VSA value.
> Can you forward one or more packet captures of multi-pair VSAs? I want to write a suite of > packet parsing tests to include in the distribution, and need some exotic samples for this.
I've attached the test code with sample packet inside, required dictionary and correct output. Thank you very much for you work, I'll be happy to help. With best regards, -- Mtv Europe http://www.frox25.no-ip.org/~mtve/
Download dictionary.broadsoft
application/octet-stream 10.3k

Message body not shown because it is not plain text.

Message body is not shown because sender requested not to inline it.

Download output
application/octet-stream 1.7k

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #13998] patch to support multiple subattributes in one VSA
Date: Fri, 12 Jan 2007 18:47:30 -0400
To: bug-Net-Radius [...] rt.cpan.org
From: "Luis E. Muñoz" <luismunoz [...] cpan.org>
On Jan 12, 2007, at 3:47 PM, j7@id.ru via RT wrote: Show quoted text
> Thank you very much for you work, I'll be happy to help.
Hey, thanks a lot! This will be getting into the next release. Any chance you can send me a tcpdump packet capture for the RADIUS transaction? You can use this command tcpdump -s 0 -c 1 -w radius.dump -e 'udp and portrange 1812-1813' Thanks a lot. -lem
On Fri Jan 12 14:47:32 2007, j7@id.ru wrote: Show quoted text
> I've attached the test code with sample packet inside, required > dictionary and correct output.
Hi there I managed to reconstruct the binary packet out of the test you provided. I added the dictionary to the next release and built a standard test for the distribution. The dict you sent was lacking a 'BroadSoft' at one of the VSAs. I added that to the dist code. Thanks for your help. -lem