[GH-ISSUE #48] radius acct class attribute not working (with fix) #28

New issue

Closed

opened 2026-03-04 14:52:30 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 14:52:30 +03:00

Owner

Originally created by @sophana on GitHub (Oct 23, 2025).
Original GitHub issue: https://github.com/f00b4r0/uspot/issues/48

I noticed that radius accounting requests didn't include the Class attribute. The fix is simple and could be tested:

diff --git a/files/usr/share/uspot/uspot.uc b/files/usr/share/uspot/uspot.uc
index 500bc87..3beb5a4 100755
--- a/files/usr/share/uspot/uspot.uc
+++ b/files/usr/share/uspot/uspot.uc
@@ -263,8 +263,8 @@ function radius_acct(uspot, mac, payload) {
                        }
                }
        }
-       if (client.data?.radius?.reply?.Class)
-               payload.Class = client.data.radius.reply.Class;
+       if (client.radius?.reply?.Class)
+               payload.Class = client.radius.reply.Class;
 
        radius_call(payload);
 }

Originally created by @sophana on GitHub (Oct 23, 2025). Original GitHub issue: https://github.com/f00b4r0/uspot/issues/48 I noticed that radius accounting requests didn't include the Class attribute. The fix is simple and could be tested: ``` diff --git a/files/usr/share/uspot/uspot.uc b/files/usr/share/uspot/uspot.uc index 500bc87..3beb5a4 100755 --- a/files/usr/share/uspot/uspot.uc +++ b/files/usr/share/uspot/uspot.uc @@ -263,8 +263,8 @@ function radius_acct(uspot, mac, payload) { } } } - if (client.data?.radius?.reply?.Class) - payload.Class = client.data.radius.reply.Class; + if (client.radius?.reply?.Class) + payload.Class = client.radius.reply.Class; radius_call(payload); } ```