diff options
author | Eric Wong <e@yhbt.net> | 2020-06-10 07:04:47 +0000 |
---|---|---|
committer | Eric Wong <e@yhbt.net> | 2020-06-13 07:55:45 +0000 |
commit | 0e6cf7c861657f79011b90036b5ade4d209ae60f (patch) | |
tree | da22899a2338fc77a58e5fcb55a6a6b628fb43b3 /lib/PublicInbox/IMAP.pm | |
parent | 59f7d24923af08155469420e43e04bce398065da (diff) | |
download | public-inbox-0e6cf7c861657f79011b90036b5ade4d209ae60f.tar.gz |
No point in spewing "uninitialized" warnings into logs when the cat jumps on the Enter key.
Diffstat (limited to 'lib/PublicInbox/IMAP.pm')
-rw-r--r-- | lib/PublicInbox/IMAP.pm | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/lib/PublicInbox/IMAP.pm b/lib/PublicInbox/IMAP.pm index e726307a..0452d6df 100644 --- a/lib/PublicInbox/IMAP.pm +++ b/lib/PublicInbox/IMAP.pm @@ -862,14 +862,17 @@ sub process_line ($$) { cmd_done($self, $tag); } else { # this is weird auth_challenge_ok($self) // - "$tag BAD Error in IMAP command $req: ". - "Unknown command\r\n"; + ($tag // '*') . + ' BAD Error in IMAP command '. + ($req // '(???)'). + ": Unknown command\r\n"; } }; my $err = $@; if ($err && $self->{sock}) { $l =~ s/\r?\n//s; err($self, 'error from: %s (%s)', $l, $err); + $tag //= '*'; $res = "$tag BAD program fault - command not performed\r\n"; } return 0 unless defined $res; |