Bug #59544 for Net-IMAP-Server: Search Filter missing ? issue

Mon Jul 19 12:17:39 2010 jettero [...] cpan.org - Ticket created

Subject:

Search Filter missing ? issue

Hi, it's me again. Sorry in advance. I'm adding some uid search features to the simple client... on a UID SEARCH 1:2 I'm getting the expected results, but UID SEARCH 1,2 dies with an error about unrecognized tokens... I found the issue in ::Command::Search::filter here: } elsif ($token =~ /^\d+(:\d+|:\*)?(,\d+(:\d+|:\*))*$/) { To the naked eye, it seems like there's a missing ?: } elsif ($token =~ /^\d+(:\d+|:\*)?(,\d+(:\d+|:\*)?)*$/) { -- If riding in an airplane is flying, then riding in a boat is swimming. 116 jumps, 48.6 minutes of freefall, 92.9 freefall miles.

Mon Jul 19 12:21:25 2010 jettero [...] cpan.org - Correspondence added

Corollary... the token parser doesn't seem to accept the "*" token either. RFC: seq-number = nz-number / "*" That may be harder to fix. I didn't go source diving on it. -- If riding in an airplane is flying, then riding in a boat is swimming. 116 jumps, 48.6 minutes of freefall, 92.9 freefall miles.

Mon Jul 19 12:21:25 2010 jettero [...] cpan.org - Status changed from 'new' to 'open'

Tue Jul 20 04:02:05 2010 alex [...] chmrr.net - Correspondence added

Subject:	Re: [rt.cpan.org #59544] Search Filter missing ? issue
Date:	Tue, 20 Jul 2010 04:01:58 -0400
To:	bug-Net-IMAP-Server <bug-net-imap-server [...] rt.cpan.org>
From:	Alex Vandiver <alex [...] chmrr.net>

At Mon Jul 19 12:17:40 -0400 2010, Paul Miller via RT wrote: Show quoted text

> on a UID SEARCH 1:2 I'm getting the expected results, but UID SEARCH 1,2 > dies with an error about unrecognized tokens...

Thanks, appllied in 5d3e9a2. - Alex -- Networking -- only one letter away from not working

Tue Jul 20 04:09:46 2010 alex [...] chmrr.net - Correspondence added

Subject:	Re: [rt.cpan.org #59544] Search Filter missing ? issue
Date:	Tue, 20 Jul 2010 04:09:38 -0400
To:	bug-Net-IMAP-Server <bug-net-imap-server [...] rt.cpan.org>
From:	Alex Vandiver <alex [...] chmrr.net>

At Mon Jul 19 12:21:25 -0400 2010, Paul Miller via RT wrote: Show quoted text

> Corollary... the token parser doesn't seem to accept the "*" token either.

Fixed in 60edd5f. For your amusement, observe how well Cyrus deals: a select inbox * FLAGS (\Answered \Flagged \Draft \Deleted \Seen Junk $Label1 $Label2 $Label3 $Label4 $Label5 $MDNSent SomeNewFlag NotJunk receipt-handled) * OK [PERMANENTFLAGS (\Answered \Flagged \Draft \Deleted \Seen Junk $Label1 $Label2 $Label3 $Label4 $Label5 $MDNSent SomeNewFlag NotJunk receipt-handled \*)] * 13258 EXISTS * 0 RECENT * OK [UNSEEN 88] * OK [UIDVALIDITY 989731130] * OK [UIDNEXT 420644] a OK [READ-WRITE] Completed b uid search * * SEARCH b OK Completed (0 msgs in 0.010 secs) c uid search 1 * SEARCH 314162 c OK Completed (1 msgs in 0.010 secs) d uid search * * SEARCH d OK Completed (0 msgs in 0.010 secs) e uid search 13258 * SEARCH 420643 e OK Completed (1 msgs in 0.010 secs) f uid search * * SEARCH 337836 f OK Completed (1 msgs in 0.010 secs) g uid search *,* * SEARCH g OK Completed (0 msgs in 0.000 secs) h uid search *:* * SEARCH 420643 h OK Completed (1 msgs in 0.010 secs) - Alex -- Networking -- only one letter away from not working

Tue Jul 20 07:03:31 2010 jettero [...] cpan.org - Correspondence added

On Tue Jul 20 04:09:46 2010, alex@chmrr.net wrote: Show quoted text

> For your amusement, observe how well Cyrus deals:

Heh. So perhaps no IMAP server is really complete. The specification is pretty complicated. It may be of further amusement that I didn't even understand what '*' did at the time. I figured it was just as good as 1:*, to be honest. Show quoted text

> Fixed in 60edd5f.

Hrm, this is in a git repo. I have my own fork on github. Why didn't I just make a patch? Kids make your memory work not as well. -- If riding in an airplane is flying, then riding in a boat is swimming. 116 jumps, 48.6 minutes of freefall, 92.9 freefall miles.

Wed Jul 28 12:54:40 2010 cpan [...] chmrr.net - Status changed from 'open' to 'resolved'