]> Sergey Matveev's repositories - public-inbox.git/blob - script/public-inbox-edit
edit: propagate correct editor exit code
[public-inbox.git] / script / public-inbox-edit
1 #!/usr/bin/perl -w
2 # Copyright (C) 2019 all contributors <meta@public-inbox.org>
3 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
4 #
5 # Used for editing messages in a public-inbox.
6 # Supports v2 inboxes only, for now.
7 use strict;
8 use warnings;
9 use Getopt::Long qw(:config gnu_getopt no_ignore_case auto_abbrev);
10 use PublicInbox::AdminEdit;
11 use File::Temp qw(tempfile);
12 use PublicInbox::ContentId qw(content_id);
13 use PublicInbox::MID qw(mid_clean mids);
14 PublicInbox::Admin::check_require('-index');
15 require PublicInbox::MIME;
16 require PublicInbox::InboxWritable;
17 require PublicInbox::Import;
18
19 my $usage = "$0 -m MESSAGE_ID [--all] [INBOX_DIRS]";
20 my $opt = { verbose => 1, all => 0, -min_inbox_version => 2, raw => 0 };
21 my @opt = qw(mid|m=s file|F=s raw);
22 GetOptions($opt, @PublicInbox::AdminEdit::OPT, @opt) or
23         die "bad command-line args\n$usage\n";
24
25 my $editor = $ENV{MAIL_EDITOR}; # e.g. "mutt -f"
26 unless (defined $editor) {
27         my $k = 'publicinbox.mailEditor';
28         if (my $cfg = PublicInbox::Admin::config()) {
29                 $editor = $cfg->{lc($k)};
30         }
31         unless (defined $editor) {
32                 warn "\`$k' not configured, trying \`git var GIT_EDITOR'\n";
33                 chomp($editor = `git var GIT_EDITOR`);
34                 warn "Will use $editor to edit mail\n";
35         }
36 }
37
38 my $mid = $opt->{mid};
39 my $file = $opt->{file};
40 if (defined $mid && defined $file) {
41         die "the --mid and --file options are mutually exclusive\n";
42 }
43
44 my @ibxs = PublicInbox::Admin::resolve_inboxes(\@ARGV, $opt);
45 PublicInbox::AdminEdit::check_editable(\@ibxs);
46
47 my $found = {}; # cid => [ [ibx, smsg] [, [ibx, smsg] ] ]
48
49 sub find_mid ($) {
50         my ($mid) = @_;
51         foreach my $ibx (@ibxs) {
52                 my $over = $ibx->over;
53                 my ($id, $prev);
54                 while (my $smsg = $over->next_by_mid($mid, \$id, \$prev)) {
55                         my $ref = $ibx->msg_by_smsg($smsg);
56                         my $mime = PublicInbox::MIME->new($ref);
57                         my $cid = content_id($mime);
58                         my $tuple = [ $ibx, $smsg ];
59                         push @{$found->{$cid} ||= []}, $tuple
60                 }
61                 delete @$ibx{qw(over mm git search)}; # cleanup
62         }
63         $found;
64 }
65
66 sub show_cmd ($$) {
67         my ($ibx, $smsg) = @_;
68         " GIT_DIR=$ibx->{inboxdir}/all.git \\\n    git show $smsg->{blob}\n";
69 }
70
71 sub show_found () {
72         foreach my $to_edit (values %$found) {
73                 foreach my $tuple (@$to_edit) {
74                         my ($ibx, $smsg) = @$tuple;
75                         warn show_cmd($ibx, $smsg);
76                 }
77         }
78 }
79
80 if (defined($mid)) {
81         $mid = mid_clean($mid);
82         $found = find_mid($mid);
83         my $nr = scalar(keys %$found);
84         die "No message found for <$mid>\n" unless $nr;
85         if ($nr > 1) {
86                 warn <<"";
87 Multiple messages with different content found matching
88 <$mid>:
89
90                 show_found();
91                 die "Use --force to edit all of them\n" if !$opt->{force};
92                 warn "Will edit all of them\n";
93         }
94 } else {
95         open my $fh, '<', $file or die "open($file) failed: $!";
96         my $orig = do { local $/; <$fh> };
97         my $mime = PublicInbox::MIME->new(\$orig);
98         my $mids = mids($mime->header_obj);
99         find_mid($_) for (@$mids); # populates $found
100         my $cid = content_id($mime);
101         my $to_edit = $found->{$cid};
102         unless ($to_edit) {
103                 my $nr = scalar(keys %$found);
104                 if ($nr > 0) {
105                         warn <<"";
106 $nr matches to Message-ID(s) in $file, but none matched content
107 Partial matches below:
108
109                         show_found();
110                 } elsif ($nr == 0) {
111                         $mids = join('', map { "  <$_>\n" } @$mids);
112                         warn <<"";
113 No matching messages found matching Message-ID(s) in $file
114 $mids
115
116                 }
117                 exit 1;
118         }
119         $found = { $cid => $to_edit };
120 }
121
122 my $tmpl = 'public-inbox-edit-XXXXXX';
123 foreach my $to_edit (values %$found) {
124         my ($edit_fh, $edit_fn) = tempfile($tmpl, TMPDIR => 1, UNLINK => 1);
125         $edit_fh->autoflush(1);
126         my ($ibx, $smsg) = @{$to_edit->[0]};
127         my $old_raw = $ibx->msg_by_smsg($smsg);
128         delete @$ibx{qw(over mm git search)}; # cleanup
129
130         my $tmp = $$old_raw;
131         if (!$opt->{raw}) {
132                 my $oid = $smsg->{blob};
133                 print $edit_fh "From mboxrd\@$oid Thu Jan  1 00:00:00 1970\n";
134                 $tmp =~ s/^(>*From )/>$1/gm;
135         }
136         print $edit_fh $tmp or
137                 die "failed to write tempfile for editing: $!";
138
139         # run the editor, respecting spaces/quote
140 retry_edit:
141         if (system(qw(sh -c), $editor.' "$@"', $editor, $edit_fn)) {
142                 if (!(-t STDIN) && !$opt->{force}) {
143                         die "E: $editor failed: $?\n";
144                 }
145                 print STDERR "$editor failed, ";
146                 print STDERR "continuing as forced\n" if $opt->{force};
147                 while (!$opt->{force}) {
148                         print STDERR "(r)etry, (c)ontinue, (q)uit?\n";
149                         chomp(my $op = <STDIN> || '');
150                         $op = lc($op);
151                         goto retry_edit if $op eq 'r';
152                         if ($op eq 'q') {
153                                 # n.b. we'll lose the exit signal, here,
154                                 # oh well; "q" is user-specified anyways.
155                                 exit($? >> 8);
156                         }
157                         last if $op eq 'c'; # continuing
158                         print STDERR "\`$op' not recognized\n";
159                 }
160         }
161
162         # reread the edited file, not using $edit_fh since $EDITOR may
163         # rename/relink $edit_fn
164         open my $new_fh, '<', $edit_fn or
165                 die "can't read edited file ($edit_fn): $!\n";
166         my $new_raw = do { local $/; <$new_fh> };
167
168         if (!$opt->{raw}) {
169                 # get rid of the From we added
170                 $new_raw =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s;
171
172                 # check if user forgot to purge (in mutt) after editing
173                 if ($new_raw =~ /^From /sm) {
174                         if (-t STDIN) {
175                                 print STDERR <<'';
176 Extra "From " lines detected in new mbox.
177 Did you forget to purge the original message from the mbox after editing?
178
179                                 while (1) {
180                                         print STDERR <<"";
181 (y)es to re-edit, (n)o to continue
182
183                                         chomp(my $op = <STDIN> || '');
184                                         $op = lc($op);
185                                         goto retry_edit if $op eq 'y';
186                                         last if $op eq 'n'; # continuing
187                                         print STDERR "\`$op' not recognized\n";
188                                 }
189                         } else { # non-interactive path
190                                 # unlikely to happen, as extra From lines are
191                                 # only a common mistake (for me) with
192                                 # interactive use
193                                 warn <<"";
194 W: possible message boundary splitting error
195
196                         }
197                 }
198                 # unescape what we escaped:
199                 $new_raw =~ s/^>(>*From )/$1/gm;
200         }
201
202         my $new_mime = PublicInbox::MIME->new(\$new_raw);
203         my $old_mime = PublicInbox::MIME->new($old_raw);
204
205         # make sure we don't compare unwanted headers, since mutt adds
206         # Content-Length, Status, and Lines headers:
207         PublicInbox::Import::drop_unwanted_headers($new_mime);
208         PublicInbox::Import::drop_unwanted_headers($old_mime);
209
210         # allow changing Received: and maybe other headers which can
211         # contain sensitive info.
212         my $nhdr = $new_mime->header_obj;
213         my $ohdr = $old_mime->header_obj;
214         if (($nhdr->as_string eq $ohdr->as_string) &&
215             (content_id($new_mime) eq content_id($old_mime))) {
216                 warn "No change detected to:\n", show_cmd($ibx, $smsg);
217
218                 next unless $opt->{verbose};
219                 # should we consider this machine-parseable?
220                 PublicInbox::AdminEdit::show_rewrites(\*STDOUT, $ibx, []);
221                 next;
222         }
223
224         foreach my $tuple (@$to_edit) {
225                 $ibx = PublicInbox::InboxWritable->new($tuple->[0]);
226                 $smsg = $tuple->[1];
227                 my $im = $ibx->importer(0);
228                 my $commits = $im->replace($old_mime, $new_mime);
229                 $im->done;
230                 unless ($commits) {
231                         warn "Failed to replace:\n", show_cmd($ibx, $smsg);
232                         next;
233                 }
234                 next unless $opt->{verbose};
235                 # should we consider this machine-parseable?
236                 PublicInbox::AdminEdit::show_rewrites(\*STDOUT, $ibx, $commits);
237         }
238 }