You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Need to check against RFC if this is a valid syntax.
*** Test close command 1/4 (line 3)
- failed: Expected tagged reply 'ok', got 'NO Flags must be a list'
- Command (tag 4.13): store 1,3 +flags \deleted
Activity
foxcpp commentedon May 14, 2019
Tried to run imaptest stress testing against maddy.
Using go-imap 1.0.0.beta.4 and go-imap-sql (dev branch).
Command:
Mentioned mbox file is here: http://www.dovecot.org/tmp/dovecot-crlf.
Outputer after running is two seconds: imaptest_stress.log (look for "Error:" lines)
Looks like there is some issue that makes responses get intermixed with literals in responses.
Adding
cilents=1
leaves onlyfoxcpp commentedon May 14, 2019
Need to check against RFC if this is a valid syntax.
foxcpp commentedon May 14, 2019
Relevant ABNF from RFC 3501.
Turns out
STORE * +FLAGS $FlagA $FlagB
is also valid and equivalent toSTORE * +FLAGS ($FlagA $FlagB)
.Handle STORE FLAGS A B syntax
Handle "STORE FLAGS seqset flag1 flag2" syntax
Handle "STORE FLAGS seqset flag1 flag2" syntax
foxcpp commentedon Dec 15, 2019
Adding to the v2 milestone. There is number of issues detected by imaptest that could be resolved only using breaking changes (see #323).