]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/V2Writable.pm
v2writable: generated Message-ID goes first
[public-inbox.git] / lib / PublicInbox / V2Writable.pm
index 0470fb0ef5a14e6fc44218585000ee337e2095a0..c73d859b8fb880e835fc35ef0457eec8a94bec04 100644 (file)
@@ -11,6 +11,9 @@ use PublicInbox::SearchIdxSkeleton;
 use PublicInbox::MIME;
 use PublicInbox::Git;
 use PublicInbox::Import;
+use PublicInbox::MID qw(mids);
+use PublicInbox::ContentId qw(content_id content_digest);
+use PublicInbox::Inbox;
 
 # an estimate of the post-packed size to the raw uncompressed size
 my $PACKING_FACTOR = 0.4;
@@ -46,22 +49,27 @@ sub new {
 # mimics Import::add and wraps it for v2
 sub add {
        my ($self, $mime, $check_cb) = @_;
-       my $existing = $self->lookup_content($mime);
 
-       if ($existing) {
-               return undef if $existing->type eq 'mail'; # duplicate
+       # spam check:
+       if ($check_cb) {
+               $mime = $check_cb->($mime) or return;
        }
 
-       my $im = $self->importer;
+       # All pipes (> $^F) known to Perl 5.6+ have FD_CLOEXEC set,
+       # as does SQLite 3.4.1+ (released in 2007-07-20), and
+       # Xapian 1.3.2+ (released 2015-03-15).
+       # For the most part, we can spawn git-fast-import without
+       # leaking FDs to it...
+       $self->idx_init;
 
-       # im->add returns undef if check_cb fails
-       my $cmt = $im->add($mime, $check_cb) or return;
+       my $num = num_for($self, $mime);
+       defined $num or return; # duplicate
+       my $im = $self->importer;
+       my $cmt = $im->add($mime);
        $cmt = $im->get_mark($cmt);
        my $oid = $im->{last_object_id};
        my ($len, $msgref) = @{$im->{last_object}};
 
-       $self->idx_init;
-       my $num = $self->{skel}->index_mm($mime, 1);
        my $nparts = $self->{partitions};
        my $part = $num % $nparts;
        my $idx = $self->idx_part($part);
@@ -74,6 +82,70 @@ sub add {
        $mime;
 }
 
+sub num_for {
+       my ($self, $mime) = @_;
+       my $mids = mids($mime->header_obj);
+       if (@$mids) {
+               my $mid = $mids->[0];
+               my $num = $self->{skel}->{mm}->mid_insert($mid);
+               return $num if defined($num); # common case
+
+               # crap, Message-ID is already known, hope somebody just resent:
+               $self->done; # write barrier, clears $self->{skel}
+               foreach my $m (@$mids) {
+                       # read-only lookup now safe to do after above barrier
+                       my $existing = $self->lookup_content($mime, $m);
+                       if ($existing) {
+                               warn "<$m> resent\n";
+                               return; # easy, don't store duplicates
+                       }
+               }
+
+               # very unlikely:
+               warn "<$mid> reused for mismatched content\n";
+               $self->idx_init;
+
+               # try the rest of the mids
+               foreach my $i (1..$#$mids) {
+                       my $m = $mids->[$i];
+                       $num = $self->{skel}->{mm}->mid_insert($m);
+                       if (defined $num) {
+                               warn "alternative <$m> for <$mid> found\n";
+                               return $num;
+                       }
+               }
+       }
+       # none of the existing Message-IDs are good, generate a new one:
+       num_for_harder($self, $mime);
+}
+
+sub num_for_harder {
+       my ($self, $mime) = @_;
+
+       my $hdr = $mime->header_obj;
+       my $dig = content_digest($mime);
+       my $mid = $dig->clone->hexdigest . '@localhost';
+       my $num = $self->{skel}->{mm}->mid_insert($mid);
+       unless (defined $num) {
+               # it's hard to spoof the last Received: header
+               my @recvd = $hdr->header_raw('Received');
+               $dig->add("Received: $_") foreach (@recvd);
+               $mid = $dig->clone->hexdigest . '@localhost';
+               $num = $self->{skel}->{mm}->mid_insert($mid);
+
+               # fall back to a random Message-ID and give up determinism:
+               until (defined($num)) {
+                       $dig->add(rand);
+                       $mid = $dig->clone->hexdigest . '@localhost';
+                       warn "using random Message-ID <$mid> as fallback\n";
+                       $num = $self->{skel}->{mm}->mid_insert($mid);
+               }
+       }
+       my @cur = $hdr->header_raw('Message-Id');
+       $hdr->header_set('Message-Id', "<$mid>", @cur);
+       $num;
+}
+
 sub idx_part {
        my ($self, $part) = @_;
        $self->{idx_parts}->[$part];
@@ -83,6 +155,12 @@ sub idx_part {
 sub idx_init {
        my ($self) = @_;
        return if $self->{idx_parts};
+       my $ibx = $self->{-inbox};
+
+       # do not leak read-only FDs to child processes, we only have these
+       # FDs for duplicate detection so they should not be
+       # frequently activated.
+       delete $ibx->{$_} foreach (qw(git mm search));
 
        # first time initialization, first we create the skeleton pipe:
        my $skel = $self->{skel} = PublicInbox::SearchIdxSkeleton->new($self);
@@ -241,7 +319,29 @@ sub import_init {
 }
 
 sub lookup_content {
-       undef # TODO
+       my ($self, $mime, $mid) = @_;
+       my $ibx = $self->{-inbox};
+
+       my $srch = $ibx->search;
+       my $cid = content_id($mime);
+       my $found;
+       $srch->each_smsg_by_mid($mid, sub {
+               my ($smsg) = @_;
+               $smsg->load_expand;
+               my $msg = $ibx->msg_by_smsg($smsg);
+               if (!defined($msg)) {
+                       warn "broken smsg for $mid\n";
+                       return 1; # continue
+               }
+               my $cur = PublicInbox::MIME->new($msg);
+               if (content_id($cur) eq $cid) {
+                       $smsg->{mime} = $cur;
+                       $found = $smsg;
+                       return 0; # break out of loop
+               }
+               1; # continue
+       });
+       $found;
 }
 
 sub atfork_child {