croak in subroutines

skip maintainer files if they are empty *or* undef

use https for CVE tracker by default

fix usertag_package_domain to strip protocol if its present

fix the spam lockfile name

handle SIGPIPE in spamc by skipping the message

fix debbugs source location

add missing package_tracking_domain configuration variable

21bugclosers does not need common.pl

Use List::AllUtils instead of List::Utils and List::MoreUtils

Because some commands have migrated from List::MoreUtils and List::Utils at
different times, we would have to run slightly different code in unstable,
stable, and oldstable. List::AllUtils wraps around this madness so we don't have
to think about it.

older versions of List::Util (<1.26) do not have sum0; just use sum

have the travis build script run the tests

fix failure to import locking funcs and don't save empty spamlogs

remove useless return in record_regex

add debbugs-spamscan-log to run spamc on log files

include links to anchors of html messages too

Debbugs::CGI needs to load usertags

if there is not a msgid, it cannot be spam

add missing =back to Debbugs::Log::Spam

forcibly wrap msgreceived

fix links to merged and blocked bugs

Add new Debbugs::Log::Spam module and use it to skip log spam

Debbugs::Log now handles compressed bug logs

set indent variables

handle compressed logs in Debbugs::Log

use feature state in Debbugs::Log for some small speed gain

use UTF-8 encoding for 22oldbugs output

try to forcibly wrap messages which have not been properly wrapped

support the raw category again

properly handle orderings in pkgreport.cgi

convert to perl's internal utf8 in reply_headers

This fixes mojibake in the reply-to links

add envelope_from option so a valid user can be set

Provide a link to the current source code location for the BTS.

support singular tag/usertag in psuedoheaders

Thanks to James Clarke for the suggested patch

Reply Sent messages are not informational; they change state.

pass terse to handle_record

Include the protocol (http://) in gWebDomain and gCGIDomain

We now include the protocol so that we can easily specify all https links
instead of http links.

add cc all mails to addr config

This address gets all messages; this probably won't get used, but I wrote the
code.

pkgreport does not require common.pl; remove cd

remove common.pl and old cgi code which is unused

fix flie/file typo which broke usertags in bug reports

Fix breakage caused by cdn.libravatar.org not setting Content-Type

 - Actually test using libmagic and adjust type on that basis

remove UTF8 warnings

change fix to #858671 to split source with split_status_fields

Some consumers of get_bug_status via SOAP were not handling split packages

remove two checks where dpkg now follows policy

 foo- is not a legal version number

prefix src: to all source packages

Split source file properly

Thanks to James McCoy for the start of the patch

include experimental in the default removal tags

properly check close and closedir

 - Thanks to Ian Jackson for noticing that we weren't properly checking
   whether the decompression program properly exited by checking the
   return value of close(). See
   22635.45523.288493.760480@chiark.greenend.org.uk for discussion

the versions database is rebuilt each time

 - Because we allow more than one version, we need to rebuild the
   versions database each time

fix xzip/bzip2 regex

support bzip2 (oldstable)

build-versions-db now updates in-place

 - remove all of the mv bits

add update and --no-update options

 + Abstract out main routine
 + move versions time cleanup here
 + Hopefully this will avoid the issues of failures screwing up the
   database

IO::Uncompress is too slow; use open_compressed_file

debbugs-web now breaks/replaces debbugs << 2.4.2

use dh $@ and debehlper version 9

Fix clone removing all other blocks instead of add

  + (Closes: #820044). Thanks to James Clarke.

add back set -e in case of failure

remove compprefix

build-versions-db doesn't need hostname

Handle Packages.xz etc.

Handle the removal of packages.gz while building the versions database

send out the right charset on all requests

document fix of #584922 by Niels

D/Config: Add missing </P> in default html_tail

Signed-off-by: Niels Thykier <niels@thykier.net>

Drop more unused variables

Signed-off-by: Niels Thykier <niels@thykier.net>

Libravatar: Add caching of "negative" libravatar results

Signed-off-by: Niels Thykier <niels@thykier.net>

Set LC_ALL to an UTF-8 locale before running tests

To avoid noisy warnings from some tests that implicitly expect to be
run under an UTF-8 locale.

Signed-off-by: Niels Thykier <niels@thykier.net>

quitcgi: Return 400/500 status codes

Signed-off-by: Niels Thykier <niels@thykier.net>

add notification settings to travis.yml

Fix (and test) setting summary/outlook in Control: messages (Closes: #836613).

create_config now calls BAIL_OUT on its own

abstract out create config in tests

make already runs test rules

depend on libmagic, gravatar-url and www-perl

try a different approach to using system perl

system is in quotes

only use the system perl install

add PREREQ_PM for makefile

add travis.yml

stop double printing in run_rsync

update testcover rule

remove double colon in Debbugs::CGI

Use ETags; return timestamp not is_valid

 - Add test for libravatar.cgi
 - retrieve_libravatar and cache_location now return a timestamp, not
   is_valid which can then be used to calculate the Etag
 - Use HTTP::Server::Simple::CGI::Environment

Drop unused my $vars

Signed-off-by: Niels Thykier <niels@thykier.net>

Libravatar: Refactor cache handling to avoid unncessary stat

The basic pattern of the code we are dealing with is:

  my $path = cache_location(...);
  if (valid_cache($path)) {
    ...;
  }

With cache_location possibly doing a stat() of $path and valid_cache
doing at least one stat() as well.  Rewrite it so that cache_location
always stats and checks the cache (since we always need it).  This way
we can now guarantee this flow requires at *most* 3 stats rather than
5.

The replacement for the above code becomes:

  my ($path, $is_valid) = cache_location(...);
  if ($is_valid) {
     ...;
  }

Signed-off-by: Niels Thykier <niels@thykier.net>

libravatar.cgi: Drop unused import/module loads

Signed-off-by: Niels Thykier <niels@thykier.net>

Libravatar: Lazy-load LWP::UserAgent

LWP::UserAgent costs about 0.05s to load, but we only need it when the
image is not already cached locally.  Given we cache results for an
hour, it is fairly likely to be cached on subsequent requests.

Signed-off-by: Niels Thykier <niels@thykier.net>

Prefer "use Exporter qw(import)" to inheriting from it

Signed-off-by: Niels Thykier <niels@thykier.net>

d/control: Add missing Build-Depends for test

Signed-off-by: Niels Thykier <niels@thykier.net>

Standardize on using <br> (as opposed to <br/>)

Previously, the code used a bit of both.  Since the HTML pages are
HTML, standardize on "<br>".

Signed-off-by: Niels Thykier <niels@thykier.net>

html_tail.tmpl: Add missing </p>

Signed-off-by: Niels Thykier <niels@thykier.net>

ignore Extra-Source-Only: yes packages and make sure the last entry gets inserted

Add viewport in HTML

Signed-off-by: Niels Thykier <niels@thykier.net>

it should be decode_utf8, not encode_utf8

switch mailbox mime type to application/mbox

Call encode_utf8 in generate_package_info

- This change fixes the issue where package maintainers names were
  double encoded

fix captialization of cache control

set cache-control for pkgreport to 300 seconds

test for Etag support in bugreport