|Anonymous | Login||09-19-2021 19:42 UTC|
|Main | My View | View Issues | Change Log | Docs|
|Viewing Issue Advanced Details [ Jump to Notes ]||[ View Simple ] [ Issue History ] [ Print ]|
|ID||Category||Severity||Reproducibility||Date Submitted||Last Update|
|0002119||[Squeak] Network||minor||always||10-24-05 21:30||07-12-06 22:35|
|ETA||none||Fixed in Version||3.9||Product Version||3.8|
|Summary||0002119: incorrect content length in HTTPSocket(class)>>httpPostDocument:args:accept:request:|
In HTTPSocket (class)>>httpPostDocument:args:accept:request: the content length is sent as "argString size printString" and then a few lines later we see:
argString first = $? ifTrue: [ argString := argString copyFrom: 2 to: argString size].
"umur - IE sends argString without a $? and swiki expects so"
s sendCommand: argString.
This causes two less characters to be sent than we reported. The server's ability to handle this may vary but in my case, the last post variable has a CR included in its value.
I don't have the time to check if this is still an issue in 3.9 as I am at work but I didn't want this to get lost.
|Steps To Reproduce|
(0002958 - 343 - 365 - 365 - 365 - 365 - 365)
I hadn't look at it in details, but the HTTPSocket class code is *awful*. Code is duplicated everywhere and would benefit from an extreme refactoring.
Concerning your precise point, I think the hack should be removed and callers must ensure that no '?' is ever passed with the meaning "beginning of arguments" at the beginning of argString.
(0002959 - 637 - 677 - 677 - 677 - 677 - 677)
I agree, the code could definitely do with improvement in general.
As for removing the hack, this method generates the argString itself using HTTPSocket(class)>>argString: so that could be corrected to not include the ? and the relevent GET senders could add the ? in. Otherwise, the question mark *does* need to be removed to maintain compatibility. Even if the hack was moved to before the connection was made the content length would be correct. This is certainly by far the simplest fix.
It's almost certainly not the most "correct" fix but I'm not sure it's worth worrying about correctness much without starting a rewrite.
(0003326 - 184 - 184 - 184 - 184 - 184 - 184)
|I opted for the simplest fix discussed - moving the check up before sending the content length. I of course agree that this code is... well due for a real overhaul. But not tonight. :)|
(0005051 - 315 - 339 - 509 - 509 - 509 - 509)
Goran, could you revisit this one? It looks like your referenced change is in
however attempting to merge that now into 3.9b-7032 appears to cause additional, I have to assume unrelated changes. However I'm not sure I'm looking at this correctly.
(0005820 - 22 - 22 - 22 - 22 - 22 - 22)
|in 3.9 for a long time|
|10-24-05 21:30||jfitzell||New Issue|
|10-24-05 21:39||sam||Note Added: 0002958|
|10-24-05 21:39||sam||Issue Monitored: sam|
|10-24-05 22:17||jfitzell||Note Added: 0002959|
|12-10-05 20:59||gokr||Status||new => assigned|
|12-10-05 20:59||gokr||Assigned To||=> gokr|
|12-15-05 01:04||gokr||Status||assigned => resolved|
|12-15-05 01:04||gokr||Fixed in Version||=> 3.9|
|12-15-05 01:04||gokr||Resolution||open => fixed|
|12-15-05 01:04||gokr||Note Added: 0003326|
|05-25-06 05:31||KenCausey||Status||resolved => feedback|
|05-25-06 05:31||KenCausey||Resolution||fixed => reopened|
|05-25-06 05:31||KenCausey||Note Added: 0005051|
|07-12-06 22:35||MarcusDenker||Status||feedback => closed|
|07-12-06 22:35||MarcusDenker||Note Added: 0005820|
|07-12-06 22:35||MarcusDenker||Resolution||reopened => fixed|
| Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
62 total queries executed.|
42 unique queries executed.