]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Config.pm
No ext_urls
[public-inbox.git] / lib / PublicInbox / Config.pm
index 87a03fd378214aac91dad4b9e9741ebeb9cfb809..cdf06d850e764c7e46f312043414b0fda12df904 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 2014-2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 #
 # Used throughout the project for reading configuration
@@ -12,23 +12,31 @@ use strict;
 use v5.10.1;
 use PublicInbox::Inbox;
 use PublicInbox::Spawn qw(popen_rd);
+our $LD_PRELOAD = $ENV{LD_PRELOAD}; # only valid at startup
+our $DEDUPE; # set to {} to dedupe or clear cache
 
 sub _array ($) { ref($_[0]) eq 'ARRAY' ? $_[0] : [ $_[0] ] }
 
 # returns key-value pairs of config directives in a hash
 # if keys may be multi-value, the value is an array ref containing all values
 sub new {
-       my ($class, $file) = @_;
+       my ($class, $file, $errfh) = @_;
        $file //= default_file();
        my $self;
+       my $set_dedupe;
        if (ref($file) eq 'SCALAR') { # used by some tests
                open my $fh, '<', $file or die;  # PerlIO::scalar
                $self = config_fh_parse($fh, "\n", '=');
+               bless $self, $class;
        } else {
-               $self = git_config_dump($file);
+               if (-f $file && $DEDUPE) {
+                       $file = rel2abs_collapsed($file);
+                       $self = $DEDUPE->{$file} and return $self;
+                       $set_dedupe = 1;
+               }
+               $self = git_config_dump($class, $file, $errfh);
                $self->{'-f'} = $file;
        }
-       bless $self, $class;
        # caches
        $self->{-by_addr} = {};
        $self->{-by_list_id} = {};
@@ -38,7 +46,6 @@ sub new {
        $self->{-no_obfuscate} = {};
        $self->{-limiters} = {};
        $self->{-code_repos} = {}; # nick => PublicInbox::Git object
-       $self->{-cgitrc_unparsed} = $self->{'publicinbox.cgitrc'};
 
        if (my $no = delete $self->{'publicinbox.noobfuscate'}) {
                $no = _array($no);
@@ -61,7 +68,7 @@ sub new {
        if (my $css = delete $self->{'publicinbox.css'}) {
                $self->{css} = _array($css);
        }
-
+       $DEDUPE->{$file} = $self if $set_dedupe;
        $self;
 }
 
@@ -88,12 +95,17 @@ sub lookup_list_id {
 
 sub lookup_name ($$) {
        my ($self, $name) = @_;
-       $self->{-by_name}->{$name} // _fill($self, "publicinbox.$name");
+       $self->{-by_name}->{$name} // _fill_ibx($self, $name);
 }
 
 sub lookup_ei {
        my ($self, $name) = @_;
-       $self->{-ei_by_name}->{$name} //= _fill_ei($self, "extindex.$name");
+       $self->{-ei_by_name}->{$name} //= _fill_ei($self, $name);
+}
+
+sub lookup_eidx_key {
+       my ($self, $eidx_key) = @_;
+       _lookup_fill($self, '-by_eidx_key', $eidx_key);
 }
 
 # special case for [extindex "all"]
@@ -158,17 +170,17 @@ sub config_fh_parse ($$$) {
 }
 
 sub git_config_dump {
-       my ($file) = @_;
-       return {} unless -e $file;
+       my ($class, $file, $errfh) = @_;
+       return bless {}, $class unless -e $file;
        my $cmd = [ qw(git config -z -l --includes), "--file=$file" ];
-       my $fh = popen_rd($cmd);
+       my $fh = popen_rd($cmd, undef, { 2 => $errfh // 2 });
        my $rv = config_fh_parse($fh, "\0", "\n");
-       close $fh or die "failed to close (@$cmd) pipe: $?";
-       $rv;
+       close $fh or die "@$cmd failed: \$?=$?\n";
+       bless $rv, $class;
 }
 
-sub valid_inbox_name ($) {
-       my ($name) = @_;
+sub valid_foo_name ($;$) {
+       my ($name, $pfx) = @_;
 
        # Similar rules found in git.git/remote.c::valid_remote_nick
        # and git.git/refs.c::check_refname_component
@@ -176,6 +188,7 @@ sub valid_inbox_name ($) {
        if ($name eq '' || $name =~ /\@\{/ ||
            $name =~ /\.\./ || $name =~ m![/:\?\[\]\^~\s\f[:cntrl:]\*]! ||
            $name =~ /\A\./ || $name =~ /\.\z/) {
+               warn "invalid $pfx name: `$name'\n" if $pfx;
                return 0;
        }
 
@@ -213,7 +226,6 @@ sub cgit_repo_merge ($$$) {
                        $rel =~ s!/?\.git\z!!;
        }
        $self->{"coderepo.$rel.dir"} //= $path;
-       $self->{"coderepo.$rel.cgiturl"} //= _array($rel);
 }
 
 sub is_git_dir ($) {
@@ -263,6 +275,7 @@ sub scan_projects_coderepo ($$$) {
 
 sub parse_cgitrc {
        my ($self, $cgitrc, $nesting) = @_;
+       $cgitrc //= $self->{'publicinbox.cgitrc'} // return;
        if ($nesting == 0) {
                # defaults:
                my %s = map { $_ => 1 } qw(/cgit.css /cgit.png
@@ -311,49 +324,36 @@ sub parse_cgitrc {
                } elsif (m!\A(?:css|favicon|logo|repo\.logo)=(/.+)\z!) {
                        # absolute paths for static files via PublicInbox::Cgit
                        $self->{-cgit_static}->{$1} = 1;
+               } elsif (s!\Asnapshots=\s*!!) {
+                       $self->{'coderepo.snapshots'} = $_;
                }
        }
        cgit_repo_merge($self, $repo->{dir}, $repo) if $repo;
 }
 
-# parse a code repo
-# Only git is supported at the moment, but SVN and Hg are possibilities
-sub _fill_code_repo {
+# parse a code repo, only git is supported at the moment
+sub fill_code_repo {
        my ($self, $nick) = @_;
        my $pfx = "coderepo.$nick";
-
-       # TODO: support gitweb and other repository viewers?
-       if (defined(my $cgitrc = delete $self->{-cgitrc_unparsed})) {
-               parse_cgitrc($self, $cgitrc, 0);
-       }
-       my $dir = $self->{"$pfx.dir"}; # aka "GIT_DIR"
-       unless (defined $dir) {
+       my $dir = $self->{"$pfx.dir"} // do { # aka "GIT_DIR"
                warn "$pfx.dir unset\n";
                return;
-       }
-
+       };
        my $git = PublicInbox::Git->new($dir);
-       foreach my $t (qw(blob commit tree tag)) {
-               $git->{$t.'_url_format'} =
-                               _array($self->{lc("$pfx.${t}UrlFormat")});
-       }
-
        if (defined(my $cgits = $self->{"$pfx.cgiturl"})) {
                $git->{cgit_url} = $cgits = _array($cgits);
                $self->{"$pfx.cgiturl"} = $cgits;
-
-               # cgit supports "/blob/?id=%s", but it's only a plain-text
-               # display and requires an unabbreviated id=
-               foreach my $t (qw(blob commit tag)) {
-                       $git->{$t.'_url_format'} //= map {
-                               "$_/$t/?id=%s"
-                       } @$cgits;
-               }
        }
-
+       $git->{nick} = $nick;
        $git;
 }
 
+sub get_all {
+       my ($self, $key) = @_;
+       my $v = $self->{$key} // return;
+       _array($v);
+}
+
 sub git_bool {
        my ($val) = $_[-1]; # $_[0] may be $self, or $val
        if ($val =~ /\A(?:false|no|off|[\-\+]?(?:0x)?0+)\z/i) {
@@ -375,26 +375,57 @@ sub rel2abs_collapsed {
        Cwd::abs_path($p);
 }
 
-sub _fill {
-       my ($self, $pfx) = @_;
-       my $ibx = {};
+sub get_1 {
+       my ($self, $key) = @_;
+       my $v = $self->{$key};
+       return $v if !ref($v);
+       warn "W: $key has multiple values, only using `$v->[-1]'\n";
+       $v->[-1];
+}
 
-       for my $k (qw(watch nntpserver)) {
+sub repo_objs {
+       my ($self, $ibxish) = @_;
+       my $ibx_code_repos = $ibxish->{coderepo} // return;
+       $ibxish->{-repo_objs} // do {
+               parse_cgitrc($self, undef, 0);
+               my $code_repos = $self->{-code_repos};
+               my @repo_objs;
+               for my $nick (@$ibx_code_repos) {
+                       my @parts = split(m!/!, $nick);
+                       for (@parts) {
+                               @parts = () unless valid_foo_name($_);
+                       }
+                       unless (@parts) {
+                               warn "invalid coderepo name: `$nick'\n";
+                               next;
+                       }
+                       my $repo = $code_repos->{$nick} //=
+                                               fill_code_repo($self, $nick);
+                       push @repo_objs, $repo if $repo;
+               }
+               if (scalar @repo_objs) {
+                       for (@repo_objs) {
+                               push @{$_->{ibx_names}}, $ibxish->{name};
+                       }
+                       $ibxish->{-repo_objs} = \@repo_objs;
+               } else {
+                       delete $ibxish->{coderepo};
+               }
+       }
+}
+
+sub _fill_ibx {
+       my ($self, $name) = @_;
+       my $pfx = "publicinbox.$name";
+       my $ibx = {};
+       for my $k (qw(watch)) {
                my $v = $self->{"$pfx.$k"};
                $ibx->{$k} = $v if defined $v;
        }
        for my $k (qw(filter inboxdir newsgroup replyto httpbackendmax feedmax
-                       indexlevel indexsequentialshard)) {
-               if (defined(my $v = $self->{"$pfx.$k"})) {
-                       if (ref($v) eq 'ARRAY') {
-                               warn <<EOF;
-W: $pfx.$k has multiple values, only using `$v->[-1]'
-EOF
-                               $ibx->{$k} = $v->[-1];
-                       } else {
-                               $ibx->{$k} = $v;
-                       }
-               }
+                       indexlevel indexsequentialshard boost)) {
+               my $v = get_1($self, "$pfx.$k") // next;
+               $ibx->{$k} = $v;
        }
 
        # "mainrepo" is backwards compatibility:
@@ -403,9 +434,8 @@ EOF
                warn "E: `$dir' must not contain `\\n'\n";
                return;
        }
-       foreach my $k (qw(obfuscate)) {
-               my $v = $self->{"$pfx.$k"};
-               defined $v or next;
+       for my $k (qw(obfuscate)) {
+               my $v = $self->{"$pfx.$k"} // next;
                if (defined(my $bval = git_bool($v))) {
                        $ibx->{$k} = $bval;
                } else {
@@ -414,19 +444,15 @@ EOF
        }
        # TODO: more arrays, we should support multi-value for
        # more things to encourage decentralization
-       foreach my $k (qw(address altid nntpmirror coderepo hide listid url
-                       infourl watchheader)) {
-               if (defined(my $v = $self->{"$pfx.$k"})) {
-                       $ibx->{$k} = _array($v);
-               }
-       }
-
-       my $name = substr($pfx, length('publicinbox.'));
-       if (!valid_inbox_name($name)) {
-               warn "invalid inbox name: '$name'\n";
-               return;
+       for my $k (qw(address altid nntpmirror imapmirror
+                       coderepo hide listid url
+                       infourl watchheader
+                       nntpserver imapserver pop3server)) {
+               my $v = $self->{"$pfx.$k"} // next;
+               $ibx->{$k} = _array($v);
        }
 
+       return unless valid_foo_name($name, 'publicinbox');
        $ibx->{name} = $name;
        $ibx->{-pi_cfg} = $self;
        $ibx = PublicInbox::Inbox->new($ibx);
@@ -473,21 +499,6 @@ EOF
                $ibx->{-no_obfuscate_re} = $self->{-no_obfuscate_re};
                fill_all($self); # noop to populate -no_obfuscate
        }
-
-       if (my $ibx_code_repos = $ibx->{coderepo}) {
-               my $code_repos = $self->{-code_repos};
-               my $repo_objs = $ibx->{-repo_objs} = [];
-               foreach my $nick (@$ibx_code_repos) {
-                       my @parts = split(m!/!, $nick);
-                       my $valid = 0;
-                       $valid += valid_inbox_name($_) foreach (@parts);
-                       $valid == scalar(@parts) or next;
-
-                       my $repo = $code_repos->{$nick} //=
-                                               _fill_code_repo($self, $nick);
-                       push @$repo_objs, $repo if $repo;
-               }
-       }
        if (my $es = ALL($self)) {
                require PublicInbox::Isearch;
                $ibx->{isrch} = PublicInbox::Isearch->new($ibx, $es);
@@ -496,14 +507,31 @@ EOF
 }
 
 sub _fill_ei ($$) {
-       my ($self, $pfx) = @_;
+       my ($self, $name) = @_;
        eval { require PublicInbox::ExtSearch } or return;
-       my $d = $self->{"$pfx.topdir"};
-       defined($d) && -d $d ? PublicInbox::ExtSearch->new($d) : undef;
+       my $pfx = "extindex.$name";
+       my $d = $self->{"$pfx.topdir"} // return;
+       -d $d or return;
+       if (index($d, "\n") >= 0) {
+               warn "E: `$d' must not contain `\\n'\n";
+               return;
+       }
+       my $es = PublicInbox::ExtSearch->new($d);
+       for my $k (qw(indexlevel indexsequentialshard)) {
+               my $v = get_1($self, "$pfx.$k") // next;
+               $es->{$k} = $v;
+       }
+       for my $k (qw(coderepo hide url infourl)) {
+               my $v = $self->{"$pfx.$k"} // next;
+               $es->{$k} = _array($v);
+       }
+       return unless valid_foo_name($name, 'extindex');
+       $es->{name} = $name;
+       $es;
 }
 
 sub urlmatch {
-       my ($self, $key, $url) = @_;
+       my ($self, $key, $url, $try_git) = @_;
        state $urlmatch_broken; # requires git 1.8.5
        return if $urlmatch_broken;
        my $file = $self->{'-f'} // default_file();
@@ -512,13 +540,20 @@ sub urlmatch {
        my $fh = popen_rd($cmd);
        local $/ = "\0";
        my $val = <$fh>;
-       if (close($fh)) {
-               chomp($val);
-               $val;
-       } else {
-               $urlmatch_broken = 1 if (($? >> 8) != 1);
-               undef;
+       if (!close($fh)) {
+               undef $val;
+               if (($? >> 8) != 1) {
+                       $urlmatch_broken = 1;
+               } elsif ($try_git) { # n.b. this takes cwd into account
+                       $cmd = [qw(git config -z --get-urlmatch), $key, $url];
+                       $fh = popen_rd($cmd);
+                       $val = <$fh>;
+                       close($fh) or undef($val);
+               }
        }
+       $? = 0; # don't influence lei exit status
+       chomp $val if defined $val;
+       $val;
 }
 
 sub json {
@@ -533,4 +568,13 @@ sub json {
        };
 }
 
+sub squote_maybe ($) {
+       my ($val) = @_;
+       if ($val =~ m{([^\w@\./,\%\+\-])}) {
+               $val =~ s/(['!])/'\\$1'/g; # '!' for csh
+               return "'$val'";
+       }
+       $val;
+}
+
 1;