You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@perl.apache.org by Doug MacEachern <do...@covalent.net> on 2002/05/23 18:25:30 UTC

Re: [PATCH] CGI.pm vs. mod_perl2's POST.

On Fri, 24 May 2002, Autrijus Tang wrote:

> Hello there. I've been using CGI.pm with the CVS Apache2 module, and
> found out that read_from_client doesn't quite work -- it sometimes
> blocks the socket, but most of the time just returns an empty list.
> 
> As the new Apache->request provide a 'content' method already, I patched
> it so it tests for the method, and prefers it if it is available.

the 'content' method is deprecated in 2.0.  CGI.pm should be moving away 
from modperl-isms rather than adding new ones.  can we see your perl -V 
please?  there is a known issue with 5.6.1 and ithreads enabled that might 
be causing this problem.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [PATCH] CGI.pm vs. mod_perl2's POST.

Posted by Doug MacEachern <do...@covalent.net>.
On Fri, 24 May 2002, Autrijus Tang wrote:
 
> It's bleadperl-16610. I'll try to hunt down the bug -- any quick pointers?

the bug i was thinking of has been fixed in bleedperl (changes 11536 + 
11553), and there's a workaround in modperl2 for 5.6.x

that bug caused STDIN to loose its tied-ness, so CGI.pm would end up 
trying to read stdin-STDIN rather than tied-to-apache-STDIN.

the other workaround for that bug was to change CGI.pm from:
$self->read_from_client(\*STDIN,\$query_string,$content_length,0)
to:         
$self->read_from_client('STDIN',\$query_string,$content_length,0)

do you have a simple test case to reproduce the problem?




---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [PATCH] CGI.pm vs. mod_perl2's POST.

Posted by Autrijus Tang <au...@autrijus.org>.
On Thu, May 23, 2002 at 09:25:30AM -0700, Doug MacEachern wrote:
> On Fri, 24 May 2002, Autrijus Tang wrote:
> > As the new Apache->request provide a 'content' method already, I patched
> > it so it tests for the method, and prefers it if it is available.
> the 'content' method is deprecated in 2.0.  CGI.pm should be moving away 
> from modperl-isms rather than adding new ones.

I see. Thanks for your kind explanation.

> can we see your perl -V please?  there is a known issue with 5.6.1 and
> ithreads enabled that might be causing this problem.

It's bleadperl-16610. I'll try to hunt down the bug -- any quick pointers?

Thanks,
/Autrijus/

Summary of my perl5 (revision 5 version 7 subversion 3) configuration:
  Platform:
    osname=MSWin32, osvers=4.0, archname=MSWin32-x86-multi-thread
    uname=''
    config_args='undef'
    hint=recommended, useposix=true, d_sigaction=undef
    usethreads=undef use5005threads=undef useithreads=define usemultiplicity=define
    useperlio=define d_sfio=undef uselargefiles=undef usesocks=undef
    use64bitint=undef use64bitall=undef uselongdouble=undef
    usemymalloc=n, bincompat5005=undef
  Compiler:
    cc='cl', ccflags ='-nologo -Gf -W3 -MD -DNDEBUG -O1 -DWIN32 -D_CONSOLE -DNO_STRICT -DHAVE_DES_FCRYPT  -DPERL_IMPLICIT_CONTEXT -DPERL_IMPLICIT_SYS -DUSE_PERLIO -DPERL_MSVCRT_READFIX',
    optimize='-MD -DNDEBUG -O1',
    cppflags='-DWIN32'
    ccversion='', gccversion='', gccosandvers=''
    intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234
    d_longlong=undef, longlongsize=8, d_longdbl=define, longdblsize=10
    ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=4
    alignbytes=8, prototype=define
  Linker and Libraries:
    ld='link', ldflags ='-nologo -nodefaultlib -release  -libpath:"c:\perl58\lib\CORE"  -machine:x86'
    libpth=C:\PROGRA~1\MICROS~3\VC98\lib
    libs=  oldnames.lib kernel32.lib user32.lib gdi32.lib winspool.lib  comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib  netapi32.lib uuid.lib wsock32.lib mpr.lib winmm.lib  version.lib odbc32.lib odbccp32.lib msvcrt.lib
    perllibs=  oldnames.lib kernel32.lib user32.lib gdi32.lib winspool.lib  comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib  netapi32.lib uuid.lib wsock32.lib mpr.lib winmm.lib  version.lib odbc32.lib odbccp32.lib msvcrt.lib
    libc=msvcrt.lib, so=dll, useshrplib=yes, libperl=perl58.lib
  Dynamic Linking:
    dlsrc=dl_win32.xs, dlext=dll, d_dlsymun=undef, ccdlflags=' '
    cccdlflags=' ', lddlflags='-dll -nologo -nodefaultlib -release  -libpath:"c:\perl58\lib\CORE"  -machine:x86'


Characteristics of this binary (from libperl): 
  Compile-time options: MULTIPLICITY USE_ITHREADS PERL_IMPLICIT_CONTEXT PERL_IMPLICIT_SYS
  Locally applied patches:
  	DEVEL16610
  Built under MSWin32
  Compiled at May 17 2002 11:27:50
  %ENV:
    PERLDB_OPTS="RemotePort=127.0.0.1:2000"
  @INC:
    C:/perl58/lib
    C:/perl58/site/lib
    .