No ext_urls

[public-inbox.git] / lib / PublicInbox / WwwHighlight.pm

diff --git a/lib/PublicInbox/WwwHighlight.pm b/lib/PublicInbox/WwwHighlight.pm
index 0191640182461b1f752158e856aa6d5ba3efc643..75338806a449f3270232c179ecdf3218c5b3f4b0 100644 (file)
--- a/lib/PublicInbox/WwwHighlight.pm
+++ b/lib/PublicInbox/WwwHighlight.pm
@@ -1,4 +1,4 @@
-# Copyright (C) 2019 all contributors <meta@public-inbox.org>
+# Copyright (C) 2019-2021 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
 # Standalone PSGI app to provide syntax highlighting as-a-service
@@ -20,22 +20,16 @@
 
 package PublicInbox::WwwHighlight;
 use strict;
-use warnings;
-use bytes (); # only for bytes::length
-use HTTP::Status qw(status_message);
+use v5.10.1;
 use parent qw(PublicInbox::HlMod);
+use PublicInbox::Linkify qw();
+use PublicInbox::Hval qw(ascii_html);
+use PublicInbox::WwwStatic qw(r);
 
 # TODO: support highlight(1) for distros which don't package the
 # SWIG extension.  Also, there may be admins who don't want to
 # have ugly SWIG-generated code in a long-lived Perl process.
 
-sub r ($) {
-       my ($code) = @_;
-       my $msg = status_message($code);
-       my $len = length($msg);
-       [ $code, [qw(Content-Type text/plain Content-Length), $len], [$msg] ]
-}
-
 # another slurp API hogging up all my memory :<
 # This is capped by whatever the PSGI server allows,
 # $ENV{GIT_HTTP_MAX_REQUEST_BUFFER} for PublicInbox::HTTP (10 MB)
@@ -52,7 +46,7 @@ sub read_in_full ($) {
                return \$buf if $r == 0;
                $off += $r;
        }
-       $env->{'psgi.errors'}->print("input read error: $!\n");
+       warn "input read error: $!";
        undef;
 }
 
@@ -64,10 +58,17 @@ sub call {
        return r(405) if $req_method ne 'PUT';
 
        my $bref = read_in_full($env) or return r(500);
-       $bref = $self->do_hl($bref, $env->{PATH_INFO});
+       my $l = PublicInbox::Linkify->new;
+       $l->linkify_1($$bref);
+       if (my $res = $self->do_hl($bref, $env->{PATH_INFO})) {
+               $bref = $res;
+       } else {
+               $$bref = ascii_html($$bref);
+       }
+       $l->linkify_2($$bref);
 
        my $h = [ 'Content-Type', 'text/html; charset=UTF-8' ];
-       push @$h, 'Content-Length', bytes::length($$bref);
+       push @$h, 'Content-Length', length($$bref);
 
        [ 200, $h, [ $$bref ] ]
 }