You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2013/10/22 05:25:30 UTC

[Bug 55686] New: make request.getParameterNames() and request.getParameterMap ordered

https://issues.apache.org/bugzilla/show_bug.cgi?id=55686

            Bug ID: 55686
           Summary: make request.getParameterNames() and
                    request.getParameterMap ordered
           Product: Tomcat 8
           Version: trunk
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Catalina
          Assignee: dev@tomcat.apache.org
          Reporter: zhouyanming@gmail.com

it should ordered by query string then post form data.
for example 
POST http://localhost:8080/test?a=a&b=b with form data "c=c&d=d&e=e"
request.getParameterNames() should return ["a","b","c","d","e"]

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 55686] make request.getParameterNames() and request.getParameterMap ordered

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55686

Violeta Georgieva <vi...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
         Resolution|---                         |DUPLICATE

--- Comment #3 from Violeta Georgieva <vi...@apache.org> ---
It is fixed in 7.0.x also.

Currently we are voting 7.0.47 [1]. The fix is available in that version.

Regards
Violeta 

[1]
http://markmail.org/message/yus4osg2bgaoekp6?q=7%2E0%2E47+list:org%2Eapache%2Etomcat%2Edev/

*** This bug has been marked as a duplicate of bug 55576 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 55686] make request.getParameterNames() and request.getParameterMap ordered

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55686

quaff <zh...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zhouyanming@gmail.com
                 OS|                            |All

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 55686] make request.getParameterNames() and request.getParameterMap ordered

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55686

--- Comment #2 from quaff <zh...@gmail.com> ---
wonderful,thanks very much,I have tested with 8.0.0-RC5,can you fix this in
7.0.x?

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 55686] make request.getParameterNames() and request.getParameterMap ordered

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55686

Violeta Georgieva <vi...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #1 from Violeta Georgieva <vi...@apache.org> ---
Hi,

There was already a report for a similar issue - Bug 55576. The fix is
available in 8.0.0-RC4 and yesterday it was announced the availability of
8.0.0-RC6.

I tried your scenario with the current trunk and the parameters are ordered.
Please provide a reproducible scenario against trunk.

Regards
Violeta

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 55686] make request.getParameterNames() and request.getParameterMap ordered

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55686

--- Comment #5 from quaff <zh...@gmail.com> ---
@Christopher Schultz, it should be [a, b, c],and
request.getParameterValues("a") will be [1,4]

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 55686] make request.getParameterNames() and request.getParameterMap ordered

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55686

--- Comment #4 from Christopher Schultz <ch...@christopherschultz.net> ---
The fix for bug #55576 might not entirely "fix" this enhancement as written.

For example, in this case:

/path/to/servlet?a=1&b=2&c=3&a=4

One might expect the return value of getParameterNames to be:

[ a, b, c, a ]

...but it will instead be [ a, b, c ].

"Ordering" the parameter map has the same "problem", but there is absolutely
nothing to be done about that AFAICT.

I'm not suggesting that either of the above problems can be fixed... I just
wanted to point-out the possible expectations users might have and how those
expectations can't necessarily be met.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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