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

switch 17_version_cgi.t to use DebbugsTest::HTTPServer

add etag and caching support to bugreport.cgi

 - Calculate etag based on file modification times and all parameters
 - Set etag everywhere
 - Use reasonable cache retention times
   - 5m minimum
   - 10m for head requests and bug logs
   - 7d for bug attachments (which should never change, really)

abstract out no_such_bug

move CGI parameter definitions earlier

Make usertag_file_from_email a public function

add test for version.cgi cache control

set etags and cache-control headers

This should make the version.cgi significantly faster
 - the html now should be alive for one day
 - the images should live for 300 seconds
 - etags should be calculated quickly, and in general before very many
   things are parsed

add calculate_etags and etags_do_not_match methods

Make munge_url use repeatable sorting

document fixes that Frank Lichtenheld contributed

fix inversion of date/versions in Debbugs::Status leading to negative times argument

Control: Add missing full stop at the end of "Changed" messages

This leads to broken links for at least the forwarded-to case.
(Closes: #782740)

Signed-off-by: Don Armstrong <don@donarmstrong.com>

Bugreport: Fix problems with reassign message

* Matched hardcoded "Bug" instead of $config{bug} (which leads
  to problems at least in the test suite)
* Use package_links() wrong. package_links() already adds HTML.
  (Closes: #767327)

Signed-off-by: Don Armstrong <don@donarmstrong.com>

Extend bugreport test cases to check the output of some control messages

This exposes #767327 (wrong package links in "reassigned" message)
as a test failure.

Signed-off-by: Don Armstrong <don@donarmstrong.com>

cloning had the blocking reversed

Bugs which were cloned had the block/blockedby sense reversed. This has
now been cleaned up.

Remove an obsolete comment

handle receips state properly; check for unescaped output in mbox

Fix mbox output of bugreports to properly use filehandle
Write test to actually test mbox output

Closes: #800497

import :read in bugreport.cgi

Add IO::InnerFile support to Debbugs::Log and use it

Use the inner_file handle in cgi/bugreport and Debbugs::CGI::Bugreport
to avoid having the entire file in memory all of the time

Eventually we will want to use an index to avoid having to read every
single line of the bug log for every single message. Some day. [If
you're reading this, you could make that day sooner!]

change the scope of tempdir so that it stays in scope until after display_entity is done

also use a filesystem in handle_email_message to avoid running out of memory

use tmpdir for bugs instead of memory to avoid trouble with large bugs

document fix for #790716 in changelog

Plural/singular agreement in error message

Make the error text message correctly respect the singular/plural
agreement with the number of errors.  We have now "with 1 error"
instead of "with 1 errors".

Also, change the 07_control_limit unit test accordingly.

point at config.debian's new location

remove debbugging effective_type output

use Sys::Hostname::hostname instead of hostname --fqdn

Fix SOAP module hack to work around Date/Time encoding which broke in newer versions of SOAP::Lite. (Closes: #785405)

we know that the statement is ok; use $1 to avoid taint issues

next release will be 2.6.0; document more issues which have been fixed

perl 5.14 does not support named capture groups, apparently

it's tags, not tag

allow arbitrarily complex boolean operations in usercategories selection statements

add test for get_bug_order_index; fix faulty regex in while()

add missing /g

support or matching in fields in usercategories