Bug 127399 – Ab2 benchmark tool (from apache 2.0.5x) has been bug with repeatable functions -C (cookie)

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Mike Baikov 2006-03-24 01:55:53 UTC]

dds ~ # ab2 -C a=1 -C b=2 -v 2 http://baikov.com/
This is ApacheBench, Version 2.0.41-dev <$Revision: 1.121.2.12 $> apache-2.0
Copyright (c) 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Copyright (c) 1998-2002 The Apache Software Foundation, http://www.apache.org/

Benchmarking baikov.com (be patient)...INFO: POST header ==
---
GET / HTTP/1.0
User-Agent: ApacheBench/2.0.41-dev
Cookie: b=2
Host: baikov.com
Accept: */*


---
LOG: header received:
HTTP/1.1 200 OK
Server: nginx/0.3.34
Date: Fri, 24 Mar 2006 09:55:08 GMT
Content-Type: text/html; charset=utf8
Connection: close
Set-Cookie: PHPSESSID=6b7456c196cad1a693e55a1f945ca0e7; path=/
Expires: Thu, 19 Nov 1981 08:52:00 GMT
Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0
Pragma: no-cache

[Comment 1 Mike Baikov 2006-03-24 02:00:43 UTC]

Send only last cookie, not all

[Comment 2 Mike Baikov 2006-03-24 02:13:12 UTC]

send only last cookie, but from docs:

-C cookie-name=value
              Add a Cookie: line to the request. The argument is typically in the form of a name=value pair. This field is repeatable.


i.e parameter IS REPEATABLE.

[Comment 3 Mike Baikov 2006-03-24 02:25:01 UTC]

not tested fix (add "cookie+"):

case 'C':
            cookie = apr_pstrcat(cntxt, cookie+"Cookie: ", optarg, "\r\n", NULL);
            break;

sorry, no diff

[Comment 4 Mike Baikov 2006-03-24 02:25:31 UTC]

Workaround:

Use -H flag

[Comment 5 Benedikt Böhm (RETIRED) 2006-06-06 03:47:56 UTC]

fixed in svn, next bump will contain it, thanks

[Comment 6 Mike Baikov 2006-07-04 02:52:00 UTC]

ok, bug closed.

[Comment 7 Mike Baikov 2006-10-27 01:29:13 UTC]

>fixed in svn, next bump will contain it, thanks

Where bump? seven month's w/o bump ... :-(

[Comment 8 Mike Baikov 2006-10-27 01:39:57 UTC]

>fixed in svn, next bump will contain it, thanks

Where bump? seven month's w/o bump ... :-(

[Comment 9 Michael Stewart (vericgar) (RETIRED) 2006-10-28 08:35:54 UTC]

(In reply to comment #8)
> >fixed in svn, next bump will contain it, thanks
> 
> Where bump? seven month's w/o bump ... :-(
> 

This was fixed in SVN on 06 Jun 2006.

There have been several bumps since then:

*apache-2.0.58-r1 (08 Jul 2006)
*apache-2.0.58-r2 (26 Jul 2006)
*apache-2.0.59 (30 Jul 2006)
*apache-2.0.59-r1 (15 Oct 2006)

This patch is included in all of these.

[Comment 10 Mike Baikov 2006-11-01 04:03:51 UTC]

Sorry, version of the ab2 is same.

dds ~ # ab2 -V
This is ApacheBench, Version 2.0.41-dev <$Revision: 1.121.2.12 $> apache-2.0
Copyright (c) 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Copyright (c) 1998-2002 The Apache Software Foundation, http://www.apache.org/

dds ~ # emerge -pv apache

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R   ] net-www/apache-2.0.58-r2  USE="apache2 mpm-prefork static-modules -debug -doc -ldap -mpm-itk -mpm-leader -mpm-peruser -mpm-threadpool -mpm-worker (-selinux) -ssl -threads (-no-suexec%*)" 0 kB

Total size of downloads: 0 kB

Why do not change version number? Too small bug? :)

[Comment 11 Michael Stewart (vericgar) (RETIRED) 2006-11-01 08:38:42 UTC]

We are not upstream for the package, we don't have a right to change the version number. If you want the version of ab2 to change, you have to convince upstream to do it.