Closed
Bug 442313
Opened 17 years ago
Closed 17 years ago
currenturl contains post information
Categories
(Webtools Graveyard :: Verbatim, defect)
Webtools Graveyard
Verbatim
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dschafer, Assigned: dschafer)
References
Details
Attachments
(1 file)
3.08 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
jToolkit merges post and get parameters, so when we reconstruct the URL, it includes both POST and GET in the URL. We remove "password", so that should never appear, but this still is a problem.
Assignee | ||
Comment 1•17 years ago
|
||
This overrides the default handle(); in addition to doing all the work of the original function, the new handle extracts the path from the GET request and stores it in the session. This ensures currenturl is the correct, requested URL.
Attachment #332245 -
Flags: review?(clouserw)
Comment 2•17 years ago
|
||
Comment on attachment 332245 [details] [diff] [review]
Modified handle() to extract the URL, path and getargs
Add the s? we talked about on the mailing list please
Attachment #332245 -
Flags: review?(clouserw) → review+
Assignee | ||
Comment 3•17 years ago
|
||
This is in revision 7993; marking FIXED.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•