rlm_sql.c in 2.0.0-pre2

[Alexander Serkin]

Hi,
Is the "read_groups" configuration paramter reading strings 
intentionally removed from rlm_sql.c? Why?

Let me suggest a patch:

*** rlm_sql.c.orig   2007-05-15 14:10:35.000000000 +0400
--- rlm_sql.c       2007-06-18 19:46:59.000000000 +0400
***************
*** 57,62 ****
--- 57,64 ----
          offsetof(SQL_CONFIG,tracefile), NULL, SQLTRACEFILE},
         {"readclients", PW_TYPE_BOOLEAN,
          offsetof(SQL_CONFIG,do_clients), NULL, "no"},
+         {"read_groups", PW_TYPE_BOOLEAN,
+          offsetof(SQL_CONFIG,read_groups), NULL, "yes"},
         {"deletestalesessions", PW_TYPE_BOOLEAN,
          offsetof(SQL_CONFIG,deletestalesessions), NULL, "yes"},
         {"num_sql_socks", PW_TYPE_INTEGER,


read_groups is checked on line 959 of rlm_sql.c, but it's not set 
anywhere before.

thanks,
-- 
Alexander