rlm_sql: zero session length

[Arran Cudbard-Bell]

> 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 at freeradius.org>
FreeRADIUS Development Team