about summary refs log tree commit homepage
path: root/lib/PublicInbox/ExtMsg.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2015-09-03 01:57:11 +0000
committerEric Wong <e@80x24.org>2015-09-03 02:04:40 +0000
commit81a9c1b476987d845b340ab9013d26cf4487cb9a (patch)
tree65a8c150f97ca35dbc987711ed14d8070027aaf3 /lib/PublicInbox/ExtMsg.pm
parentfd138a0197450d30677441afb014584f34a661da (diff)
downloadpublic-inbox-81a9c1b476987d845b340ab9013d26cf4487cb9a.tar.gz
We'll continue to compress long Message-IDs in URLs (which we know
about), but we will store entire Message-IDs in the Xapian database
to facilitate ease-of-lookups in external databases.
Diffstat (limited to 'lib/PublicInbox/ExtMsg.pm')
-rw-r--r--lib/PublicInbox/ExtMsg.pm9
1 files changed, 4 insertions, 5 deletions
diff --git a/lib/PublicInbox/ExtMsg.pm b/lib/PublicInbox/ExtMsg.pm
index 1c0887cd..bdbff78f 100644
--- a/lib/PublicInbox/ExtMsg.pm
+++ b/lib/PublicInbox/ExtMsg.pm
@@ -12,7 +12,6 @@ sub ext_msg {
         my $pi_config = $ctx->{pi_config};
         my $listname = $ctx->{listname};
         my $mid = $ctx->{mid};
-        my $cmid = mid_compress($mid);
 
         eval { require PublicInbox::Search };
         my $have_xap = $@ ? 0 : 1;
@@ -35,13 +34,13 @@ sub ext_msg {
                 if ($have_xap) {
                         my $doc_id = eval {
                                 my $s = PublicInbox::Search->new($git_dir);
-                                $s->find_unique_doc_id('mid', $cmid);
+                                $s->find_unique_doc_id('mid', $mid);
                         };
                         if ($@) {
                                 # xapian not configured for this repo
                         } else {
                                 # maybe we found it!
-                                return r302($url, $cmid) if (defined $doc_id);
+                                return r302($url, $mid) if (defined $doc_id);
 
                                 # no point in trying the fork fallback if we
                                 # know Xapian is up-to-date but missing the
@@ -55,7 +54,7 @@ sub ext_msg {
         }
 
         # Xapian not installed or configured for some repos
-        my $path = "HEAD:" . mid2path($cmid);
+        my $path = "HEAD:" . mid2path($mid);
 
         foreach my $n (@nox) {
                 my @cmd = ('git', "--git-dir=$n->{git_dir}", 'cat-file',
@@ -70,7 +69,7 @@ sub ext_msg {
                         my $type = eval { local $/; <$fh> };
                         close $fh;
                         if ($? == 0 && $type eq "blob\n") {
-                                return r302($n->{url}, $cmid);
+                                return r302($n->{url}, $mid);
                         }
                 }
         }