[Phplogcon-dev] phplogcon without user managment

[Michael Meckelein]

> So what do you think of the 1.2.6_bgs?

First I took a look the online demo on your page. Works great.

Installing it in my lab caused some trouble :(

After adapting config.php I got an "Improper session table formatting"
(btw: formating is misspelled in your version) error. 

I think there is a bug in DB_PEAR_sess_drv.php in line 155.

Replaced:
switch( $field['name'] )
with:
switch( _DBSESS_FILED_PRE.$field['name'] )

After this change it works for me, too. I have to admit that I have
taken a quick view only, no testing. However, please let me add some
notes here: 

1) You use the same error message twice:

echo( "Improper session table formating. Please contact
administrator.<br>");

is used for check

if( count( $info ) < 3 )

and for

if( $fld_cnt < 3 )

in DB_PEAR_sess_drv.php. Probably it would be better to attach a unique
error number or something similar to the error messages in order to make
trouble shooting easier.

2) scripts/session_table.sql contains no valid sql statement.

I guess the following is sufficient

CREATE TABLE `sess_Users` (
  `sess_id` text NOT NULL,
  `sess_data` text NOT NULL,
  `sess_expire` datetime NOT NULL
) 

> You can use as much/little as you want and i can help put what you
need
> into
> 1.2.1/1.2.2.

We will release the current cvs version as 1.2.2. The only fix adapted
from you is the '%' security fix. All other should be considered in the
next release.
 
> Also, i played with the Auth_PrefManager from PEAR, it works okay
maybe
> that
> sould be considered for a futur release of phpLogCon along with
PEAR:DB
> which works nicely.
> 
> PEAR::Auth_PrefManager lacks one function
Auth_PrefManager::getUserPrefs(
> string userId ).  Otherwise it would work great for all the Quick
filters
> and definable filters, and maybe even supporting multiple DB's/Tables
for
> log viewing.
> 
> I'll probably write the function and email it to them.

Sounds really useful.

Michael