Bug 1460466 - Phab bot does not create r+ for acceptance when there are still blocking reviewers

Revert "Bug 1440086 - Refactor PhabBugz extension code to use new User.pm module for better type checking"

This reverts commit 739676cf4b122cdec12981c2bc3a79c3f54aa7e4.

Bug 1440086 - Refactor PhabBugz extension code to use new User.pm module for better type checking

Bug 1459336 - feed daemon skips setting r+ for accepted revision if the same user already has a flag set even if flag is status of ?

bump version to 20180508.1

Bug 1413328 - Use tct (tocotrienol on npm) to encrypt bugmail

no bug - untaint default values in localconfig

no bug - fix typo in meta referer directive

no bug - add tct to dev environment

Bug 1453759 - Port OrangeFactor extension to treeherder

Bug 1452984 - double-check new accounts with BMO to catch and notify of username squatting

Bug 1454647 - Mirror all BMO groups as Phabricator projects and keep them in sync

bump version to 20180426.4 (#556)

no bug - remove name label

fix minor bug in release process

Bug 1457031 - When a revision does not have an bug id, the bug is made public but we also need to remove secure-revision tag

Update CONTRIBUTING.md

Bug 1441732 - Improve missing module error in Bugzilla::Extensions and catch more compile errors in tests

no bug - fix permissions on log files and a few other things (#552)

bump version to 20180426.1 (#550)

Bug 1456529 - Support SameSite attribute on session cookies

Bug 1430367 - "preconnect" to google-analytics domain for improved performance

Bug 1373280 - Highlight private comments in new bug modal UI

Prior to this commit, the only indication that a comment was private
was the small "Private" tag in the comment header. This commit restores
the red font color to private comments and adds a small dashed border
for those that have trouble seeing the color red.

Bug 1455772 - Label bug bounty form credit fields

Bug 1427395 - Allow request_cache to be constant-folded in Bugzilla.pm

Bug 1455493 - cleanup push connector  logging

Bug 1453124 - extensions/PhabBugz/bin/update_project_members.pl should be combined with the normal feed daemon

Bug 1452241 - Improve feed error handling and logging

Bug 1440828 - Phabricator review requests should show up on the BMO dashboard

 Code Pull requests 33 Insights Settings Bug 1452531 - PhabBugz code should add allow visibility to reviewers when creating custom policies

Bug 1438205 - Preserve comments in progress across page reloads

Comments being typed will be saved in Local Storage for a week.
When the user somehow reloads the show bug page, the comment box
will be filled in with what was last saved.

This also fixes the bug where you type up a comment, go to change
the bug details, hit cancel, and ultimately lose your comment in
progress because of the reloading nature of the cancel button.

Bug 1451599 - Move guidelines checkbox on signup page to the left

Bug 1450325 - add unsubscribe link to html bug email template

Bump version to 20180412.2 (#538)

Bug 1453697 - ensure error_message is escaped in opengraph description

Bug 1453681 - Phabricator project.search when searching for a specific project name can return more than one match

Bug 1451573 - Remove Firefox Marketplace custom forms

bump version to 20180412.1

Bug 1451960 - Improve opengraph metadata for access-denied bugs

no bug - fix PerlSetEnv

Bug 1453122 - The phabbugz feed daemon should use IO::Async::Timer instead of using while loop

no bug - remove stack traces from badhosts error messages (#528)

Bug 1421110 - Add REST API endpoint for /bug/possible_duplicates

Bug 1453126 - Bugzilla::Bloomfilter should encourage preserving the input files for its filters

Bug 1451416 - Bugzilla sometimes sends emails to accounts when it shouldn't

Bug 1450990 - Refactor a bunch of the logging config files

Bug 1328900 - Create new group called 'disableusers' that can only edit the bugmail and disabledtext fields of a user

Bug 1450679 - Replace custom Sentry integration with Logging

Bug 1449282 - add jobqueue_status api

Bug 1450791 - SES handler needs to support both "event" and "notification" messages

no bug - add mailmap file to fix historical stats

Bug 1446431 - Allow Baseline scan to ignore forms that dont need CSRF Tokens

The data-no-csrf attribute is used to signify that a form is 'safe' (ie
doesn't actually make any permanent changes) and so doesn't need an
anti-csrf token.

Bug 1447028 - Add auth delegation test script

no bug - add dev build pipeline to circleci

no bug - circleci config tweaks

Bug 1450920 - Don't pass blank API token to JSON-PRC in instant search

When an API token is seen by JSON-RPC, it will attempt to authenticate
it, including blank tokens (empty strings). Thus, avoid passing an
empty string in the first place.

The pattern to pass an empty string if the absence of
BUGZILLA.api_token is the most common way to include the
Bugzilla_api_token in the JSON-PRC requests. However, most places
which pass a token to JSON-RPC in JavaScript are in contexts where a
user is expected to be logged in, and this is not the case for instant
search.

Although this could have been fixed by patching
Bugzilla::Auth::Login::Cookie::get_login_info to treat empty API
tokens as if none were given, this method was chosen, as this is also
the approach used in the ProdCompSearch extension (where a login
session also does not need to be required), and to avoid possible
breakage in JSON-RPC consumers.

Bug 1450283 - JobQueue should treat "no jobs" as a trace-level message, and all other logs as info

bump version to 20180330.1

Bug 1450343 - Make the SES handler use Bugzilla::Logging and log more details

bump base image to bmo-slim:20180330.1

Bug 1441244 - prevent compounding error messages in tests

Bug 1447027 - Change default mail method for vagrant and update README

Updates the default mail_delivery_method to be Sendmail for vagrant
development. This allows developers to connect to the imap server
running at bmo-web.vm:143 to view mail using a real mail client.

The default method remains the same for docker users, 'Test'.

Both methods are described in the README in a new section.

Bug 1441897 - Improve opengraph metadata for bug pages

Bug 1450010 - The jobqueue supervisor's pidfile should not be stored in the data directory

Bug 1450008 - documentation link in API errors is wrong

Bug 1200695 - API-key-creation emails should reflect if the action was a result of auth delegation

Bug 1441063: Fix the unaccepted revision comment (PhabBugz)

* PhabBugz: Fix the unaccepted revision comment

Summary:
Currently, we're sending the "User removed from revision" comment when
the Accept flag has been removed from a revision. This could happen in a
number of use cases:
1. The reviewer resigned from being a reviewer.
2. The reviewer has been removed from reviewers list.
3. Someone (author, reviewer) removed acceptance of the revision by changing
   its status to "Needs Review".

This patch is sending a "flag is deactivated" or "reviewer removed from
revision" depending on the current status of the reviewer.

Test Plan:
Accept a revision.
Change the revision to "Needs Review".
Check the bug comment.
Accept the revision.
Remove the user from reviewers list.
Check the bug comment.

Reviewers: dkl

Bug #: 1441063

Differential Revision: https://phabricator.services.mozilla.com/D809

* Styling fixed as requested in review.

* Perl style fixes

bump version to 20180328.1

no bug - always build latest tag

no bug - skip tests if the only thing changed is the version

no bug - existing tag is non-fatal

Bug 1449168 - Remove warning --function from jobqueue worker

Bug 1449156 - Bugzilla::Memcached should use smaller timeouts and ping servers at instantiation time

Bug 1449413 - Refactor circleci container building stuff

Bug 1440829 - Bugzilla comment for Phabricator commit should include entire commit message, not just first line

bump version to 20180327.1

Bug 1448681 - Bugmail Message-ID header format changed without changing In-Reply-To/References, breaking threading

bump version to 20180326.1

no bug - run jobqueue faster

no bug - fix jobqueue command to run with multiplexed json output

no bug - TheSchwartz debug log is really more like info level in how we have used it

no bug - add interdiff

bump version to 20180324.1

no bug - enable HTTPS the proper way

bump version to 20180323.1

no bug - increase httpd limits

Bug 1447410 - Make it so you can always request review from .bugs or .tld accounts

Bug 1446974 - Revert changes of the subscribers list on secure revisions

bump version to 20180321.1

Bug 1447669 - follow-up fix - use override shadowsdb value to determine to use it or not

Bug 1399713 - ensure existing production redirects work in a cloud hosted environment

Bug 1447669 - Add localconfig parameter for changing shadowdb user and pass

Bug 1444008 - Fix sanity tests for unfiltered urlbase

bump version to 20180320.3

no bug - remove debugging for now

bump version to 20180320.2

Bug 1444008 - Form action injection in Bugzilla /user_profile (leads to XSS/single-factor credential leakage)

bump version to 20180320.1

Bug 1447289 - heartbeat check should not check for enabled features

Bug 1447291 - Remove Apache2::Log from PhabBugs/Push in favor of logging framework