Re: rlm_sql: zero session length

Why is this if statement there anyway, some buggy NASes?

Almost certainly.

If so, why not just let user handle it in unlang?

The rlm_sql module was written before unlang, so that's why the code is there.

IMHO that needs to be removed, or if written for a good reason, made configurable. With rfc-compliant default:)

I'll remove it. It's easy enough to do the check in unlang. Arran Cudbard-Bell <a.cudbardb@freeradius.org> FreeRADIUS Development Team