18 Dec
2020
18 Dec
'20
9:15 a.m.
On Dec 17, 2020, at 10:23 PM, Arran Cudbard-Bell <a.cudbardb@freeradius.org> wrote:
I've reverted it. Breaking changes in point releases are not appropriate. This would break the majority of the users of rlm_rest who inspect the values of enumeration attributes.
Sure. The other argument here is that even if Tunnel-Medium-Type is of type "integer", the values of those integers have no meaning. You're not adding together the values. Instead, the names of the values have meaning. So the recommendation is for the REST back-end to *not* parse Tunnel-Medium-Type as "integer". Because that doesn't give you any information. Alan DeKok.