www: make interface more OO

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

diff --git a/lib/PublicInbox/WWW.pm b/lib/PublicInbox/WWW.pm
index b4b012f962fb81ce0f029f59231fa162b0b5db57..e87e559428ffc55400b9942d4c1561c7197dd44e 100644 (file)
--- a/lib/PublicInbox/WWW.pm
+++ b/lib/PublicInbox/WWW.pm
@@ -21,15 +21,27 @@ require PublicInbox::Git;
 use PublicInbox::GitHTTPBackend;
 our $LISTNAME_RE = qr!\A/([\w\.\-]+)!;
 our $MID_RE = qr!([^/]+)!;
-our $END_RE = qr!(f/|T/|t/|t\.mbox(?:\.gz)?|t\.atom|raw|)!;
-our $pi_config;
+our $END_RE = qr!(f/|T/|t/|R/|t\.mbox(?:\.gz)?|t\.atom|raw|)!;
 
-sub run {
-       my ($cgi, $method) = @_;
+sub new {
+       my ($class, $pi_config) = @_;
        $pi_config ||= PublicInbox::Config->new;
-       my $ctx = { cgi => $cgi, pi_config => $pi_config };
+       bless { pi_config => $pi_config }, $class;
+}
+
+# backwards compatibility, do not use
+sub run {
+       my ($req, $method) = @_;
+       PublicInbox::WWW->new->call($req->env);
+}
+
+sub call {
+       my ($self, $env) = @_;
+       my $cgi = Plack::Request->new($env);
+       my $ctx = { cgi => $cgi, pi_config => $self->{pi_config} };
        my $path_info = $cgi->path_info;
 
+       my $method = $cgi->method;
        if ($method eq 'POST' &&
                 $path_info =~ m!$LISTNAME_RE/(git-upload-pack)\z!) {
                my $path = $2;
@@ -58,7 +70,7 @@ sub run {
                msg_page($ctx, $1, $2, $3);
 
        # in case people leave off the trailing slash:
-       } elsif ($path_info =~ m!$LISTNAME_RE/$MID_RE/(f|T|t)\z!o) {
+       } elsif ($path_info =~ m!$LISTNAME_RE/$MID_RE/(f|T|t|R)\z!o) {
                my ($listname, $mid, $suffix) = ($1, $2, $3);
                $suffix .= $suffix =~ /\A[tT]\z/ ? '/#u' : '/';
                r301($ctx, $listname, $mid, $suffix);
@@ -107,7 +119,7 @@ sub r { [ $_[0], ['Content-Type' => 'text/plain'], [ join(' ', @_, "\n") ] ] }
 # returns undef if valid, array ref response if invalid
 sub invalid_list {
        my ($ctx, $listname) = @_;
-       my $git_dir = $pi_config->get($listname, "mainrepo");
+       my $git_dir = $ctx->{pi_config}->get($listname, "mainrepo");
        if (defined $git_dir) {
                $ctx->{git_dir} = $git_dir;
                $ctx->{git} = PublicInbox::Git->new($git_dir);
@@ -200,6 +212,19 @@ sub get_full_html {
          [ PublicInbox::View::msg_html($ctx, $mime, undef, $foot)] ];
 }
 
+# /$LISTNAME/$MESSAGE_ID/R/                   -> HTML content (fullquotes)
+sub get_reply_html {
+       my ($ctx) = @_;
+       my $x = mid2blob($ctx) or return r404($ctx);
+
+       require PublicInbox::View;
+       my $foot = footer($ctx);
+       require Email::MIME;
+       my $hdr = Email::MIME->new($x)->header_obj;
+       [ 200, [ 'Content-Type' => 'text/html; charset=UTF-8' ],
+         [ PublicInbox::View::msg_reply($ctx, $hdr, $foot)] ];
+}
+
 # /$LISTNAME/$MESSAGE_ID/t/
 sub get_thread {
        my ($ctx, $flat) = @_;
@@ -210,12 +235,6 @@ sub get_thread {
        PublicInbox::View::thread_html($ctx, $foot, $srch);
 }
 
-sub self_url {
-       my ($cgi) = @_;
-                                               # Plack::Request
-       ref($cgi) eq 'CGI' ? $cgi->self_url : $cgi->uri->as_string;
-}
-
 sub ctx_get {
        my ($ctx, $key) = @_;
        my $val = $ctx->{$key};
@@ -244,18 +263,20 @@ sub footer {
 
        my $urls = try_cat("$git_dir/cloneurl");
        my @urls = split(/\r?\n/, $urls || '');
-       my $nurls = scalar @urls;
-       if ($nurls == 0) {
-               $urls = '($GIT_DIR/cloneurl missing)';
-       } elsif ($nurls == 1) {
-               $urls = "git URL for <a\nhref=\"" . SSOMA_URL .
-                       '">ssoma</a>: ' . $urls[0];
+       my %seen = map { $_ => 1 } @urls;
+       my $http = $ctx->{cgi}->base->as_string . $listname;
+       $seen{$http} or unshift @urls, $http;
+       if (scalar(@urls) == 1) {
+               $urls = "URL for <a\nhref=\"" . SSOMA_URL .
+                       qq(">ssoma</a> or <b>git clone --mirror \$URL</b> :) .
+                       $urls[0];
        } else {
-               $urls = "git URLs for <a\nhref=\"" . SSOMA_URL .
-                       "\">ssoma</a>:\n" . join("\n", map { "\t$_" } @urls);
+               $urls = "URLs for <a\nhref=\"" . SSOMA_URL .
+                       qq(">ssoma</a> or <b>git clone --mirror \$URL</b>\n) .
+                       join("\n", map { "\t$_" } @urls);
        }
 
-       my $addr = $pi_config->get($listname, 'address');
+       my $addr = $ctx->{pi_config}->get($listname, 'address');
        if (ref($addr) eq 'ARRAY') {
                $addr = $addr->[0]; # first address is primary
        }
@@ -308,7 +329,7 @@ sub get_thread_mbox {
 sub get_thread_atom {
        my ($ctx) = @_;
        searcher($ctx) or return need_search($ctx);
-       $ctx->{self_url} = self_url($ctx->{cgi});
+       $ctx->{self_url} = $ctx->{cgi}->uri->as_string;
        require PublicInbox::Feed;
        PublicInbox::Feed::generate_thread_atom($ctx);
 }
@@ -373,16 +394,8 @@ sub r301 {
        my ($ctx, $listname, $mid, $suffix) = @_;
        my $cgi = $ctx->{cgi};
        my $url;
-       my $qs;
-       if (ref($cgi) eq 'CGI') {
-               $url = $cgi->url(-base) . '/';
-               $qs = $cgi->query_string;
-       } else { # Plack::Request
-               $url = $cgi->base->as_string;
-               $qs = $cgi->env->{QUERY_STRING};
-       }
-
-       $url .= $listname . '/';
+       my $qs = $cgi->env->{QUERY_STRING};
+       $url = $cgi->base->as_string . $listname . '/';
        $url .= (uri_escape_utf8($mid) . '/') if (defined $mid);
        $url .= $suffix if (defined $suffix);
        $url .= "?$qs" if $qs ne '';
@@ -403,6 +416,7 @@ sub msg_page {
                'T/' eq $e and return get_thread($ctx, 1);
                'raw' eq $e and return get_mid_txt($ctx);
                'f/' eq $e and return get_full_html($ctx);
+               'R/' eq $e and return get_reply_html($ctx);
        }
        r404($ctx);
 }