Bug #66004 for Mail-IMAPClient: internaldate() returns corrupt data if server does not include internaldate in reply

Mon Feb 21 11:57:51 2011 jason [...] long.name - Ticket created

Subject:

internaldate() returns corrupt data if server does not include internaldate in reply

The code: my $datestr = $IMAP->internaldate($i) or die "Could not internaldate: $@\n"; What the IMAPClient sends: 192 UID FETCH 23 INTERNALDATE What the IMAPClient gets from the IMAP server: * 23 FETCH (UID 23) 192 OK UID FETCH completed Yeah, I suppose the server is in error there; I guess it is supposed to include the internaldate. But it doesn't. The internaldate() method returns garbage in this case. Please make it so that internaldate() throws an error or returns undef. Either would be better than returning '* 23 FETCH...'. Thanks.

Mon Feb 28 22:56:52 2011 PLOBBES [...] cpan.org - Correspondence added

Agreed, this behavior is not desirable. Either undef or "" would be better candidates. I believe undef is probably the best solution so I'll include a patch to the next release to do that. Thanks for reporting this bug!

Mon Feb 28 22:56:53 2011 The RT System itself - Status changed from 'new' to 'open'

Mon Feb 28 22:56:53 2011 PLOBBES [...] cpan.org - Given to PLOBBES

Fri Mar 04 00:41:21 2011 PLOBBES [...] cpan.org - Correspondence added

Here's a preview of the patch to be released with version 3.28: @@ -2674,10 +2674,8 @@ sub internaldate { my ( $self, $msg ) = @_; $self->_imap_uid_command( FETCH => $msg, 'INTERNALDATE' ) or return undef; - my $internalDate = join '', $self->History; - $internalDate =~ s/^.*INTERNALDATE "//si; - $internalDate =~ s/\".*$//s; - $internalDate; + my $hist = join '', $self->History; + return $hist =~ /\bINTERNALDATE "([^"]*)"/i ? $1 : undef; }

Fri Mar 04 00:41:22 2011 PLOBBES [...] cpan.org - Status changed from 'open' to 'patched'

Sat Mar 05 00:12:11 2011 PLOBBES [...] cpan.org - Fixed in 3.28 added

Sat Mar 05 00:12:11 2011 PLOBBES [...] cpan.org - Broken in 3.00 added

Sat Mar 05 00:12:11 2011 PLOBBES [...] cpan.org - Broken in 3.25 deleted

Sat Mar 05 00:12:11 2011 PLOBBES [...] cpan.org - Correspondence added

Mail::IMAPClient 3.28 released today!

Sat Mar 05 00:12:12 2011 The RT System itself - Status changed from 'patched' to 'open'

Sat Mar 05 00:12:12 2011 PLOBBES [...] cpan.org - Status changed from 'open' to 'resolved'