[Bug 571192] New: $ENV{"HTTP_TRANSFER_ENCODING"} may be undefined in SOAP::Transport::HTTP

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: $ENV{"HTTP_TRANSFER_ENCODING"} may be undefined in SOAP::Transport::HTTP

https://bugzilla.redhat.com/show_bug.cgi?id=571192

           Summary: $ENV{"HTTP_TRANSFER_ENCODING"} may be undefined in
                    SOAP::Transport::HTTP
           Product: Fedora
           Version: 12
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: low
         Component: perl-SOAP-Lite
        AssignedTo: mmcgrath at redhat.com
        ReportedBy: uckelman at nomic.net
         QAContact: extras-qa at fedoraproject.org
                CC: fedora-perl-devel-list at redhat.com, mmcgrath at redhat.com
    Classification: Fedora


Description of problem:

I get the following error message when calling Bugzilla's xmlrpc.cgi, which
uses SOAP::Transport::HTTP:

[Sun Mar 07 07:17:07 2010] [error] [client 146.50.208.39] [Sun Mar  7 07:17:07
2010] xmlrpc.cgi: Use of uninitialized value $ENV{"HTTP_TRANSFER_ENCODING"} in
pattern match (m//) at /usr/lib/perl5/vendor_perl/5.10.0/SOAP/Transport/HTTP.pm
line 530.

The offending line looks like this:

  my $chunked = ( $ENV{'HTTP_TRANSFER_ENCODING'} =~ /^chunked.*$/ ) || 0;

Since the Transfer-Encoding header will often not be present, wouldn't it be
better to first check whether the key we're asking for exists?

  my $chunked = (exists $ENV{'HTTP_TRANSFER_ENCODING'} &&
                 $ENV{'HTTP_TRANSFER_ENCODING'} =~ /^chunked.*$/ ) || 0;


Version-Release number of selected component (if applicable):

perl-SOAP-Lite-0.710.10-1.fc12.noarch


How reproducible:

Always.


Steps to Reproduce:
1. Load the URL for Bugzilla's xmlrpc.cgi.

Actual results:

Call fails with the above error.


Expected results:

No error.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.