[RADIATOR] Typo on radpwtst?
Heikki Vatiainen
hvn at open.com.au
Mon Oct 27 14:46:06 UTC 2025
On 22.10.2025 20.36, Bruno Tiago Rodrigues via radiator wrote:
> Does line 1411 on radpwtst have a typo? Shouldn't it be Change-Filter-
> Request ?
>
> return "\000" x 16 if
> $code eq 'Accounting-Request' ||
> $code eq 'Disconnect-Request' ||
> $code eq 'Change-Filter-Requet';
Hello Bruno, as Stefan already wrote, you're correct, it's a typo.
This got broken when some lines were moved in radpwtst, but fortunately
those lines are no longer needed at all. Long time ago they were, but
now the Radius fixed header is set differently. Initialising it to
zeroes is no longer needed at this point.
The function that does the above is still used if you want to, for
example, test duplicate handling and want to send the exact same request
again. The function allows setting the fixed authenticator in
Access-Request messages. However, zero initialisation is redundant.
In short: thanks for finding and reporting code that is no longer used
and can be removed :)
Heikki
--
Heikki Vatiainen
Radiator Software, makers of Radiator
Visit radiatorsoftware.com for Radiator AAA server software
More information about the radiator
mailing list