Skip Menu |
 

This queue is for tickets about the CGI-Session CPAN distribution.

Report information
The Basics
Id: 17979
Status: resolved
Priority: 0/
Queue: CGI-Session

People
Owner: Nobody in particular
Requestors: MARKSTOS [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)



Subject: wish: make ->name() work as an instance method
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 294
Download (untitled) / with headers
text/plain 294b
The current way of handling ->name() is sub-optimal. Using a class method is like setting a global, and can cause problems under mod-perl. Support for calling name() as an instance method should be added, and recommended as the default use, while still supporting the Old Way. Mark
X-Scanned-BY: AMaViS-ng at bestpractical
MIME-Version: 1.0
X-Spam-Status: No, hits=-2.6 required=8.0 tests=BAYES_00
In-Reply-To: <rt-3.5.HEAD-20565-1141492363-1440.17979-3-0 [...] rt.cpan.org> <fd8e28ef0603040626h21d6918l95904365d3c67fe3 [...] mail.gmail.com>
Content-Disposition: inline
Received-SPF: pass (x1.develooper.com: local policy)
References: <RT-Ticket-17979 [...] rt.cpan.org> <rt-3.5.HEAD-20565-1141492363-1440.17979-3-0 [...] rt.cpan.org> <20060304141442.GI14094 [...] summersault.com> <fd8e28ef0603040626h21d6918l95904365d3c67fe3 [...] mail.gmail.com>
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: us-ascii
Received: from localhost (localhost.localdomain [127.0.0.1]) by diesel.bestpractical.com (Postfix) with ESMTP id BE9E94D80C6 for <cpan-bug+cgi-session [...] diesel.bestpractical.com>; Sat, 4 Mar 2006 12:14:44 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id 6DF744D8004 for <bug-CGI-Session [...] rt.cpan.org>; Sat, 4 Mar 2006 12:14:44 -0500 (EST)
Received: (qmail 22371 invoked by alias); 4 Mar 2006 17:14:42 -0000
Received: from tanagra.summersault.com (HELO tanagra.summersault.com) (12.161.105.149) by la.mx.develooper.com (qpsmtpd/0.28) with SMTP; Sat, 04 Mar 2006 09:14:35 -0800
Received: (qmail 3352 invoked by uid 1002); 4 Mar 2006 17:14:28 -0000
Delivered-To: cpan-bug+cgi-session [...] diesel.bestpractical.com
Subject: Re: using different session names in the ::Session plugin [rt.cpan.org #17979]
User-Agent: Mutt/1.5.6i
Return-Path: <mark [...] summersault.com>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: cpan-bug+cgi-session [...] diesel.bestpractical.com
Date: Sat, 4 Mar 2006 12:14:28 -0500
Message-Id: <20060304171428.GK14094 [...] summersault.com>
To: Bugs in CGI-Session via RT <bug-CGI-Session [...] rt.cpan.org>
From: Mark Stosberg <mark [...] summersault.com>
X-RT-Original-Encoding: utf-8
RT-Message-ID: <rt-3.5.HEAD-20585-1141492488-1554.17979-0-0 [...] rt.cpan.org>
Content-Length: 980
Download (untitled) / with headers
text/plain 980b
A related comment: On Sat, Mar 04, 2006 at 09:26:24AM -0500, Cees Hek wrote: Show quoted text
>
> > I wanted to run this idea by you: > > > > When using a different cookie name, let's call CGI::Session->name() > > automatically. > > > > I can't think of a reason not to, this is currently the only place that > > requires talking to CGI::Session directly.
> > The reason I haven't done this automatically is because I don't like > the way that CGI::Session handles this. You are effectively setting a > global variable, and that can have nasty repercussions if you are > running under mod_perl (ie sessions failing for some programs > intermittantly as the session name is changed dynamically at runtime > for only some of the apache children). ideally, CGI::Session should > allow you to change the name of the cookie in the options when you > create your Session object. If that change was made to CGI::Session, > then I'll quickly add code to CAP::Session to make it all more > seemless.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.