[issue13940] imaplib: Mailbox names are not quoted

[Matej Cepl]

Change by Matej Cepl :


--
nosy: +mcepl

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Alexander Harkness]

Alexander Harkness  added the comment:

Pull Request opened on GH to fix this issue: 
https://github.com/python/cpython/pull/6395

--
versions: +Python 3.6, Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Mark Lawrence]

Change by Mark Lawrence :


--
nosy:  -BreamoreBoy

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Alexander Harkness]

Change by Alexander Harkness :


--
nosy: +bearbin

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Alexander Harkness]

Change by Alexander Harkness :


--
pull_requests: +6103
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Maciej Szulik]

Maciej Szulik added the comment:

siemer thx for the comments in the review. I'll try to address them by the end 
of this week.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Maciej Szulik]

Maciej Szulik added the comment:

>From imaplib.IMAP4 class description we're stating to quote parameters if 
>necessary. This patch adds that capability according to 
>https://tools.ietf.org/html/rfc3501#section-5.1

--
keywords: +patch
Added file: http://bugs.python.org/file41037/issue13940.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Maciej Szulik]

Maciej Szulik added the comment:

Currently working on a patch for this.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Maciej Szulik]

Maciej Szulik added the comment:

Joe is correct, according to the spec the names containing special chars 
(atom-specials) require names to be quoted 
(https://tools.ietf.org/html/rfc3501#section-5.1): 

   1)Any character which is one of the atom-specials (see the Formal
 Syntax) will require that the mailbox name be represented as a
 quoted string or literal.

I'll be working on a patch for that.

--
nosy: +maciej.szulik

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13940
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Mark Lawrence]

Mark Lawrence added the comment:

@Joe sorry for the delay in getting back to you.

@David is this within your remit?

--
nosy: +BreamoreBoy, r.david.murray
versions: +Python 3.4, Python 3.5 -Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13940
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[R. David Murray]

R. David Murray added the comment:

Yes.  It will be necessary to check the RFC to figure out what should be done 
here.

--
components: +email
nosy: +barry

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13940
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue13940] imaplib: Mailbox names are not quoted

[Johannes Bauer]

New submission from Johannes Bauer dfnsonfsdu...@gmx.de:

imaplib does not qupote mailbox names when it's sending it's query to the 
server in response to a status request, for example. This will lead to very 
strange errors if mailboxes are accessed which contain spaces:

i.e.

connection.status(mailbox name, (MESSAGES))

will return

  File /home/joe/test/imaplib.py, line 920, in _command_complete
raise self.error('%s command error: %s %s' % (name, typ, data))
imaplib.error: STATUS command error: BAD [b'Error in IMAP command STATUS: 
Status items must be list.']

which indicates that the error is within the actual flag list. It is not, 
however:

connection.status(\mailbox name\, (MESSAGES))

works as expected. This may arguably be or not be a bug -- however it is REALLY 
confusing to the user. Maybe at least a note should be included somewhere that 
-- just to be safe -- mailbox names should be quoted.

All the best,
Joe

--
components: Library (Lib)
messages: 152612
nosy: joebauer
priority: normal
severity: normal
status: open
title: imaplib: Mailbox names are not quoted
type: behavior
versions: Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13940
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com