update for correspondent in maintainer

the maintainer address should be a correspondent

document verbose and progress; add back quiet tag

only remove the progress bar if $p is defined

use Term::Progress bar; output information on failure

handle tags, and new submitter/correspondent setup

we now require libdbix-class-perl and libdatetime-format-pg-perl

try very hard to output a reported date for bugs even when summary is missing information

update result classes for new relationships

get rid of obsolete results

remove bug_severity
add bug_status cache
change how bug submitters, done and owner are tracked

Switch to severity table
Add more links into the correspondent table so we can show all a persons bugs together
stop using _id
Add correspondent full name

add submitter field to bug table

abstract out some of the bug loading code into Debbugs::DB::Load

alter binary version view to include based on versions

cache arch because there aren't very many of them

cache src_pkgs for a single file as it's highly likely to be the same.

fix handle_queue typo

document that the spool directory can be set

add .gitignore file to ignore the table comments

delete mispelled corespondent packages

update db::result class and add correspondent and message correspondent

update db::result class

fix timezone and correspondent typos

add the rest of the sql documentation

add more documentation of the debbugs schema

add bug message bits to sql

add message tables

add initial version of debbugs-loadsql-debinfo and debbugs-loadsql-versions

Add bin and source associations
Add suites and maintainer tables
Update srcver with maintainer

switch to debversion

add initial work on sql for debbugs

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