no bug - Update README on VS Code integration

Update the description as Tasks menu has been merged to Terminal menu.
Also mention an extension bundle made by @dylanwh.

Bug 1497234 - Remove Personas Plus GitHub link from Custom Bug Entry Forms index

Bug 1496803 - Suggested component links ignore cloned bug data

Bug 1496832 - Add monitoring for feed daemon becoming unresponsive

Use Datadog to count the number of times that jobs are run by the
phabbugz job scheduler.  This should catch problems if the process
responsible for running the jobs dies.

bump version to 20181005.1

no bug - Don't know => Unknown

Bug 1496172 - Attachments with unicode filenames cause timeout errors when retreiving attachments

Bug 1496570 - Bugzilla doesn't attempt to decode  form-urlencoded data without a content-type header

bump version to 20181004.1

Bug 1495746 - /app/static/metricsgraphics/css/google-OpenSans.css missing in app container

Bug 1495901 - SES type checking error

Bug 1489120 - Add a rest API to get triage owners for each product/component pair (#797)

no bug - document the mojo-specific environmental variables

Bug 1496233 - "Dunno" -> "Don't know" in approval request form (#795)

no bug - make libcmark-gfm optional (#792)

no bug - add chart-clicker dependency

no bug - reformat all new quantum files using new standard

Bug 1495906 - After mojo update /latest/configuration API call no longer works and gives page not found

Bug 1495834 - xmlrpc.cgi outputs headers in request-body

Bug 1495869 - Crash graph not found after mojo migration

Bug 1263502 - Add duplicates to /rest/bug/id

add helpers for handling logins and error handling

no bug - adopt mojolicious code formatting guidelines (#784)

@kyoshino likes 2-char indent for all the frontend.
Mojolicious seems to use 2-space too.
Let's just adopt their perltidyrc.

Included in this is a script (in scripts/) that will modify files according to these rules.

Bug 1495349 - Remove Persona extension

bump version to 20181002.1

Bug 1495071 - Mojolicious Cleanup

There are some things that should've been in the first patch but were missed:

1. Calling $c->finish in the finally block should not happen if an exception has
   been raised.
2. Bugzilla->cleanup() should be called at the same time the mojolicious stash
   is cleared.
3. Code referencing the shutdownhtml should be removed
4. The conditionals that ran code in Bugzilla.pm when it was not run under
   mod_perl should instead check where the Bugzilla.pm module was loaded from.
5. Revert the default template from #770
6. Also removed some stuff that manipulates the PATH and signals, which we
   shouldn't do

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