Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

urlparse fails on imap:// #37255

Closed
apederse mannequin opened this issue Oct 3, 2002 · 5 comments
Closed

urlparse fails on imap:// #37255

apederse mannequin opened this issue Oct 3, 2002 · 5 comments
Labels
stdlib Python modules in the Lib dir type-feature A feature request or enhancement

Comments

@apederse
Copy link
Mannequin

apederse mannequin commented Oct 3, 2002

BPO 618024

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2003-01-06.06:59:51.000>
created_at = <Date 2002-10-03.11:20:54.000>
labels = ['type-feature', 'library']
title = 'urlparse fails on imap://'
updated_at = <Date 2003-01-06.06:59:51.000>
user = 'https://bugs.python.org/apederse'

bugs.python.org fields:

activity = <Date 2003-01-06.06:59:51.000>
actor = 'nnorwitz'
assignee = 'nnorwitz'
closed = True
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2002-10-03.11:20:54.000>
creator = 'apederse'
dependencies = []
files = []
hgrepos = []
issue_num = 618024
keywords = []
message_count = 5.0
messages = ['53649', '53650', '53651', '53652', '53653']
nosy_count = 3.0
nosy_names = ['nnorwitz', 'apederse', 'pierslauder']
pr_nums = []
priority = 'normal'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue618024'
versions = []
@apederse
Copy link
Mannequin Author

apederse mannequin commented Oct 3, 2002

urlparse does not parse imap:// urls (RFC 2192). I am not
all sure but this seem to be an appropriate patch:

	urlparse.uses_relative.append('imap')
	urlparse.uses_netloc.append('imap')
	urlparse.non_hierarchical.append('imap')
	urlparse.uses_params.append('imap')
	urlparse.uses_query.append('imap')

@apederse apederse mannequin closed this as completed Oct 3, 2002
@apederse apederse mannequin assigned nnorwitz Oct 3, 2002
@apederse apederse mannequin added stdlib Python modules in the Lib dir type-feature A feature request or enhancement labels Oct 3, 2002
@apederse apederse mannequin closed this as completed Oct 3, 2002
@apederse apederse mannequin assigned nnorwitz Oct 3, 2002
@apederse apederse mannequin added stdlib Python modules in the Lib dir type-feature A feature request or enhancement labels Oct 3, 2002
@nnorwitz
Copy link
Mannequin

nnorwitz mannequin commented Nov 5, 2002

Logged In: YES
user_id=33168

Piers, is this right? If so, I can make the change.

@pierslauder
Copy link
Mannequin

pierslauder mannequin commented Nov 5, 2002

Logged In: YES
user_id=196212

I've never tried to use the imap URL form, and I'm unaware of
which IETF RFC documents its allowable syntax. Seems ok,
but why not "urlparse.uses_fragment.append('imap')" as well?

(Searching google...) This looks like the document that
defines it:
http://ftp.ics.uci.edu/pub/ietf/uri/rfc2192.txt
"IMAP URL Scheme"

No mention of '#', so I guess no fragments :-)

@pierslauder
Copy link
Mannequin

pierslauder mannequin commented Nov 22, 2002

Logged In: YES
user_id=196212

Just to reiterate - this change is fine by me, so please
make the suggested changes to the urlparse module's
classification schemes.

@nnorwitz
Copy link
Mannequin

nnorwitz mannequin commented Jan 6, 2003

Logged In: YES
user_id=33168

Checked in as:

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
stdlib Python modules in the Lib dir type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
0 participants