2.0.0-pre sqlippool

[Peter Nixon]

On Wed 27 Jun 2007, Hugh Messenger wrote:
> Peter Nixon [mailto:listuser at peternixon.net] said
>
> > From now on (20 min ago), MySQL should be a fully support dialect for
> > rlm_sqlippool so the table is part of the normal schema and the queries
> > _should_ work by default. Please send patches for anything that is still
> > broken as I don't test against MySQL myself.
>
> Okie Dokie.
>
> > > It seems to want %{check:Pool-Name}.  Which is weird in itself,
> > > because 'man unlang' doesn't mention a 'check' list type, it should be
> >
> > 'request'.
> >
> > > But that doesn't work!  The only thing that seems to work in 2.0.0 is
> > > %{check:Pool-Name}.  Can't just be %{Pool-Name}, either.
> >
> > fixed. Alan changed things after I wrote it ;-)
>
> I just noticed the 'man unlang' paragraph about 'check' being a synonym
> for 'control'.  Did that just get added, or was I being blind first time
> round? I do try and read the docs before posting here, and I'd hate to
> think I missed that.
>
> > > 2) In the allocate-update query ...
>
> [snip]
>
> > > ... I have to change the %{FramedIPAddress} to %I to get this query to
> > > work.
> >
> > I dont see this problem...
>
> I'll look at that again.  Alan pointed out the typo, but that was in this
> (email) copy of the text, where I'd copied and (incorrectly) edited my
> working version of the file.  However, it's possible I may have done the
> same while testing, so I'll get back to you on that one.
>
> > OK. I have significantly updated the mysql queries according to the
> > second set of queries at:
> > http://wiki.freeradius.org/Talk:Rlm_sqlippool
> >
> > Can you please update to current cvs and tell me if anything else needs
> > fixing?
>
> First problem is the new version is Windows Unfriendly, as it uses all
> lower case column names in the queries, although the schema in mysql.sql
> still uses mixed case names.  Which of course you are committed to using
> anyway, because of awkward people like me who already have Windows MySQL
> radippool's, and are unwilling to enable the 'ignore case' option.  :)
>
> I'll friendly-up the new mysql-ippool-dialup.conf and send you a copy.

Sounds like a plan :-)

-- 

Peter Nixon
http://www.peternixon.net/
PGP Key: http://www.peternixon.net/public.asc