]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/IMAP.pm
imap: use git-cat-file asynchronously
[public-inbox.git] / lib / PublicInbox / IMAP.pm
index ca9a0ea7d420feb3dcaa562d84fb2f8f2efe72c7..2282e3ce78c6be097b289d04f68ce9c9b0e0842b 100644 (file)
@@ -18,10 +18,13 @@ use base qw(PublicInbox::DS);
 use fields qw(imapd logged_in ibx long_cb -login_tag
        -idle_tag -idle_max);
 use PublicInbox::Eml;
+use PublicInbox::EmlContentFoo qw(parse_content_disposition);
 use PublicInbox::DS qw(now);
 use PublicInbox::Syscall qw(EPOLLIN EPOLLONESHOT);
+use PublicInbox::GitAsyncCat;
 use Text::ParseWords qw(parse_line);
 use Errno qw(EAGAIN);
+
 my $Address;
 for my $mod (qw(Email::Address::XS Mail::Address)) {
        eval "require $mod" or next;
@@ -32,9 +35,6 @@ die "neither Email::Address::XS nor Mail::Address loaded: $@" if !$Address;
 sub LINE_MAX () { 512 } # does RFC 3501 have a limit like RFC 977?
 
 my %FETCH_NEED_BLOB = ( # for future optimization
-       'BODY.PEEK[HEADER]' => 1,
-       'BODY.PEEK[TEXT]' => 1,
-       'BODY.PEEK[]' => 1,
        'BODY[HEADER]' => 1,
        'BODY[TEXT]' => 1,
        'BODY[]' => 1,
@@ -123,10 +123,16 @@ sub cmd_login ($$$$) {
        login_success($self, $tag);
 }
 
+sub cmd_close ($$) {
+       my ($self, $tag) = @_;
+       delete $self->{ibx} ? "$tag OK Close done\r\n"
+                               : "$tag BAD No mailbox\r\n";
+}
+
 sub cmd_logout ($$) {
        my ($self, $tag) = @_;
        delete $self->{logged_in};
-       $self->write(\"* BYE logging out\r\n$tag OK logout completed\r\n");
+       $self->write(\"* BYE logging out\r\n$tag OK Logout done\r\n");
        $self->shutdn; # PublicInbox::DS::shutdn
        undef;
 }
@@ -139,10 +145,10 @@ sub cmd_authenticate ($$$) {
 
 sub cmd_capability ($$) {
        my ($self, $tag) = @_;
-       '* '.capa($self)."\r\n$tag OK\r\n";
+       '* '.capa($self)."\r\n$tag OK Capability done\r\n";
 }
 
-sub cmd_noop ($$) { "$_[1] OK NOOP completed\r\n" }
+sub cmd_noop ($$) { "$_[1] OK Noop done\r\n" }
 
 # called by PublicInbox::InboxIdle
 sub on_inbox_unlock {
@@ -176,7 +182,7 @@ sub cmd_done ($$) {
                return "$tag BAD internal bug\r\n";
        };
        $ibx->unsubscribe_unlock(fileno($self->{sock}));
-       "$idle_tag OK Idle completed\r\n";
+       "$idle_tag OK Idle done\r\n";
 }
 
 sub cmd_examine ($$$) {
@@ -203,7 +209,7 @@ EOF
        $ret .= "* OK [UNSEEN $max]\r\n" if $max;
        $ret .= "* OK [UIDNEXT $uidnext]\r\n" if defined $uidnext;
        $ret .= "* OK [UIDVALIDITY $uidvalidity]\r\n" if defined $uidvalidity;
-       $ret .= "$tag OK [READ-ONLY] EXAMINE/SELECT complete\r\n";
+       $ret .= "$tag OK [READ-ONLY] EXAMINE/SELECT done\r\n";
 }
 
 sub _esc ($) {
@@ -248,11 +254,149 @@ sub eml_envelope ($) {
        ) . ')';
 }
 
-sub uid_fetch_cb { # called by git->cat_async
+sub _esc_hash ($) {
+       my ($hash) = @_;
+       if ($hash && scalar keys %$hash) {
+               $hash = [ %$hash ]; # flatten hash into 1-dimensional array
+               '(' . join(' ', map { _esc($_) } @$hash) . ')';
+       } else {
+               'NIL';
+       }
+}
+
+sub body_disposition ($) {
+       my ($eml) = @_;
+       my $cd = $eml->header_raw('Content-Disposition') or return 'NIL';
+       $cd = parse_content_disposition($cd);
+       my $buf = '('._esc($cd->{type});
+       $buf .= ' ' . _esc_hash(delete $cd->{attributes});
+       $buf .= ')';
+}
+
+sub body_leaf ($$;$) {
+       my ($eml, $structure, $hold) = @_;
+       my $buf = '';
+       $eml->{is_submsg} and # parent was a message/(rfc822|news|global)
+               $buf .= eml_envelope($eml). ' ';
+       my $ct = $eml->ct;
+       $buf .= '('._esc($ct->{type}).' ';
+       $buf .= _esc($ct->{subtype});
+       $buf .= ' ' . _esc_hash(delete $ct->{attributes});
+       $buf .= ' ' . _esc($eml->header_raw('Content-ID'));
+       $buf .= ' ' . _esc($eml->header_raw('Content-Description'));
+       my $cte = $eml->header_raw('Content-Transfer-Encoding') // '7bit';
+       $buf .= ' ' . _esc($cte);
+       $buf .= ' ' . $eml->{imap_body_len};
+       $buf .= ' '.($eml->body_raw =~ tr/\n/\n/) if lc($ct->{type}) eq 'text';
+
+       # for message/(rfc822|global|news), $hold[0] should have envelope
+       $buf .= ' ' . (@$hold ? join('', @$hold) : 'NIL') if $hold;
+
+       if ($structure) {
+               $buf .= ' '._esc($eml->header_raw('Content-MD5'));
+               $buf .= ' '. body_disposition($eml);
+               $buf .= ' '._esc($eml->header_raw('Content-Language'));
+               $buf .= ' '._esc($eml->header_raw('Content-Location'));
+       }
+       $buf .= ')';
+}
+
+sub body_parent ($$$) {
+       my ($eml, $structure, $hold) = @_;
+       my $ct = $eml->ct;
+       my $type = lc($ct->{type});
+       if ($type eq 'multipart') {
+               my $buf = '(';
+               $buf .= @$hold ? join('', @$hold) : 'NIL';
+               $buf .= ' '._esc($ct->{subtype});
+               if ($structure) {
+                       $buf .= ' '._esc_hash(delete $ct->{attributes});
+                       $buf .= ' '.body_disposition($eml);
+                       $buf .= ' '._esc($eml->header_raw('Content-Language'));
+                       $buf .= ' '._esc($eml->header_raw('Content-Location'));
+               }
+               $buf .= ')';
+               @$hold = ($buf);
+       } else { # message/(rfc822|global|news)
+               @$hold = (body_leaf($eml, $structure, $hold));
+       }
+}
+
+# this is gross, but we need to process the parent part AFTER
+# the child parts are done
+sub bodystructure_prep {
+       my ($p, $q) = @_;
+       my ($eml, $depth) = @$p; # ignore idx
+       # set length here, as $eml->{bdy} gets deleted for message/rfc822
+       $eml->{imap_body_len} = length($eml->body_raw);
+       push @$q, $eml, $depth;
+}
+
+# for FETCH BODY and FETCH BODYSTRUCTURE
+sub fetch_body ($;$) {
+       my ($eml, $structure) = @_;
+       my @q;
+       $eml->each_part(\&bodystructure_prep, \@q, 0, 1);
+       my $cur_depth = 0;
+       my @hold;
+       do {
+               my ($part, $depth) = splice(@q, -2);
+               my $is_mp_parent = $depth == ($cur_depth - 1);
+               $cur_depth = $depth;
+
+               if ($is_mp_parent) {
+                       body_parent($part, $structure, \@hold);
+               } else {
+                       unshift @hold, body_leaf($part, $structure);
+               }
+       } while (@q);
+       join('', @hold);
+}
+
+sub dummy_message ($$) {
+       my ($seqno, $ibx) = @_;
+       my $ret = <<EOF;
+From: nobody\@localhost\r
+To: nobody\@localhost\r
+Date: Thu, 01 Jan 1970 00:00:00 +0000\r
+Message-ID: <dummy-$seqno\@$ibx->{newsgroup}>\r
+Subject: dummy message #$seqno\r
+\r
+You're seeing this message because your IMAP client didn't use UIDs.\r
+The message which used to use this sequence number was likely spam\r
+and removed by the administrator.\r
+EOF
+       \$ret;
+}
+
+sub requeue_once ($) {
+       my ($self) = @_;
+       # COMPRESS users all share the same DEFLATE context.
+       # Flush it here to ensure clients don't see
+       # each other's data
+       $self->zflush;
+
+       # no recursion, schedule another call ASAP,
+       # but only after all pending writes are done.
+       # autovivify wbuf:
+       my $new_size = push(@{$self->{wbuf}}, \&long_step);
+
+       # wbuf may be populated by $cb, no need to rearm if so:
+       $self->requeue if $new_size == 1;
+}
+
+sub uid_fetch_cb { # called by git->cat_async via git_async_msg
        my ($bref, $oid, $type, $size, $fetch_m_arg) = @_;
        my ($self, undef, $ibx, undef, undef, $msgs, $want) = @$fetch_m_arg;
        my $smsg = shift @$msgs or die 'BUG: no smsg';
-       $smsg->{blob} eq $oid or die "BUG: $smsg->{blob} != $oid";
+       if (!defined($oid)) {
+               # it's possible to have TOCTOU if an admin runs
+               # public-inbox-(edit|purge), just move onto the next message
+               return requeue_once($self) unless defined $want->{-seqno};
+               $bref = dummy_message($smsg->{num}, $ibx);
+       } else {
+               $smsg->{blob} eq $oid or die "BUG: $smsg->{blob} != $oid";
+       }
        $$bref =~ s/(?<!\r)\n/\r\n/sg; # make strict clients happy
 
        # fixup old bug from import (pre-a0c07cba0e5d8b6a)
@@ -265,8 +409,8 @@ sub uid_fetch_cb { # called by git->cat_async
        $want->{INTERNALDATE} and
                $self->msg_more(' INTERNALDATE "'.$smsg->internaldate.'"');
        $want->{FLAGS} and $self->msg_more(' FLAGS ()');
-       for ('RFC822', 'BODY[]', 'BODY.PEEK[]') {
-               next unless $want->{$_};
+       for ('RFC822', 'BODY[]') {
+               $want->{$_} or next;
                $self->msg_more(" $_ {".length($$bref)."}\r\n");
                $self->msg_more($$bref);
        }
@@ -276,18 +420,25 @@ sub uid_fetch_cb { # called by git->cat_async
        $want->{ENVELOPE} and
                $self->msg_more(' ENVELOPE '.eml_envelope($eml));
 
-       for my $f ('RFC822.HEADER', 'BODY[HEADER]', 'BODY.PEEK[HEADER]') {
-               next unless $want->{$f};
-               $self->msg_more(" $f {".length(${$eml->{hdr}})."}\r\n");
+       for ('RFC822.HEADER', 'BODY[HEADER]') {
+               $want->{$_} or next;
+               $self->msg_more(" $_ {".length(${$eml->{hdr}})."}\r\n");
                $self->msg_more(${$eml->{hdr}});
        }
-       for my $f ('RFC822.TEXT', 'BODY[TEXT]') {
-               next unless $want->{$f};
-               $self->msg_more(" $f {".length($$bref)."}\r\n");
+       for ('RFC822.TEXT', 'BODY[TEXT]') {
+               $want->{$_} or next;
+               $self->msg_more(" $_ {".length($$bref)."}\r\n");
                $self->msg_more($$bref);
        }
-       # TODO BODY/BODYSTRUCTURE, specific headers
+       $want->{BODYSTRUCTURE} and
+               $self->msg_more(' BODYSTRUCTURE '.fetch_body($eml, 1));
+       $want->{BODY} and
+               $self->msg_more(' BODY '.fetch_body($eml));
+       if (my $partial = $want->{-partial}) {
+               partial_emit($self, $partial, $eml);
+       }
        $self->msg_more(")\r\n");
+       requeue_once($self);
 }
 
 sub uid_fetch_m { # long_response
@@ -300,11 +451,7 @@ sub uid_fetch_m { # long_response
                }
                $$beg = $msgs->[-1]->{num} + 1;
        }
-       my $git = $ibx->git;
-       $git->cat_async_begin; # TODO: actually make async
-       $git->cat_async($msgs->[0]->{blob}, \&uid_fetch_cb, \@_);
-       $git->cat_async_wait;
-       1;
+       git_async_msg($ibx, $msgs->[0], \&uid_fetch_cb, \@_);
 }
 
 sub cmd_status ($$$;@) {
@@ -333,7 +480,7 @@ sub cmd_status ($$$;@) {
        }
        return "$tag BAD no items\r\n" if !@it;
        "* STATUS $mailbox (".join(' ', @it).")\r\n" .
-       "$tag OK Status complete\r\n";
+       "$tag OK Status done\r\n";
 }
 
 my %patmap = ('*' => '.*', '%' => '[^\.]*');
@@ -347,19 +494,166 @@ sub cmd_list ($$$$) {
                $wildcard =~ s!([^a-z0-9_])!$patmap{$1} // "\Q$1"!eig;
                $l = [ grep(/ \Q$refname\E$wildcard\r\n\z/s, @$l) ];
        }
-       \(join('', @$l, "$tag OK List complete\r\n"));
+       \(join('', @$l, "$tag OK List done\r\n"));
 }
 
-sub cmd_uid_fetch ($$$;@) {
-       my ($self, $tag, $range, @want) = @_;
+sub eml_index_offs_i { # PublicInbox::Eml::each_part callback
+       my ($p, $all) = @_;
+       my ($eml, undef, $idx) = @$p;
+       if ($idx && lc($eml->ct->{type}) eq 'multipart') {
+               $eml->{imap_bdy} = $eml->{bdy} // \'';
+       }
+       $all->{$idx} = $eml; # $idx => Eml
+}
+
+# prepares an index for BODY[$SECTION_IDX] fetches
+sub eml_body_idx ($$) {
+       my ($eml, $section_idx) = @_;
+       my $idx = $eml->{imap_all_parts} //= do {
+               my $all = {};
+               $eml->each_part(\&eml_index_offs_i, $all, 0, 1);
+               # top-level of multipart, BODY[0] not allowed (nz-number)
+               delete $all->{0};
+               $all;
+       };
+       $idx->{$section_idx};
+}
+
+# BODY[($SECTION_IDX)?(.$SECTION_NAME)?]<$offset.$bytes>
+sub partial_body {
+       my ($eml, $section_idx, $section_name) = @_;
+       if (defined $section_idx) {
+               $eml = eml_body_idx($eml, $section_idx) or return;
+       }
+       if (defined $section_name) {
+               if ($section_name eq 'MIME') {
+                       # RFC 3501 6.4.5 states:
+                       #       The MIME part specifier MUST be prefixed
+                       #       by one or more numeric part specifiers
+                       return unless defined $section_idx;
+                       return $eml->header_obj->as_string . "\r\n";
+               }
+               my $bdy = $eml->{bdy} // $eml->{imap_bdy} // \'';
+               $eml = PublicInbox::Eml->new($$bdy);
+               if ($section_name eq 'TEXT') {
+                       return $eml->body_raw;
+               } elsif ($section_name eq 'HEADER') {
+                       return $eml->header_obj->as_string . "\r\n";
+               } else {
+                       die "BUG: bad section_name=$section_name";
+               }
+       }
+       ${$eml->{bdy} // $eml->{imap_bdy} // \''};
+}
+
+# similar to what's in PublicInbox::Eml::re_memo, but doesn't memoize
+# to avoid OOM with malicious users
+sub hdrs_regexp ($) {
+       my ($hdrs) = @_;
+       my $names = join('|', map { "\Q$_" } split(/[ \t]+/, $hdrs));
+       qr/^(?:$names):[ \t]*[^\n]*\r?\n # 1st line
+               # continuation lines:
+               (?:[^:\n]*?[ \t]+[^\n]*\r?\n)*
+               /ismx;
+}
+
+# BODY[($SECTION_IDX.)?HEADER.FIELDS.NOT ($HDRS)]<$offset.$bytes>
+sub partial_hdr_not {
+       my ($eml, $section_idx, $hdrs_re) = @_;
+       if (defined $section_idx) {
+               $eml = eml_body_idx($eml, $section_idx) or return;
+       }
+       my $str = $eml->header_obj->as_string;
+       $str =~ s/$hdrs_re//g;
+       $str .= "\r\n";
+}
+
+# BODY[($SECTION_IDX.)?HEADER.FIELDS ($HDRS)]<$offset.$bytes>
+sub partial_hdr_get {
+       my ($eml, $section_idx, $hdrs_re) = @_;
+       if (defined $section_idx) {
+               $eml = eml_body_idx($eml, $section_idx) or return;
+       }
+       my $str = $eml->header_obj->as_string;
+       join('', ($str =~ m/($hdrs_re)/g), "\r\n");
+}
+
+sub partial_prepare ($$$) {
+       my ($partial, $want, $att) = @_;
+
+       # recombine [ "BODY[1.HEADER.FIELDS", "(foo", "bar)]" ]
+       # back to: "BODY[1.HEADER.FIELDS (foo bar)]"
+       return unless $att =~ /\ABODY\[/s;
+       until (rindex($att, ']') >= 0) {
+               my $next = shift @$want or return;
+               $att .= ' ' . uc($next);
+       }
+       if ($att =~ /\ABODY\[([0-9]+(?:\.[0-9]+)*)? # 1 - section_idx
+                       (?:\.(HEADER|MIME|TEXT))? # 2 - section_name
+                       \](?:<([0-9]+)(?:\.([0-9]+))?>)?\z/sx) { # 3, 4
+               $partial->{$att} = [ \&partial_body, $1, $2, $3, $4 ];
+       } elsif ($att =~ /\ABODY\[(?:([0-9]+(?:\.[0-9]+)*)\.)? # 1 - section_idx
+                               (?:HEADER\.FIELDS(\.NOT)?)\x20 # 2
+                               \(([A-Z0-9\-\x20]+)\) # 3 - hdrs
+                       \](?:<([0-9]+)(?:\.([0-9]+))?>)?\z/sx) { # 4 5
+               my $tmp = $partial->{$att} = [ $2 ? \&partial_hdr_not
+                                               : \&partial_hdr_get,
+                                               $1, undef, $4, $5 ];
+               $tmp->[2] = hdrs_regexp($3);
+       } else {
+               undef;
+       }
+}
+
+sub partial_emit ($$$) {
+       my ($self, $partial, $eml) = @_;
+       for (@$partial) {
+               my ($k, $cb, @args) = @$_;
+               my ($offset, $len) = splice(@args, -2);
+               # $cb is partial_body|partial_hdr_get|partial_hdr_not
+               my $str = $cb->($eml, @args) // '';
+               if (defined $offset) {
+                       if (defined $len) {
+                               $str = substr($str, $offset, $len);
+                               $k =~ s/\.$len>\z/>/ or warn
+"BUG: unable to remove `.$len>' from `$k'";
+                       } else {
+                               $str = substr($str, $offset);
+                               $len = length($str);
+                       }
+               } else {
+                       $len = length($str);
+               }
+               $self->msg_more(" $k {$len}\r\n");
+               $self->msg_more($str);
+       }
+}
+
+sub fetch_common ($$$$) {
+       my ($self, $tag, $range, $want) = @_;
        my $ibx = $self->{ibx} or return "$tag BAD No mailbox selected\r\n";
-       if ($want[0] =~ s/\A\(//s) {
-               $want[-1] =~ s/\)\z//s or return "$tag BAD no rparen\r\n";
+       if ($want->[0] =~ s/\A\(//s) {
+               $want->[-1] =~ s/\)\z//s or return "$tag BAD no rparen\r\n";
+       }
+       my (%partial, %want);
+       while (defined(my $att = shift @$want)) {
+               $att = uc($att);
+               $att =~ s/\ABODY\.PEEK\[/BODY\[/; # we're read-only
+               my $x = $FETCH_ATT{$att};
+               if ($x) {
+                       %want = (%want, %$x);
+               } elsif (!partial_prepare(\%partial, $want, $att)) {
+                       return "$tag BAD param: $att\r\n";
+               }
+       }
+
+       # stabilize partial order for consistency and ease-of-debugging:
+       if (scalar keys %partial) {
+               $want{-partial} = [ map {;
+                       [ $_, @{$partial{$_}} ]
+               } sort keys %partial ];
        }
-       my %want = map {;
-               my $x = $FETCH_ATT{uc($_)} or return "$tag BAD param: $_\r\n";
-               %$x;
-       } @want;
+
        my ($beg, $end);
        my $msgs = [];
        if ($range =~ /\A([0-9]+):([0-9]+)\z/s) {
@@ -367,14 +661,58 @@ sub cmd_uid_fetch ($$$;@) {
        } elsif ($range =~ /\A([0-9]+):\*\z/s) {
                ($beg, $end) =  ($1, $ibx->mm->max // 0);
        } elsif ($range =~ /\A[0-9]+\z/) {
-               my $smsg = $ibx->over->get_art($range) or return "$tag OK\r\n";
+               my $smsg = $ibx->over->get_art($range) or
+                       return "$tag OK Fetch done\r\n"; # really OK(!)
                push @$msgs, $smsg;
                ($beg, $end) = ($range, 0);
        } else {
-               return "$tag BAD\r\n";
+               return "$tag BAD fetch range\r\n";
+       }
+       [ $tag, $ibx, \$beg, $end, $msgs, \%want ];
+}
+
+sub cmd_uid_fetch ($$$;@) {
+       my ($self, $tag, $range, @want) = @_;
+       my $args = fetch_common($self, $tag, $range, \@want);
+       ref($args) eq 'ARRAY' ?
+               long_response($self, \&uid_fetch_m, @$args) :
+               $args; # error
+}
+
+sub seq_fetch_m { # long_response
+       my ($self, $tag, $ibx, $beg, $end, $msgs, $want) = @_;
+       if (!@$msgs) { # refill
+               @$msgs = @{$ibx->over->query_xover($$beg, $end)};
+               if (!@$msgs) {
+                       $self->write(\"$tag OK Fetch done\r\n");
+                       return;
+               }
+               $$beg = $msgs->[-1]->{num} + 1;
+       }
+       my $seq = $want->{-seqno}++;
+       my $cur_num = $msgs->[0]->{num};
+       if ($cur_num == $seq) { # as expected
+               git_async_msg($ibx, $msgs->[0], \&uid_fetch_cb, \@_);
+       } elsif ($cur_num > $seq) {
+               # send dummy messages until $seq catches up to $cur_num
+               my $smsg = bless { num => $seq, ts => 0 }, 'PublicInbox::Smsg';
+               unshift @$msgs, $smsg;
+               my $bref = dummy_message($seq, $ibx);
+               uid_fetch_cb($bref, undef, undef, undef, \@_);
+               $smsg; # blessed response since uid_fetch_cb requeues
+       } else { # should not happen
+               die "BUG: cur_num=$cur_num < seq=$seq";
        }
-       long_response($self, \&uid_fetch_m, $tag, $ibx,
-                               \$beg, $end, $msgs, \%want);
+}
+
+sub cmd_fetch ($$$;@) {
+       my ($self, $tag, $range, @want) = @_;
+       my $args = fetch_common($self, $tag, $range, \@want);
+       ref($args) eq 'ARRAY' ? do {
+               my $want = $args->[-1];
+               $want->{-seqno} = ${$args->[2]}; # $$beg
+               long_response($self, \&seq_fetch_m, @$args)
+       } : $args; # error
 }
 
 sub uid_search_all { # long_response
@@ -383,7 +721,7 @@ sub uid_search_all { # long_response
        if (scalar(@$uids)) {
                $self->msg_more(join(' ', '', @$uids));
        } else {
-               $self->write(\"\r\n$tag OK\r\n");
+               $self->write(\"\r\n$tag OK Search done\r\n");
                undef;
        }
 }
@@ -394,7 +732,7 @@ sub uid_search_uid_range { # long_response
        if (@$uids) {
                $self->msg_more(join('', map { " $_->[0]" } @$uids));
        } else {
-               $self->write(\"\r\n$tag OK\r\n");
+               $self->write(\"\r\n$tag OK Search done\r\n");
                undef;
        }
 }
@@ -417,12 +755,12 @@ sub cmd_uid_search ($$$;) {
                } elsif ($rest[0] =~ /\A[0-9]+\z/s) {
                        my $uid = $rest[0];
                        $uid = $ibx->over->get_art($uid) ? " $uid" : '';
-                       "* SEARCH$uid\r\n$tag OK\r\n";
+                       "* SEARCH$uid\r\n$tag OK Search done\r\n";
                } else {
-                       "$tag BAD\r\n";
+                       "$tag BAD Error\r\n";
                }
        } else {
-               "$tag BAD\r\n";
+               "$tag BAD Error\r\n";
        }
 }
 
@@ -484,17 +822,8 @@ sub long_step {
        } elsif ($more) { # $self->{wbuf}:
                $self->update_idle_time;
 
-               # COMPRESS users all share the same DEFLATE context.
-               # Flush it here to ensure clients don't see
-               # each other's data
-               $self->zflush;
-
-               # no recursion, schedule another call ASAP, but only after
-               # all pending writes are done.  autovivify wbuf:
-               my $new_size = push(@{$self->{wbuf}}, \&long_step);
-
-               # wbuf may be populated by $cb, no need to rearm if so:
-               $self->requeue if $new_size == 1;
+               # control passed to $more may be a GitAsyncCat object
+               requeue_once($self) if !ref($more);
        } else { # all done!
                delete $self->{long_cb};
                my $elapsed = now() - $t0;