]> Sergey Matveev's repositories - public-inbox.git/blobdiff - script/public-inbox-gcf2
gcf2: require git dir with OID
[public-inbox.git] / script / public-inbox-gcf2
index 51811698dae645b9faf4ecdfa37e1ddffd49259f..4a44b654efe0bedd62939834599d38f4eb25b832 100755 (executable)
@@ -4,11 +4,28 @@
 eval { require PublicInbox::Gcf2 };
 die "libgit2 development package or Inline::C missing for $0: $@\n" if $@;
 my $gcf2 = PublicInbox::Gcf2::new();
+use IO::Handle; # autoflush
+STDERR->autoflush(1);
+STDOUT->autoflush(1);
+
 while (<STDIN>) {
        chomp;
-       if (m!\A/!) { # +/path/to/git-dir
-               $gcf2->add_alternate("$_/objects");
-       } else {
-               $gcf2->cat_oid(1, $_);
+       my ($oid, $git_dir) = split(/ /, $_, 2);
+       $gcf2->add_alternate("$git_dir/objects");
+       if (!$gcf2->cat_oid(1, $oid)) {
+               # retry once if missing.  We only get unabbreviated OIDs
+               # from SQLite or Xapian DBs, here, so malicious clients
+               # can't trigger excessive retries:
+               warn "I: $$ $oid missing, retrying in $git_dir...\n";
+
+               $gcf2 = PublicInbox::Gcf2::new();
+               $gcf2->add_alternate("$git_dir/objects");
+
+               if ($gcf2->cat_oid(1, $oid)) {
+                       warn "I: $$ $oid found after retry\n";
+               } else {
+                       warn "W: $$ $oid missing after retry\n";
+                       print "$oid missing\n"; # mimic git-cat-file
+               }
        }
 }