Bug #740 for CGI-Session: CGI::Session::File requires 5.006 ?

Thu Jun 13 04:07:21 2002 Guest - Ticket created

Subject:

CGI::Session::File requires 5.006 ?

I just created a port for Freebsd 4.5-STABLE (http://www.freebsd.org/cgi/query-pr.cgi?pr=39191). Firstly, there were some warnings from pod2man (patch attached; I'm not sure the =head3 to =item is the best way, but at least it silences pod2man: pod2man: Unrecognized pod directive in paragraph 255 of Session.pm: head3 ). The patch doesn't silence one warning: /usr/bin/pod2man: Session/MD5.pm is missing required section: DESCRIPTION Secondly, CGI::Session::File fails with Can't locate object method "new" via package "CGI::Session::File" at ../lib/SelfCare.pm line 52. on 4.5-STABLE with perl version 5.005_03 built for i386-freebsd. It works fine on a similar system, but with perl 5.6.1. I wasn't able to patch this, so any help appreciated. Cheers, /erwin

--- Session.pm.orig Wed Jun 12 11:56:43 2002 +++ Session.pm Wed Jun 12 12:04:39 2002 @@ -915,7 +915,7 @@ =head2 STANDARD INSTALLATION The library is installed with just like other Perl libraries, or via CPAN interactive -shell (Perl -MCPAN -e install CGI::Session). +shell (Perl -MCPAN C<-e> install CGI::Session). Installation can also be done by following below instructions: @@ -1117,7 +1117,7 @@ above example. So, how do we "mark" the user? There are several ways of "marking". -=head3 IDENTIFYING THE USER VIA CGI QUERY +=item IDENTIFYING THE USER VIA CGI QUERY One way of doing it is to append the session id to every single link in the web site: @@ -1167,7 +1167,7 @@ session id appended to them. Too bad, because the client has to start everything over again. -=head3 INDENTIFYING THE USER VIA COOKIES +=item INDENTIFYING THE USER VIA COOKIES We can deal with the above problem by sending the client a cookie. This cookie will hold the session id only! Thus if the client visits some other site, or @@ -1211,7 +1211,7 @@ the C<sid> parameter in the URL, and if that fails, then it will default to undef, which will force C<CGI::Session> to create a new id for the client. -=head3 IDENTIFYING THE USER VIA PATH_INFO +=item IDENTIFYING THE USER VIA PATH_INFO The least common, but at the same time quite convenient way of C<marking> users with a session id is appending the session id to the url of the script @@ -1363,7 +1363,7 @@ =back -=head3 SPECIAL NAMES +=item SPECIAL NAMES When you create a fresh-blank session, it's not blank as it seems. It is initialized with the following 4 parameters, which are serialized together

Thu Jun 13 13:43:27 2002 Guest - Correspondence added

Hi Erwin Show quoted text

> I'm not sure the > =head3 to =item is the best way, but at least it silences pod2man: > pod2man: Unrecognized pod directive in paragraph 255 of Session.pm: > head3 > ). The patch doesn't silence one warning: > /usr/bin/pod2man: Session/MD5.pm is missing required section: > DESCRIPTION

I am not a great manual writer, but I thought =head3 was a valid POD directive (at least that's what my "Programming Perl 3rd edition says). But I'll try to do some more research on it this weekend. About Session/MD5.pm warning, I am not quite sure, because DESCRIPTION section should be there, I just checked CGI::Session/2.91. Show quoted text

> Can't locate object method "new" via package "CGI::Session::File" at > ../lib/SelfCare.pm line 52. > on 4.5-STABLE with perl version 5.005_03 built for i386-freebsd. It > works fine on a similar system, but with perl 5.6.1. I wasn't able > to patch this, so any help appreciated.

I tried to make the library compatible with Perl 5.003+, but looks like it is not. File.pm should've inheritted new() from base class CGI::Session, so it's not happening in 5.005_3. If you can send me the full test results from "make test" I'll try to solve the problem. Thanks sherzod

Fri Jun 21 14:40:14 2002 sherzodr [...] cpan.org - Status changed from 'new' to 'resolved'