You are viewing a plain text version of this content. The canonical link for it is here.
Posted to test-dev@httpd.apache.org by Zubin Kika <zu...@yahoo.com> on 2005/10/26 10:30:22 UTC
Perl framework patch for Apache bug 37145
Hi,
I have developed a patch to the perl-framework, that
tests the server for Bug 37145
http://issues.apache.org/bugzilla/show_bug.cgi?id=37145
This script performs 9 tests:
1.With reverse proxy-backend in HTTP:HTTPS mode
a)File size:Exactly 8kb
b)File size:Less than 8kb
c)File size:Greater than 8kb
2.Same tests as above with reverse proxy-backend in
HTTPS:HTTPS mode
3.Same tests with reverse proxy-backend in HTTPS:HTTP
mode
The patch is attached.
It contains the following files:
t/ssl/proxy_bug37145.t :The script file
t/htdocs/test.txt :Test file of size 100Kb
t/htdocs/test1.txt :Test file of size less than 8 Kb
t/htdocs/test2.txt :Test file of size 8Kb
t/htdocs/modules/cgi/testrev_proxy.pl :Perl file that
will handle the posted data
Request you to please review the same.
Regards,
Zubin
__________________________________
Yahoo! Mail - PC Magazine Editors' Choice 2005
http://mail.yahoo.com
Re: Perl framework patch for Apache bug 37145
Posted by Zubin Kika <zu...@yahoo.com>.
Attaching the .t file after compressing.
--- "William A. Rowe, Jr." <wr...@rowe-clan.net>
wrote:
> And that answers that <grrr>
>
> Zubin, would you .gz compress your .t file to see if
> it comes through?
>
>
__________________________________
Yahoo! FareChase: Search multiple travel sites in one click.
http://farechase.yahoo.com
Re: Perl framework patch for Apache bug 37145
Posted by "William A. Rowe, Jr." <wr...@rowe-clan.net>.
And that answers that <grrr>
Zubin, would you .gz compress your .t file to see if it comes through?
Re: Perl framework patch for Apache bug 37145
Posted by "William A. Rowe, Jr." <wr...@rowe-clan.net>.
Since Zubin's posts for the .t file kept failing (yet I can see them in
my own email client from Zubin) - I thought I'd try resending for him
the new pr37145.t - and see if Thunderbird + apmail does any better.
Zubin Kika wrote:
> Don't know why the other file didn't make it.
> Attaching pr37145.t again.
>
> Zubin
>
> --- Zubin Kika <zu...@yahoo.com> wrote:
>
>
>>Hi,
>>
>>Thank you Joe for your suggestions.
>>I have incorporated the changes and attached two
>>files:
>> 1)pr37145.t: This file is to be placed in
>>t/ssl/
>> 2)testrev_proxy.pl: This file is to be
>>placed
>>in t/htdocs/modules/cgi/
>>
>>Please review.
>>
>>With regards,
>>Zubin
>>
>>--- Joe Orton <jo...@redhat.com> wrote:
>>
>>
>>>On Wed, Oct 26, 2005 at 01:30:22AM -0700, Zubin
>>
>>Kika
>>
>>>wrote:
>>>
>>>>The patch is attached.
>>>>It contains the following files:
>>>>t/ssl/proxy_bug37145.t :The script file
>>>>t/htdocs/test.txt :Test file of size 100Kb
>>>>t/htdocs/test1.txt :Test file of size less than
>>
>>8
>>
>>>Kb
>>>
>>>>t/htdocs/test2.txt :Test file of size 8Kb
>>>>t/htdocs/modules/cgi/testrev_proxy.pl :Perl file
>>>
>>>that
>>>
>>>>will handle the posted data
>>>>
>>>>Request you to please review the same.
>>>
>>>Thanks for the contribution.
>>>
>>>- the CGI script should not rely on CGI.pm
>>>- call the new test simply "t/ssl/pr37145.t"
>>>- attach files not tarballs when posting new code
>>>- create a new directory for new htdocs
>>>(t/htdocs/modules/ssl)
>>>- a CGI script which outputs 100Kb of data would
>>
>>be
>>
>>>preferable to
>>>adding a 100Kb file directly to the repos
>>>- follow the normal code style (indenting,
>>>whitespace etc) for the .t
>>>code; should be something like:
>>>
>>
> http://perl.apache.org/docs/2.0/devel/core/coding_style.html#Coding_Style_Guide
>
>>>
>>>joe
>>>
>>
>>
>>
>>
>>__________________________________
>>Yahoo! Mail - PC Magazine Editors' Choice 2005
>>http://mail.yahoo.com
>
>
>
>
> __________________________________
> Yahoo! FareChase: Search multiple travel sites in one click.
> http://farechase.yahoo.com
Re: Perl framework patch for Apache bug 37145
Posted by Zubin Kika <zu...@yahoo.com>.
Don't know why the other file didn't make it.
Attaching pr37145.t again.
Zubin
--- Zubin Kika <zu...@yahoo.com> wrote:
> Hi,
>
> Thank you Joe for your suggestions.
> I have incorporated the changes and attached two
> files:
> 1)pr37145.t: This file is to be placed in
> t/ssl/
> 2)testrev_proxy.pl: This file is to be
> placed
> in t/htdocs/modules/cgi/
>
> Please review.
>
> With regards,
> Zubin
>
> --- Joe Orton <jo...@redhat.com> wrote:
>
> > On Wed, Oct 26, 2005 at 01:30:22AM -0700, Zubin
> Kika
> > wrote:
> > > The patch is attached.
> > > It contains the following files:
> > > t/ssl/proxy_bug37145.t :The script file
> > > t/htdocs/test.txt :Test file of size 100Kb
> > > t/htdocs/test1.txt :Test file of size less than
> 8
> > Kb
> > > t/htdocs/test2.txt :Test file of size 8Kb
> > > t/htdocs/modules/cgi/testrev_proxy.pl :Perl file
> > that
> > > will handle the posted data
> > >
> > > Request you to please review the same.
> >
> > Thanks for the contribution.
> >
> > - the CGI script should not rely on CGI.pm
> > - call the new test simply "t/ssl/pr37145.t"
> > - attach files not tarballs when posting new code
> > - create a new directory for new htdocs
> > (t/htdocs/modules/ssl)
> > - a CGI script which outputs 100Kb of data would
> be
> > preferable to
> > adding a 100Kb file directly to the repos
> > - follow the normal code style (indenting,
> > whitespace etc) for the .t
> > code; should be something like:
> >
>
http://perl.apache.org/docs/2.0/devel/core/coding_style.html#Coding_Style_Guide
> >
> > joe
> >
>
>
>
>
> __________________________________
> Yahoo! Mail - PC Magazine Editors' Choice 2005
> http://mail.yahoo.com
__________________________________
Yahoo! FareChase: Search multiple travel sites in one click.
http://farechase.yahoo.com
Re: Perl framework patch for Apache bug 37145
Posted by Zubin Kika <zu...@yahoo.com>.
Hi,
Thank you Joe for your suggestions.
I have incorporated the changes and attached two
files:
1)pr37145.t: This file is to be placed in
t/ssl/
2)testrev_proxy.pl: This file is to be placed
in t/htdocs/modules/cgi/
Please review.
With regards,
Zubin
--- Joe Orton <jo...@redhat.com> wrote:
> On Wed, Oct 26, 2005 at 01:30:22AM -0700, Zubin Kika
> wrote:
> > The patch is attached.
> > It contains the following files:
> > t/ssl/proxy_bug37145.t :The script file
> > t/htdocs/test.txt :Test file of size 100Kb
> > t/htdocs/test1.txt :Test file of size less than 8
> Kb
> > t/htdocs/test2.txt :Test file of size 8Kb
> > t/htdocs/modules/cgi/testrev_proxy.pl :Perl file
> that
> > will handle the posted data
> >
> > Request you to please review the same.
>
> Thanks for the contribution.
>
> - the CGI script should not rely on CGI.pm
> - call the new test simply "t/ssl/pr37145.t"
> - attach files not tarballs when posting new code
> - create a new directory for new htdocs
> (t/htdocs/modules/ssl)
> - a CGI script which outputs 100Kb of data would be
> preferable to
> adding a 100Kb file directly to the repos
> - follow the normal code style (indenting,
> whitespace etc) for the .t
> code; should be something like:
>
http://perl.apache.org/docs/2.0/devel/core/coding_style.html#Coding_Style_Guide
>
> joe
>
__________________________________
Yahoo! Mail - PC Magazine Editors' Choice 2005
http://mail.yahoo.com
Re: Perl framework patch for Apache bug 37145
Posted by Joe Orton <jo...@redhat.com>.
On Wed, Oct 26, 2005 at 01:30:22AM -0700, Zubin Kika wrote:
> The patch is attached.
> It contains the following files:
> t/ssl/proxy_bug37145.t :The script file
> t/htdocs/test.txt :Test file of size 100Kb
> t/htdocs/test1.txt :Test file of size less than 8 Kb
> t/htdocs/test2.txt :Test file of size 8Kb
> t/htdocs/modules/cgi/testrev_proxy.pl :Perl file that
> will handle the posted data
>
> Request you to please review the same.
Thanks for the contribution.
- the CGI script should not rely on CGI.pm
- call the new test simply "t/ssl/pr37145.t"
- attach files not tarballs when posting new code
- create a new directory for new htdocs (t/htdocs/modules/ssl)
- a CGI script which outputs 100Kb of data would be preferable to
adding a 100Kb file directly to the repos
- follow the normal code style (indenting, whitespace etc) for the .t
code; should be something like:
http://perl.apache.org/docs/2.0/devel/core/coding_style.html#Coding_Style_Guide
joe