This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: add urldecode() method to urllib
Type: Stage:
Components: Extension Modules Versions:
process
Status: closed Resolution: rejected
Dependencies: Superseder:
Assigned To: Nosy List: brett.cannon, irmen, loewis, mboedick
Priority: normal Keywords: patch

Created on 2003-05-21 00:43 by mboedick, last changed 2022-04-10 16:08 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
urldecode.patch mboedick, 2003-05-21 00:43 path to add urldecode() method to urllib
Messages (6)
msg43816 - (view) Author: Matthew Boedicker (mboedick) Date: 2003-05-21 00:43
Adds a urldecode() method to urllib. The method does
the opposite of urlencode, turns a url-encoded string
into a list of two-tuples of name/value pairs.
msg43817 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2003-05-21 03:21
Logged In: YES 
user_id=357491

How is this different from cgi.parse_sql ?  If it is not different, consider just 
having a wrapper for urldecode that calls cgi.parse_sql .

Do you think urllib is the best place for this?  What about urlparse?
msg43818 - (view) Author: Matthew Boedicker (mboedick) Date: 2003-05-21 14:25
Logged In: YES 
user_id=119895

It is not different than cgi.parse_qsl.  Must have missed
that module when I was looking for this functionality.  To
me it seems like urlencode and urldecode (as well as quote,
quote_plus, unquote, etc.) belong in urlparse.  Do you think
it would be valuable to include a pointer to cgi.parse_qsl
in urllib or urllparse?
msg43819 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2003-05-21 16:00
Logged In: YES 
user_id=357491

I think so, but I don't know which one.  I only use urllib.urlopen ever so if 
you have a specific suggestion from experience please make it and submit a 
patch.  Since I don't use either modules extensively someone else is going to 
need to step in and help make any final decisions.
msg43820 - (view) Author: Irmen de Jong (irmen) (Python triager) Date: 2005-01-16 15:41
Logged In: YES 
user_id=129426

I've always found it strange that url-manipulating methods
were present in the cgi module.
I agree that it would be nicer to have them grouped together
in a single module such as urlparse.
So I wouldn't add urldecode to urllib.
msg43821 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2005-02-24 20:02
Logged In: YES 
user_id=21627

It appears that all reviewers agree that the patch should
not be applied. Rejecting it.
History
Date User Action Args
2022-04-10 16:08:50adminsetgithub: 38532
2003-05-21 00:43:29mboedickcreate