diff options
| author | Eric Wong <e@80x24.org> | 2019-07-07 06:57:43 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2019-07-07 06:57:43 +0000 |
| commit | 71e8a392d465bf4de60f37ecb3dec5cea5c3a376 (patch) | |
| tree | e5cff0497bd65b1aacf2f0bbd1c59b54e6434f99 /lib/PublicInbox/NNTP.pm | |
| parent | a3c054cbbf2a51be121b3237c5d223acc5c8a2f4 (diff) | |
| download | public-inbox-71e8a392d465bf4de60f37ecb3dec5cea5c3a376.tar.gz | |
Add some checks for errors at initialization, though there's not
much that can be done with ENOMEM-type errors aside from
dropping clients.
We can also get rid of the scary FIXME for MemLevel=8. It was a
stupid error on my part in the original per-client deflate
stream implementation calling C::R::Z::{Inflate,Deflate} in
array context and getting the extra dualvar error code as a
string result, causing the {zin}/{zout} array refs to have
extra array elements.
Diffstat (limited to 'lib/PublicInbox/NNTP.pm')
| -rw-r--r-- | lib/PublicInbox/NNTP.pm | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/PublicInbox/NNTP.pm b/lib/PublicInbox/NNTP.pm index 895858b7..6fee29f4 100644 --- a/lib/PublicInbox/NNTP.pm +++ b/lib/PublicInbox/NNTP.pm @@ -922,9 +922,8 @@ sub cmd_starttls ($) { # RFC 8054 sub cmd_compress ($$) { my ($self, $alg) = @_; - return '503 Only the DEFLATE is supported' if uc($alg) ne 'DEFLATE'; + return '503 Only DEFLATE is supported' if uc($alg) ne 'DEFLATE'; return r502 if $self->compressed || !$have_deflate; - res($self, '206 Compression active'); PublicInbox::NNTPdeflate->enable($self); $self->requeue; undef |