From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.2 required=3.0 tests=ALL_TRUSTED,BAYES_00, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF shortcircuit=no autolearn=ham autolearn_force=no version=3.4.6 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id E3E1E1FA29 for ; Thu, 12 Jan 2023 14:14:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=80x24.org; s=selector1; t=1673532888; bh=ryKvN6uSAZpIVERn4pNRsnR2uoEJD7w26qjDGvRLdVE=; h=From:To:Subject:Date:In-Reply-To:References:From; b=GjTz/9f4bQpJWBtOruNTHpZGYIRLvE7lztjfvRgorJdjaUFNU68oJf9BWv9viiUg0 NbiwR0YO+XgsKpekVpxap5PYQjVg4jXlCRT0M4c6/gvRlu+sSKy86I/TIH8mXlG0Ip DVsYdSJEDJ+s9k8EwUs30haEhZeQcKIPrZRa5xiA= From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 2/3] www_coderepo: /tree/ redirects to /$OID/s/ Date: Thu, 12 Jan 2023 14:14:34 +0000 Message-Id: <20230112141435.1924376-3-e@80x24.org> In-Reply-To: <20230112141435.1924376-1-e@80x24.org> References: <20230112141435.1924376-1-e@80x24.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: This is for compatibility with cgit to ease migration. --- MANIFEST | 1 + lib/PublicInbox/GitAsyncCat.pm | 2 +- lib/PublicInbox/RepoTree.pm | 44 ++++++++++++++++++++++++++++++++++ lib/PublicInbox/WwwCoderepo.pm | 6 +++++ t/solver_git.t | 7 ++++++ 5 files changed, 59 insertions(+), 1 deletion(-) create mode 100644 lib/PublicInbox/RepoTree.pm diff --git a/MANIFEST b/MANIFEST index 3626e4d2..c494d6f7 100644 --- a/MANIFEST +++ b/MANIFEST @@ -311,6 +311,7 @@ lib/PublicInbox/Qspawn.pm lib/PublicInbox/Reply.pm lib/PublicInbox/RepoAtom.pm lib/PublicInbox/RepoSnapshot.pm +lib/PublicInbox/RepoTree.pm lib/PublicInbox/SaPlugin/ListMirror.pm lib/PublicInbox/SaPlugin/ListMirror.pod lib/PublicInbox/Search.pm diff --git a/lib/PublicInbox/GitAsyncCat.pm b/lib/PublicInbox/GitAsyncCat.pm index 2e0725a6..6dda7340 100644 --- a/lib/PublicInbox/GitAsyncCat.pm +++ b/lib/PublicInbox/GitAsyncCat.pm @@ -75,7 +75,7 @@ sub ibx_async_cat ($$$$) { } sub async_check ($$$$) { - my ($ibx, $oidish, $cb, $arg) = @_; + my ($ibx, $oidish, $cb, $arg) = @_; # $ibx may be $ctx my $git = $ibx->{git} // $ibx->git; $git->check_async($oidish, $cb, $arg); $git->{async_chk} //= do { diff --git a/lib/PublicInbox/RepoTree.pm b/lib/PublicInbox/RepoTree.pm new file mode 100644 index 00000000..7f2ff206 --- /dev/null +++ b/lib/PublicInbox/RepoTree.pm @@ -0,0 +1,44 @@ +# Copyright (C) all contributors +# License: AGPL-3.0+ +# +# cgit-compatible $REPO/tree/[PATH]?h=$tip redirector +package PublicInbox::RepoTree; +use v5.12; +use PublicInbox::ViewDiff qw(uri_escape_path); +use PublicInbox::GitAsyncCat; +use PublicInbox::WwwStatic qw(r); + +sub tree_30x { # git check_async callback + my ($oid, $type, $size, $ctx) = @_; + my $wcb = delete $ctx->{-wcb}; + return $wcb->(r(404)) if $type eq 'missing'; + my $u = $ctx->{git}->base_url($ctx->{env}); + my $path = uri_escape_path(delete $ctx->{-path}); + $u .= "$oid/s/?b=$path"; + $wcb->([ 302, [ Location => $u, 'Content-Type' => 'text/plain' ], + [ "Redirecting to $u\n" ] ]) +} + +sub srv_tree { + my ($ctx, $path) = @_; + return if index($path, '//') >= 0 || index($path, '/') == 0; + my $tip = $ctx->{qp}->{h} // 'HEAD'; + $path =~ s!/\z!!; + my $obj = $ctx->{-obj} = "$tip:$path"; + $ctx->{-path} = $path; + + # "\n" breaks with `git cat-file --batch-check', and there's no + # legitimate use of "\n" in filenames anyways. + return if index($obj, "\n") >= 0; + sub { + $ctx->{-wcb} = $_[0]; # HTTP::{Chunked,Identity} + if ($ctx->{env}->{'pi-httpd.async'}) { + async_check($ctx, $obj, \&tree_30x, $ctx); + } else { + $ctx->{git}->check_async($obj, \&tree_30x, $ctx); + $ctx->{git}->async_wait_all; + } + }; +} + +1; diff --git a/lib/PublicInbox/WwwCoderepo.pm b/lib/PublicInbox/WwwCoderepo.pm index 2fba0cd0..668b6398 100644 --- a/lib/PublicInbox/WwwCoderepo.pm +++ b/lib/PublicInbox/WwwCoderepo.pm @@ -18,6 +18,7 @@ use PublicInbox::Hval qw(ascii_html); use PublicInbox::ViewDiff qw(uri_escape_path); use PublicInbox::RepoSnapshot; use PublicInbox::RepoAtom; +use PublicInbox::RepoTree; my $EACH_REF = "git for-each-ref --sort=-creatordate --format='%(HEAD)%00". join('%00', map { "%($_)" } @@ -226,6 +227,11 @@ sub srv { # endpoint called by PublicInbox::WWW ($ctx->{git} = $cr->{$1}) and return PublicInbox::ViewVCS::show($ctx, $2); + if ($path_info =~ m!\A/(.+?)/tree/(.*)\z! and + ($ctx->{git} = $cr->{$1})) { + return PublicInbox::RepoTree::srv_tree($ctx, $2) // r(404); + } + # snapshots: if ($path_info =~ m!\A/(.+?)/snapshot/([^/]+)\z! and ($ctx->{git} = $cr->{$1})) { diff --git a/t/solver_git.t b/t/solver_git.t index 89ed0362..5519fa18 100644 --- a/t/solver_git.t +++ b/t/solver_git.t @@ -381,6 +381,13 @@ EOF $res = $cb->(GET('/public-inbox/atom/README.md')); is($res->code, 404, '404 on non-existent file Atom feed'); } + + $res = $cb->(GET('/public-inbox/tree/')); + is($res->code, 302, 'got redirect'); + $res = $cb->(GET('/public-inbox/tree/README')); + is($res->code, 302, 'got redirect for regular file'); + $res = $cb->(GET('/public-inbox/tree/Documentation')); + is($res->code, 302, 'got redirect for directory'); }; test_psgi(sub { $www->call(@_) }, $client); my $env = { PI_CONFIG => $cfgpath, TMPDIR => $tmpdir };