]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/WwwStream.pm
wwwstream: oneshot sets content-length
[public-inbox.git] / lib / PublicInbox / WwwStream.pm
index ef5897b26a2f8cc7aeadec7d7ba7ddafef41a282..985e02622048b4a2ada72510aa84d4ab6273d7be 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 2016-2019 all contributors <meta@public-inbox.org>
+# Copyright (C) 2016-2020 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 #
 # HTML body stream for which yields getline+close methods
@@ -9,6 +9,7 @@
 package PublicInbox::WwwStream;
 use strict;
 use warnings;
+use bytes (); # length
 use PublicInbox::Hval qw(ascii_html prurl);
 our $TOR_URL = 'https://www.torproject.org/';
 our $CODE_URL = 'https://public-inbox.org/public-inbox.git';
@@ -16,16 +17,21 @@ our $CODE_URL = 'https://public-inbox.org/public-inbox.git';
 # noop for HTTP.pm (and any other PSGI servers)
 sub close {}
 
+sub base_url ($) {
+       my $ctx = shift;
+       my $base_url = $ctx->{-inbox}->base_url($ctx->{env});
+       chop $base_url; # no trailing slash for clone
+       $base_url;
+}
+
 sub new {
        my ($class, $ctx, $cb) = @_;
 
-       my $base_url = $ctx->{-inbox}->base_url($ctx->{env});
-       chop $base_url; # no trailing slash for clone
        bless {
                nr => 0,
-               cb => $cb || \&close,
+               cb => $cb,
                ctx => $ctx,
-               base_url => $base_url,
+               base_url => base_url($ctx),
        }, $class;
 }
 
@@ -40,25 +46,20 @@ sub _html_top ($) {
        my $ctx = $self->{ctx};
        my $ibx = $ctx->{-inbox};
        my $desc = ascii_html($ibx->description);
-       my $title = $ctx->{-title_html} || $desc;
+       my $title = delete($ctx->{-title_html}) // $desc;
        my $upfx = $ctx->{-upfx} || '';
        my $help = $upfx.'_/text/help';
        my $color = $upfx.'_/text/color';
        my $atom = $ctx->{-atom} || $upfx.'new.atom';
-       my $tip = $ctx->{-html_tip} || '';
        my $top = "<b>$desc</b>";
        my $links = "<a\nhref=\"$help\">help</a> / ".
                        "<a\nhref=\"$color\">color</a> / ".
                        "<a\nhref=\"$atom\">Atom feed</a>";
        if ($ibx->search) {
-               my $q_val = $ctx->{-q_value_html};
-               if (defined $q_val && $q_val ne '') {
-                       $q_val = qq(\nvalue="$q_val");
-               } else {
-                       $q_val = '';
-               }
+               my $q_val = delete($ctx->{-q_value_html}) // '';
+               $q_val = qq(\nvalue="$q_val") if $q_val ne '';
                # XXX gross, for SearchView.pm
-               my $extra = $ctx->{-extra_form_html} || '';
+               my $extra = delete($ctx->{-extra_form_html}) // '';
                my $action = $upfx eq '' ? './' : $upfx;
                $top = qq{<form\naction="$action"><pre>$top} .
                          qq{\n<input\nname=q\ntype=text$q_val />} .
@@ -70,10 +71,10 @@ sub _html_top ($) {
                $top = '<pre>' . $top . "\n" . $links . '</pre>';
        }
        "<html><head><title>$title</title>" .
-               "<link\nrel=alternate\ntitle=\"Atom feed\"\n".
-               "href=\"$atom\"\ntype=\"application/atom+xml\"/>" .
+               qq(<link\nrel=alternate\ntitle="Atom feed"\n).
+               qq(href="$atom"\ntype="application/atom+xml"/>) .
                $ctx->{www}->style($upfx) .
-               "</head><body>". $top . $tip;
+               '</head><body>'. $top . (delete($ctx->{-html_tip}) // '');
 }
 
 sub code_footer ($) {
@@ -169,4 +170,19 @@ sub getline {
        delete $self->{cb} ? _html_end($self) : undef;
 }
 
+sub oneshot {
+       my ($ctx, $code, $sref) = @_;
+       my $self = bless {
+               ctx => $ctx,
+               base_url => base_url($ctx),
+       }, __PACKAGE__;
+       my @x = (_html_top($self), $sref ? $$sref : (), _html_end($self));
+       my $len = 0;
+       $len += bytes::length($_) for @x;
+       [ $code, [
+               'Content-Type' => 'text/html; charset=UTF-8',
+               'Content-Length' => $len
+       ], \@x ];
+}
+
 1;