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

Minor edits for clarity

Fixed MORE code block formatting

Fixed MORE code block formatting

Fixed code block formatting

Updated the installation instructions

Adjusted README.md with minor structural changes and edits

Merge remote-tracking branch 'don/master' into pindy/docs

move scripts/config to examples/

depend on apache2 or httpd

switch to using Require all granted from old Options allow,deny

use + for symlinks and multiviews; newer apache doesn't like it otherwise

we don't need to expose the txt directory either

remove irrelevant configuration options which have a sane default from the configuration file

we shouldn't expose db

use /var/log/apache2/ as the default error log location

allow &quot; in other places too (HTML::Escape doesn't always use &#39; anymore)

Stop URLs at &quot; too. Thanks to Vincent Lefevre (Closes: #766354).

Super basic intro

ignore addresses which do not have @ to work around people who use improperly formatted From: address lines

use uri_escape_utf8 instead of uri_escape

Add affects to bugreport.cgi package info header (Closes: #609687).

Merge remote-tracking branch 'pindy/pindy/docs'

Replaced dead website link with current pages

Converted to markdown

Added contributing section

fix the hidden parameters not being included in the pkgindex template

fix missing semicolon; check that HTTP_HOST is defined

unfinalize; document XSS fix

fix XSS in version.cgi

cleanup_eval_fail should match all croak messages; was missing /m

remove leading and trailing spaces in references and replyto

fix multipart messages when quoting the body; bodyhandle does not necessarily exist in those cases; fixes #756427

Add reply links to individual bug messages with headers (closes: #412815). Thanks to martin krafft, Paul Wise, and Julien Valroff.

eliminate convert to utf8 warning

handle undefined headers by just returning ''

fix the links to reassigned packages in the html entries

fix  #747267 by causing the closing regex to require a space, or some other non-word character