X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=cgi%2Fbugreport.cgi;h=4251fed98cbdeb92190b6824735164089150859e;hb=c85e6985055a3adbb0491888b514f8846be2d044;hp=fff927196debf25254b2560af08c0d0078ea5d51;hpb=97d6b5247dcd4351fbf1c6cdd053245d60bf19e6;p=debbugs.git diff --git a/cgi/bugreport.cgi b/cgi/bugreport.cgi index fff9271..4251fed 100755 --- a/cgi/bugreport.cgi +++ b/cgi/bugreport.cgi @@ -1,57 +1,76 @@ #!/usr/bin/perl -wT -package debbugs; - +use warnings; use strict; use POSIX qw(strftime tzset); use MIME::Parser; use MIME::Decoder; use IO::Scalar; -use IO::Lines; use IO::File; -#require '/usr/lib/debbugs/errorlib'; -require './common.pl'; - -require '/etc/debbugs/config'; -require '/etc/debbugs/text'; - -use vars(qw($gEmailDomain $gHTMLTail $gSpoolDir $gWebDomain)); +use Debbugs::Config qw(:globals :text); # for read_log_records -use Debbugs::Log; -use Debbugs::MIME qw(convert_to_utf8 decode_rfc1522); +use Debbugs::Log qw(read_log_records); +use Debbugs::MIME qw(convert_to_utf8 decode_rfc1522 create_mime_message); +use Debbugs::CGI qw(:url :html :util); +use Debbugs::Common qw(buglog getmaintainers); +use Debbugs::Packages qw(getpkgsrc); +use Debbugs::Status qw(splitpackages get_bug_status isstrongseverity); use Scalar::Util qw(looks_like_number); - -my %param = readparse(); +use CGI::Simple; +my $q = new CGI::Simple; + +my %param = cgi_parameters(query => $q, + single => [qw(bug msg att boring terse), + qw(reverse mbox mime trim), + qw(mboxstat mboxmaint archive), + qw(repeatmerged) + ], + default => {msg => '', + boring => 'no', + terse => 'no', + reverse => 'no', + mbox => 'no', + mime => 'no', + mboxstat => 'no', + mboxmaint => 'no', + archive => 'no', + repeatmerged => 'yes', + }, + ); +# This is craptacular. my $tail_html; -my $ref = $param{'bug'} || quitcgi("No bug number"); +my $ref = $param{bug} or quitcgi("No bug number"); $ref =~ /(\d+)/ or quitcgi("Invalid bug number"); $ref = $1; my $short = "#$ref"; -my $msg = $param{'msg'} || ""; +my $msg = $param{'msg'}; my $att = $param{'att'}; -my $boring = ($param{'boring'} || 'no') eq 'yes'; -my $terse = ($param{'terse'} || 'no') eq 'yes'; -my $reverse = ($param{'reverse'} || 'no') eq 'yes'; -my $mbox = ($param{'mbox'} || 'no') eq 'yes'; -my $mime = ($param{'mime'} || 'yes') eq 'yes'; +my $boring = $param{'boring'} eq 'yes'; +my $terse = $param{'terse'} eq 'yes'; +my $reverse = $param{'reverse'} eq 'yes'; +my $mbox = $param{'mbox'} eq 'yes'; +my $mime = $param{'mime'} eq 'yes'; my $trim_headers = ($param{trim} || ($msg?'no':'yes')) eq 'yes'; +my $mbox_status_message = $param{mboxstat} eq 'yes'; +my $mbox_maint = $param{mboxmaint} eq 'yes'; +$mbox = 1 if $mbox_status_message or $mbox_maint; + + # Not used by this script directly, but fetch these so that pkgurl() and # friends can propagate them correctly. -my $archive = ($param{'archive'} || 'no') eq 'yes'; -my $repeatmerged = ($param{'repeatmerged'} || 'yes') eq 'yes'; -set_option('archive', $archive); -set_option('repeatmerged', $repeatmerged); +my $archive = $param{'archive'} eq 'yes'; +my $repeatmerged = $param{'repeatmerged'} eq 'yes'; my $buglog = buglog($ref); -if ($ENV{REQUEST_METHOD} eq 'HEAD' and not defined($att) and not $mbox) { +if (defined $ENV{REQUEST_METHOD} and $ENV{REQUEST_METHOD} eq 'HEAD' and not defined($att) and not $mbox) { print "Content-Type: text/html; charset=utf-8\n"; my @stat = stat $buglog; if (@stat) { @@ -79,44 +98,46 @@ sub display_entity ($$$$\$\@) { $filename = '' unless defined $filename; $filename = decode_rfc1522($filename); - if ($top) { + if ($top and not $terse) { my $header = $entity->head; - if ($trim_headers and not $terse) { + $$this .= "
\n"; + if ($trim_headers) { my @headers; - foreach (qw(From To Subject Date Cc)) { + foreach (qw(From To Cc Subject Date)) { my $head_field = $head->get($_); next unless defined $head_field and $head_field ne ''; - push @headers, qq($_: ) . htmlsanit(decode_rfc1522($head_field)); + push @headers, qq($_: ) . html_escape(decode_rfc1522($head_field)); } $$this .= join(qq(), @headers) unless $terse; - $$this .= qq(\n); - } - elsif (not $terse) { - $$this .= htmlsanit(decode_rfc1522($entity->head->stringify)); - $$this .= qq(\n); + } else { + $$this .= html_escape(decode_rfc1522($entity->head->stringify)); } + $$this .= "\n"; } unless (($top and $type =~ m[^text(?:/plain)?(?:;|$)]) or ($type =~ m[^multipart/])) { push @$attachments, $entity; - my @dlargs = ($ref, "msg=$xmessage", "att=$#$attachments"); - push @dlargs, "filename=$filename" if $filename ne ''; + my @dlargs = ($ref, msg=>$xmessage, att=>$#$attachments); + push @dlargs, (filename=>$filename) if $filename ne ''; my $printname = $filename; $printname = 'Message part ' . ($#$attachments + 1) if $filename eq ''; - $$this .= '[$printname } . - "($type, $disposition)]\n\n"; + $$this .= '
[$printname } . + "($type, $disposition)]\n"; - if ($msg and defined($att) and $att eq $#$attachments) { + if ($msg and defined($att) and $att == $#$attachments) { my $head = $entity->head; chomp(my $type = $entity->effective_type); my $body = $entity->stringify_body; - print "Content-Type: $type\n"; + print "Content-Type: $type"; + my ($charset) = $head->get('Content-Type:') =~ m/charset\s*=\s*\"?([\w-]+)\"?/i; + print qq(; charset="$charset") if defined $charset; + print "\n"; if ($filename ne '') { my $qf = $filename; $qf =~ s/"/\\"/g; $qf =~ s[.*/][]; - print qq{Content-Disposition: attachment; filename="$qf"\n}; + print qq{Content-Disposition: inline; filename="$qf"\n}; } print "\n"; my $decoder = new MIME::Decoder($head->mime_encoding); @@ -150,11 +171,23 @@ sub display_entity ($$$$\$\@) { $$this .= "\n"; } else { if (not $terse) { - my $content_type = $entity->head->get('Content-Type:'); + my $content_type = $entity->head->get('Content-Type:') || "text/html"; my ($charset) = $content_type =~ m/charset\s*=\s*\"?([\w-]+)\"?/i; my $body = $entity->bodyhandle->as_string; $body = convert_to_utf8($body,$charset) if defined $charset; - $$this .= htmlsanit($body); + $body = html_escape($body); + # Attempt to deal with format=flowed + if ($content_type =~ m/format\s*=\s*\"?flowed\"?/i) { + $body =~ s{^\ }{}mgo; + # we ignore the other things that you can do with + # flowed e-mails cause they don't really matter. + } + # Add links to URLs + $body =~ s,((ftp|http|https)://[\S~-]+?/?)((\>\;)?[)]?[']?[:.\,]?(\s|$)),$1$3,go; + # Add links to bug closures + $body =~ s[(closes:\s*(?:bug)?\#?\s?\d+(?:,?\s*(?:bug)?\#?\s?\d+)*) + ][my $temp = $1; $temp =~ s{(\d+)}{qq($1)}ge; $temp;]gxie; + $$this .= qq( \n); } } } @@ -163,30 +196,26 @@ my %maintainer = %{getmaintainers()}; my %pkgsrc = %{getpkgsrc()}; my $indexentry; -my $descriptivehead; my $showseverity; my $tpack; my $tmain; -$ENV{"TZ"} = 'UTC'; -tzset(); - -my $dtime = strftime "%a, %e %b %Y %T UTC", localtime; -$tail_html = $debbugs::gHTMLTail; +my $dtime = strftime "%a, %e %b %Y %T UTC", gmtime; +$tail_html = $gHTMLTail; $tail_html =~ s/SUBSTITUTE_DTIME/$dtime/; -my %status = %{getbugstatus($ref)}; +my %status = %{get_bug_status(bug=>$ref)}; unless (%status) { print <
There is no record of $debbugs::gBug $short. +
There is no record of $gBug $short. Try the search page instead.
$tail_html EOF @@ -200,37 +229,69 @@ my @tpacks = splitpackages($tpack); if ($status{severity} eq 'normal') { $showseverity = ''; -#} elsif (isstrongseverity($status{severity})) { -# $showseverity = "Severity: $status{severity};\n"; +} elsif (isstrongseverity($status{severity})) { + $showseverity = "Severity: $status{severity};\n"; } else { - $showseverity = "Severity: $status{severity};\n"; + $showseverity = "Severity: $status{severity};\n"; +} + +if (@{$status{found_versions}} or @{$status{fixed_versions}}) { + $indexentry.= q(}; } -$indexentry .= "$showseverity"; -$indexentry .= htmlpackagelinks($status{package}, 0) . ";\n"; -$indexentry .= htmladdresslinks("Reported by: ", \&submitterurl, +$indexentry .= "
Bug is archived. No further changes may be made.
"; -} -my @records = read_log_records($buglogfh); +my @records; +eval{ + @records = read_log_records($buglogfh); +}; +if ($@) { + quitcgi("Bad bug log for $gBug $ref. Unable to read records: $@"); +} undef $buglogfh; =head2 handle_email_message @@ -308,13 +395,11 @@ sub handle_email_message{ my $output = ''; my $parser = new MIME::Parser; + # Because we are using memory, not tempfiles, there's no need to + # clean up here like in Debbugs::MIME $parser->tmp_to_core(1); $parser->output_to_core(1); - # $parser->output_under("/tmp"); my $entity = $parser->parse_data( $email); - # TODO: make local subdir, clean it ourselves - # the following does NOT delete the msg dirs in /tmp - END { if ( $entity ) { $entity->purge; } if ( $parser ) { $parser->filer->purge; } } my @attachments = (); display_entity($entity, $options{ref}, 1, $options{msg_number}, $output, @attachments); return $output; @@ -337,9 +422,30 @@ sub handle_record{ my $output = ''; local $_ = $record->{type}; if (/html/) { + my ($time) = $record->{text} =~ //; + my $class = $record->{text} =~ /^(?:Acknowledgement|Reply|Information|Report|Notification)/ ? 'infmessage':'msgreceived'; $output .= decode_rfc1522($record->{text}); - $output .= 'Full text and rfc822 format available.'; + # Link to forwarded http:// urls in the midst of the report + # (even though these links already exist at the top) + $output =~ s,((?:ftp|http|https)://[\S~-]+?/?)([\)\'\:\.\,]?(?:\s|\.<|$)),$1$2,go; + # Add links to the cloned bugs + $output =~ s{(Bug )(\d+)( cloned as bugs? )(\d+)(?:\-(\d+)|)}{$1.bug_links($2).$3.bug_links($4,$5)}eo; + # Add links to merged bugs + $output =~ s{(?<=Merged )([\d\s]+)(?=\.)}{join(' ',map {bug_links($_)} (split /\s+/, $1))}eo; + # Add links to blocked bugs + $output =~ s{(?<=Blocking bugs)(?:( of )(\d+))?( (?:added|set to|removed):\s+)([\d\s\,]+)} + {(defined $2?$1.bug_links($2):'').$3. + join(' ',map {bug_links($_)} (split /\,?\s+/, $4))}eo; + # Add links to reassigned packages + $output =~ s{(Bug reassigned from package \`)([^']+)((?:'|\&\#39;) to \`)([^']+)((?:'|\&\#39;))} + {$1.q($2).$3.q($4).$5}eo; + if (defined $time) { + $output .= ' ('.strftime('%a, %d %b %Y %T GMT',gmtime($time)).') '; + } + $output .= 'Full text and rfc822 format available.'; + + $output = qq(\n"; } elsif (/recips/) { my ($msg_id) = $record->{text} =~ /^Message-Id:\s+<(.+)>/im; @@ -349,12 +455,12 @@ sub handle_record{ elsif (defined $msg_id) { $$seen_msg_ids{$msg_id} = 1; } - $output .= 'View this message in rfc822 format'; - $output .= ' '; + ); } elsif (/autocheck/) { # Do nothing @@ -369,13 +475,14 @@ sub handle_record{ } # Incomming Mail Message my ($received,$hostname) = $record->{text} =~ m/Received: \(at (\S+)\) by (\S+)\;/; - $output .= qq|View this report as an mbox folder.
\n", mboxurl($ref); -print "Reply ), + qq(or subscribe ), + qq(to this bug.
\n); + print qq(); + printf qq(View this report as an mbox folder, ). + qq(status mbox, maintainer mbox
Send a report that this bug log contains spam.
\n