Bug #2224 for CGI-Session: Ability to specify MySQL column names

Thu Mar 13 13:00:32 2003 Guest - Ticket created

Subject:

Ability to specify MySQL column names

I would like to adapt CGI::Session::MySQL to my own, existing session table. I hope to make my existing code just a wrapper around CGI::Session. However, for backward compatibility, I don't want to change my column names, which are "session_key", "session_time", and "session_data". I request that the MySQL column names become modifiable constants, instead of hardcoded as "id" and "a_session". This can be easily accomplished by the patch that I will attach in a moment.

Thu Mar 13 13:02:08 2003 Guest - Correspondence added

From:

chris [...] clotho.com

Here is a patch to implement the wish that I just submitted. I have not tested it, but it's so simple and obvious that I think it should work.

Download patch
application/octet-stream 2.6k

Message body not shown because it is not plain text.

Fri Mar 14 13:42:38 2003 sherzodr [...] handalak.com - Correspondence added

From:	"Sherzod Ruzmetov" <sherzodr [...] handalak.com>
To:	<bug-CGI-Session [...] rt.cpan.org>
Subject:	RE: [cpan #2224] Ability to specify MySQL column names
Date:	Fri, 14 Mar 2003 13:42:41 -0500
RT-Send-Cc:

Hi, Thanks for the patch. I'll make sure to apply it for the next release of the library. sherzod : : : This message about CGI-Session was sent to you by guest : <> via rt.cpan.org : : Full context and any attached attachments can be found at: : <URL: https://rt.cpan.org/Ticket/Display.html?id=2224 > : : I would like to adapt CGI::Session::MySQL to my own, : existing session table. I hope to make my existing code : just a wrapper around CGI::Session. However, for : backward compatibility, I don't want to change my column : names, which are "session_key", "session_time", and : "session_data". : : I request that the MySQL column names become modifiable : constants, instead of hardcoded as "id" and "a_session". : This can be easily accomplished by the patch that I will : attach in a moment. : : :

Wed Apr 21 21:04:23 2004 Guest - Correspondence added

From:

mark [...] summersault.com

[guest - Thu Mar 13 13:00:32 2003]: Show quoted text

> I would like to adapt CGI::Session::MySQL to my own, existing session > table. I hope to make my existing code just a wrapper around > CGI::Session. However, for backward compatibility, I don't want to > change my column names, which are "session_key", "session_time", > and "session_data". > > I request that the MySQL column names become modifiable constants, > instead of hardcoded as "id" and "a_session". This can be easily > accomplished by the patch that I will attach in a moment.

Chris, I have some interest in adding support for custom column names to CGI::Session::PureSQL. Do you think this driver would appeal to you if I did that? It is different than the ::MySQL module because it stores data in the usual SQL way: one value per column. OTOH, Perhaps you could use the current data model of either ::MySQL or ::PureSQL, and create a view the column names you want. Mark

Wed Feb 09 04:09:46 2005 sherzodr [...] cpan.org - Status changed from 'new' to 'stalled'

Fri Mar 03 20:10:13 2006 MARKSTOS [...] cpan.org - Correspondence added

On Thu Mar 13 13:00:32 2003, guest wrote: Show quoted text

> I would like to adapt CGI::Session::MySQL to my own, existing session > table. I hope to make my existing code just a wrapper around > CGI::Session. However, for backward compatibility, I don't want

to Show quoted text

> change my column names, which are "session_key", "session_time", > and "session_data". > > I request that the MySQL column names become modifiable constants, > instead of hardcoded as "id" and "a_session". This can be easily > accomplished by the patch that I will attach in a moment.

Chris, Are you still interested this ancient suggestion for CGI::Session? I agree it's a good idea and am helping Sherzod maintain CGI::Session now. If so, I think we could accept a patch against the 4.x branch, using the OO-style API. The patch would need docs and automated tests to go in. I expect to put out another release in the next couple of weeks. Thanks! Mark

Fri Mar 03 20:10:14 2006 The RT System itself - Status changed from 'stalled' to 'open'

Fri Mar 03 20:10:35 2006 MARKSTOS [...] cpan.org - Status changed from 'open' to 'stalled'

Thu Apr 13 21:50:05 2006 MARKSTOS [...] cpan.org - Told changed from Fri Mar 03 20:10:15 2006 to Thu Apr 13 21:50:05 2006

Wed Oct 25 22:13:25 2006 MARKSTOS [...] cpan.org - Told changed from Thu Apr 13 21:50:05 2006 to Wed Oct 25 22:13:25 2006

Sat Mar 15 22:20:17 2008 MARKSTOS [...] cpan.org - Correspondence added

I'm giving this to RSAVAGE, who said he would have time to look at this. Mark On Fri Mar 03 20:10:13 2006, MARKSTOS wrote: Show quoted text

> On Thu Mar 13 13:00:32 2003, guest wrote:

> > I would like to adapt CGI::Session::MySQL to my own, existing session > > table. I hope to make my existing code just a wrapper around > > CGI::Session. However, for backward compatibility, I don't want

> to

> > change my column names, which are "session_key", "session_time", > > and "session_data". > > > > I request that the MySQL column names become modifiable constants, > > instead of hardcoded as "id" and "a_session". This can be easily > > accomplished by the patch that I will attach in a moment.

> > Chris, > > Are you still interested this ancient suggestion for CGI::Session? I > agree it's a good idea and am helping Sherzod maintain CGI::Session > now. > > If so, I think we could accept a patch against the 4.x branch, using > the OO-style API. The patch would need docs and automated tests to go > in. > > I expect to put out another release in the next couple of weeks. > > Thanks! > > Mark > >

Sat Mar 15 22:20:19 2008 The RT System itself - Status changed from 'stalled' to 'open'

Sat Mar 15 22:20:20 2008 MARKSTOS [...] cpan.org - Given to RSAVAGE

Sun Mar 16 00:20:06 2008 MARKSTOS [...] cpan.org - Correspondence added

It appears that Ron Savage has implemented this now in SVN. Mark

Sun Mar 16 00:20:09 2008 MARKSTOS [...] cpan.org - Status changed from 'open' to 'resolved'