currenturl contains post information

RESOLVED FIXED

Status

RESOLVED FIXED
10 years ago
3 years ago

People

(Reporter: dschafer, Assigned: dschafer)

Tracking

Dependency tree / graph

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
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)

Updated

10 years ago
Blocks: 449049
(Assignee)

Comment 1

10 years ago
Created attachment 332245 [details] [diff] [review]
Modified handle() to extract the URL, path and getargs

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)
(Assignee)

Updated

10 years ago
Blocks: 447048
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

10 years ago
This is in revision 7993; marking FIXED.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.