Bug 1418378 - Use Material Icons on Bugzilla home page instead of low-res images

Bug 1421032 - Detect Markdown attachment files as text/plain so Firefox can display the content

Bug 1494645 - Allow customizing HTML <title> of search results

fix error and bump version (#772)

no bug - use more generous timeouts

this makes all the mojo timeouts larger -- except for 'clients' which should be smaller because we're so synchronous. It also puts them into environmental variables so ops can tweak them.

Note some of the code has moved to the main application class to make future people less likely to not notice these values.

no bug - use bugzilla as the default template engine

Bug 1493490 - Remove navigator.buildID usage from Bugzilla Helper

Bug 1493500 - Remove all trailing whitespaces from all files

Bug 1493847 - Fix buglist.cgi link label on product/component dropdown lists

Bug 1489718 - Insert form widgets for approval flag requests instead of free-form comment text

Bug 1494065 - Add a basic test using Test::Mojo

bump version to 20180925.1

Bug 1493295 - Add short URL to each bug

Bug 1491850 - restoreSavedBugComment takes a really long time (100ms)

Bug 1490595 - Bugzilla update check should use https

Bug 1473417 - Show often/recently used products/components on New Bug page

Bug 1492926 - Handle DBIx::Connectors more appropriately

This is a bigger change than I anticipated, because the way we cached
DBIx::Connector objects was bad.

Now we cache the Bugzilla::DB instances in connect_main() and connect_shadow().
This is for maintaining a 1:1 mapping of Bugzilla::DB objects and
DBIx::Connector objects. This is important because we want be able to inspect
Bugzilla::DB->bz_in_transactions() from the 'connected' event.

Note that we weaken the lexical variable $self in _build_connector()
because it is referenced by the callback passed to DBI.
Without this there would be a memory cycle and stuff would never be freed.

(tested my understanding in this gist:
https://gist.github.com/dylanwh/646574a027f7b7d92cb7586676da7468)

Bug 1492346 - scripts/syncflags.pl should also add the target product to any tracking flags that the source product is a member

Bug 1470536 - Add new GeckoView product to easy product selector on Browse and Enter Bug pages

Bug 1492511 - Code to updating subscriber values for current private bugs is throwing errors in the phabbugz log

Bug 1468489 - Documentation points to https://landfill.bugzilla.org that is no longer maintained and will be shut down

Bug 1492850 - Remove places where headers are printed

There's one place where some (unused?) debug code prints out headers without using Bugzilla->cgi, and testagent.cgi which does the same. The first thing is removed and testagent.cgi is also removed, with its route handled by a simple route.

no bug - bump Alien::libcmark_gfm

Bug 1455495 - Replace apache with Mojolicious

bump version to 20180918.1

Bug 1490901 - ReviewBoard stub attachments no longer make a redirect, download a text file instead

Bug 1345673 - Open Bugzilla History in a New Window or Tab

Bug 1491973 - Add GeckoView to BMO::Data special casing

no bug - cleanup a few nits in the SecurityRiskReport (#746)

- sorted imports, with Moo and MooX::StrictConstructor at the top because they
  change the behavior of the code.
- removed 'scalar' when comparing an array to an integer as it isn't required.
- adjusted multi-line first { } to single line since it still fits and perltidy
  makes it look ugly.
- store each 'result' hash in a $result variable, again to make perltidy format
  better.
- change use of 'unshift ARRAY' to 'push ARRAY' and reverse().
  The later performs fewer mallocs (push is much more effficient than unshift).
  Please check if this logic is right.

Bug 1490708 - Ensure we always call DBIx::Connector->dbh before any DBI method (#744)

The code didn't allow a way of doing this without a lot of work.

So I had to take the following approach:
The 'dbh' attribute is now a method that delegates to DBIx::Connector's dbh
method. Per the docs, ->dbh() "Returns the connection's database handle. It will
use a an existing handle if there is one, if the process has not been forked or
a new thread spawned, and if the database is pingable. Otherwise, it will
instantiate, cache, and return a new handle."

Then there is the matter of the 'handles' on dbh. I've used Package::Stash to
insert proxy methods into the class when it is loaded.

Bug 1490687 - Stop setting r+s on Phabricator attachments

Bug 1487422 - Remove Phabricator section from MyDashboard and related WebService API

bump to version 20180910.1

no bug - add X-Bugzilla-Type to secbugsreport email

Bug 1479466 - Add Security Bugs Report

Adds the security bugs report with open count and median age open of
sec-critical and sec-high bugs.

Bug 1488292 - Remove MozReview extension from BMO code tree as MozReview is being decommissioned

Bug 1486369 - please enable "due date" for Firefox -> Security: Review Request compoent

Bug 1399721 - change canonical from bugzil.la to bugzilla.mozilla.org

bump version to 20180820.1

Bug 1482145 - PhabBot changes are showing up as from the wrong user

Bug 1482145 - Some changes required for unit tests to be written

Bug 1482145 - refactor revison reviewers to be a list of revsion reviews

no bug - add basic vscode settings

Bug 1473958 - Update Thunderbird logo, replace Data Platform and Tools icon on easy product selector

Bug 1482145 - add a scope_guard option to set_user()

Bug 1482145 - add more type checking to phabbugz code

change how builds work slightly

Bug 1480878 - Monitor the health of Push connector job processing

Revert "Bug 1456878 - Support markdown comments"

This reverts commit fd850e00db835d2b84c59014c3b1021fea2294fc.

Bug 1482475 - Add extensive testing framework

Bug 602313 - follow up - Icons on new attachment uploader are broken

Bug 1456878 - Support markdown comments

Bug 602313 - Allow creation of attachments by pasting an image from clipboard, as well as by drag-and-dropping a file from desktop

bump version to 20180808.1

Bug 1481893 - After recent push of bug 1478897 bug/revision syncing has been broken due to coding error

Bug 1480891 my dashboard does not show the revision id and title for phabricator review requests

bump version to 20180807.1

Bug 1480897 - When making a revision public, make the revision editable only by the bmo-editbugs-team project (editbugs)

Bug 1474809 - add "new to bugzilla" tag to non-comment changes

Bug 1480473 - Component description page: highlighted component lacks padding

Bug 1446855 - enter_bug.cgi: Searching for duplicate bugs should trigger on changes to the Summary, not on all keystrokes

Bug 1481207 - POST /rest/bug_user_last_visit returns random number instead of bug ID

Bug 1480599 - Add "File new bug" menu to product/component hovercard

Bug 1478897 - ensure phabbugs doesn't fail outright when encountering invalid bug ids

Bug 1480169 - Consider reducing the verbosity of phabricator 'Revision Approved' bugzilla comments

Bug 1374266 - Improve the "Zarro Boogs found" message

Bug 1479350 - "Phabricator Reviews Requested of You" lists bugs which I have reviewed

bump version to 20180803.1

Bug 1480583 - User->match is not paging properly so all user results are not return if more than 100 users

no bug - add dependencies needed for monitoring

bump version to 20180801.1

Bug 1478013 - Importance, Status and Platform section labels in show bug view are linked but not clickable

Bug 1432095 - OpenGraph image not loaded

Bug 1479563 - Wrap labels in Requests dropdown list

Bug 1466737 - "use my platform" should default to x86_64 on Mac OS X

Bug 1475687 - Remove https://bugzilla.mozilla.org/form.reps.it custom form

Bug 1320977 - performance tweaks

Bug 1479523 - Disable one-click component watching for logged out users

Bug 1478983 - WebService endpoint to check if a user can enter a bug into a given product

There is some common code, which we will need again for another
WebService call, that can be refactored into their own functions.

This WebService endpoint takes a product name and user ID in the form
`check_enter_bug/<product>/<user id>`.  It returns an object with
one attribute, `result`, that is set to 1 if the product exists and
the user is allowed to enter bugs in it, or 0 otherwise.

Bug 1320977 - Add review/feedback/needinfo request counts and block statuses to /rest/user and /rest/user/suggest responses

Bug 1478540 - Update User.pm to load more than 100 users by using the paging functionality of Conduit API

Bug 1419636 - Make Google Analytics use beacon/XHR instead of img tag

Bug 1478012 Phab allows projects to have empty descriptions so Project.pm in PhabBugz should allow the same

no bug - Include Bugzilla::Error in PhabBugz's WebService.pm.

The ThrowUserError() calls in PhabBugz's WebService.pm were
causing errors as Bugzilla::Error wasn't being imported.

Bug 1477894 - get_attachment_revisions() should be returning an empty list, instead of a list of [0]

Bug 1476841 - Various code cleanups ahead of the Mojolicious patch

Bug 1475593 - Bugzilla Emails received when patches are attached in Phabricator

no bug - add unit tests around some parts of the phabricator integration

Bug 1136271 - Make user profile page visible to anyone for easier sharing

Bug 1472954 - Implement one-click component watching on bug modal and component description pages

Bug 1476288 - Replace moz_nick with (new, revised) nick and also attempt to disallow duplicate nicks

bump version to 20180717.2

Bug 1474026 - Double HTML escaping in crash signature update

bump version to 20180717.1

Bug 1260096 - BugUserLastVisit api not working and throwing error code : 32614

Bug 1472961 - Copy Summary button should copy link as well

Bug 1476052 - Bugzilla mishandles diff attachments that are UTF-8 and contain U+FFFF

Bug 1474617 - conduit-suite phabricator local development instance missing some default configuration

Bug 1473798 - Add and remove members of a phabricator project instead of setting exact list

bump version to 20180712.1