bugzilla.py
1125 lines
| 40.6 KiB
| text/x-python
|
PythonLexer
/ hgext / bugzilla.py
Vadim Gelfer
|
r2192 | # bugzilla.py - bugzilla integration for mercurial | ||
# | ||||
# Copyright 2006 Vadim Gelfer <vadim.gelfer@gmail.com> | ||||
Jim Hague
|
r21542 | # Copyright 2011-4 Jim Hague <jim.hague@acm.org> | ||
Vadim Gelfer
|
r2192 | # | ||
Martin Geisler
|
r8225 | # This software may be used and distributed according to the terms of the | ||
Matt Mackall
|
r10263 | # GNU General Public License version 2 or any later version. | ||
Jim Hague
|
r7504 | |||
Dirkjan Ochtman
|
r8935 | '''hooks for integrating with the Bugzilla bug tracker | ||
Jim Hague
|
r7504 | |||
Martin Geisler
|
r9252 | This hook extension adds comments on bugs in Bugzilla when changesets | ||
Jim Hague
|
r13801 | that refer to bugs by Bugzilla ID are seen. The comment is formatted using | ||
the Mercurial template mechanism. | ||||
Jim Hague
|
r7504 | |||
Jim Hague
|
r16222 | The bug references can optionally include an update for Bugzilla of the | ||
Jim Hague
|
r16223 | hours spent working on the bug. Bugs can also be marked fixed. | ||
Jim Hague
|
r7504 | |||
John Mulligan
|
r30923 | Four basic modes of access to Bugzilla are provided: | ||
1. Access via the Bugzilla REST-API. Requires bugzilla 5.0 or later. | ||||
Jim Hague
|
r7504 | |||
John Mulligan
|
r30923 | 2. Access via the Bugzilla XMLRPC interface. Requires Bugzilla 3.4 or later. | ||
Jim Hague
|
r13802 | |||
John Mulligan
|
r30923 | 3. Check data via the Bugzilla XMLRPC interface and submit bug change | ||
Jim Hague
|
r13802 | via email to Bugzilla email interface. Requires Bugzilla 3.4 or later. | ||
Martin Geisler
|
r9203 | |||
John Mulligan
|
r30923 | 4. Writing directly to the Bugzilla database. Only Bugzilla installations | ||
Jim Hague
|
r13801 | using MySQL are supported. Requires Python MySQLdb. | ||
Martin Geisler
|
r9203 | |||
Jim Hague
|
r13801 | Writing directly to the database is susceptible to schema changes, and | ||
relies on a Bugzilla contrib script to send out bug change | ||||
notification emails. This script runs as the user running Mercurial, | ||||
must be run on the host with the Bugzilla install, and requires | ||||
permission to read Bugzilla configuration details and the necessary | ||||
MySQL user and password to have full access rights to the Bugzilla | ||||
database. For these reasons this access mode is now considered | ||||
deprecated, and will not be updated for new Bugzilla versions going | ||||
Jim Hague
|
r16222 | forward. Only adding comments is supported in this access mode. | ||
Martin Geisler
|
r9203 | |||
Jim Hague
|
r13801 | Access via XMLRPC needs a Bugzilla username and password to be specified | ||
in the configuration. Comments are added under that username. Since the | ||||
configuration must be readable by all Mercurial users, it is recommended | ||||
that the rights of that user are restricted in Bugzilla to the minimum | ||||
Jim Hague
|
r16223 | necessary to add comments. Marking bugs fixed requires Bugzilla 4.0 and later. | ||
Jim Hague
|
r13801 | |||
Jim Hague
|
r13871 | Access via XMLRPC/email uses XMLRPC to query Bugzilla, but sends | ||
Jim Hague
|
r13802 | email to the Bugzilla email interface to submit comments to bugs. | ||
The From: address in the email is set to the email address of the Mercurial | ||||
user, so the comment appears to come from the Mercurial user. In the event | ||||
timeless@mozdev.org
|
r17534 | that the Mercurial user email is not recognized by Bugzilla as a Bugzilla | ||
Jim Hague
|
r13871 | user, the email associated with the Bugzilla username used to log into | ||
Jim Hague
|
r16223 | Bugzilla is used instead as the source of the comment. Marking bugs fixed | ||
works on all supported Bugzilla versions. | ||||
Jim Hague
|
r13802 | |||
John Mulligan
|
r30923 | Access via the REST-API needs either a Bugzilla username and password | ||
or an apikey specified in the configuration. Comments are made under | ||||
Wagner Bruna
|
r32602 | the given username or the user associated with the apikey in Bugzilla. | ||
John Mulligan
|
r30923 | |||
Jim Hague
|
r13802 | Configuration items common to all access modes: | ||
Martin Geisler
|
r9203 | |||
Martin Geisler
|
r13833 | bugzilla.version | ||
Bryan O'Sullivan
|
r17537 | The access type to use. Values recognized are: | ||
Jim Hague
|
r13871 | |||
John Mulligan
|
r30923 | :``restapi``: Bugzilla REST-API, Bugzilla 5.0 and later. | ||
Martin Geisler
|
r13883 | :``xmlrpc``: Bugzilla XMLRPC interface. | ||
:``xmlrpc+email``: Bugzilla XMLRPC and email interfaces. | ||||
:``3.0``: MySQL access, Bugzilla 3.0 and later. | ||||
:``2.18``: MySQL access, Bugzilla 2.18 and up to but not | ||||
including 3.0. | ||||
:``2.16``: MySQL access, Bugzilla 2.16 and up to but not | ||||
including 2.18. | ||||
Martin Geisler
|
r7985 | |||
Martin Geisler
|
r13833 | bugzilla.regexp | ||
Jim Hague
|
r16223 | Regular expression to match bug IDs for update in changeset commit message. | ||
Jim Hague
|
r16222 | It must contain one "()" named group ``<ids>`` containing the bug | ||
IDs separated by non-digit characters. It may also contain | ||||
a named group ``<hours>`` with a floating-point number giving the | ||||
hours worked on the bug. If no named groups are present, the first | ||||
"()" group is assumed to contain the bug IDs, and work time is not | ||||
updated. The default expression matches ``Bug 1234``, ``Bug no. 1234``, | ||||
``Bug number 1234``, ``Bugs 1234,5678``, ``Bug 1234 and 5678`` and | ||||
variations thereof, followed by an hours number prefixed by ``h`` or | ||||
``hours``, e.g. ``hours 1.5``. Matching is case insensitive. | ||||
Martin Geisler
|
r9203 | |||
Jim Hague
|
r16223 | bugzilla.fixregexp | ||
Regular expression to match bug IDs for marking fixed in changeset | ||||
commit message. This must contain a "()" named group ``<ids>` containing | ||||
the bug IDs separated by non-digit characters. It may also contain | ||||
a named group ``<hours>`` with a floating-point number giving the | ||||
hours worked on the bug. If no named groups are present, the first | ||||
"()" group is assumed to contain the bug IDs, and work time is not | ||||
updated. The default expression matches ``Fixes 1234``, ``Fixes bug 1234``, | ||||
``Fixes bugs 1234,5678``, ``Fixes 1234 and 5678`` and | ||||
variations thereof, followed by an hours number prefixed by ``h`` or | ||||
``hours``, e.g. ``hours 1.5``. Matching is case insensitive. | ||||
bugzilla.fixstatus | ||||
The status to set a bug to when marking fixed. Default ``RESOLVED``. | ||||
bugzilla.fixresolution | ||||
The resolution to set a bug to when marking fixed. Default ``FIXED``. | ||||
Martin Geisler
|
r9203 | |||
Martin Geisler
|
r13833 | bugzilla.style | ||
Martin Geisler
|
r9203 | The style file to use when formatting comments. | ||
Martin Geisler
|
r13833 | bugzilla.template | ||
Martin Geisler
|
r9252 | Template to use when formatting comments. Overrides style if | ||
specified. In addition to the usual Mercurial keywords, the | ||||
Martin Geisler
|
r13884 | extension specifies: | ||
Martin Geisler
|
r9203 | |||
Martin Geisler
|
r13884 | :``{bug}``: The Bugzilla bug ID. | ||
:``{root}``: The full pathname of the Mercurial repository. | ||||
:``{webroot}``: Stripped pathname of the Mercurial repository. | ||||
:``{hgweb}``: Base URL for browsing Mercurial repositories. | ||||
Jim Hague
|
r7504 | |||
Martin Geisler
|
r13842 | Default ``changeset {node|short} in repo {root} refers to bug | ||
{bug}.\\ndetails:\\n\\t{desc|tabindent}`` | ||||
Martin Geisler
|
r9203 | |||
Martin Geisler
|
r13833 | bugzilla.strip | ||
Martin Geisler
|
r13841 | The number of path separator characters to strip from the front of | ||
the Mercurial repository path (``{root}`` in templates) to produce | ||||
``{webroot}``. For example, a repository with ``{root}`` | ||||
``/var/local/my-project`` with a strip of 2 gives a value for | ||||
``{webroot}`` of ``my-project``. Default 0. | ||||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | web.baseurl | ||
Jim Hague
|
r13801 | Base URL for browsing Mercurial repositories. Referenced from | ||
Jim Hague
|
r13896 | templates as ``{hgweb}``. | ||
Jim Hague
|
r13801 | |||
Jim Hague
|
r13802 | Configuration items common to XMLRPC+email and MySQL access modes: | ||
Martin Geisler
|
r13833 | bugzilla.usermap | ||
Jim Hague
|
r13802 | Path of file containing Mercurial committer email to Bugzilla user email | ||
mappings. If specified, the file should contain one mapping per | ||||
Martin Geisler
|
r13835 | line:: | ||
committer = Bugzilla user | ||||
Jim Hague
|
r13896 | See also the ``[usermap]`` section. | ||
Jim Hague
|
r13802 | |||
Martin Geisler
|
r13836 | The ``[usermap]`` section is used to specify mappings of Mercurial | ||
Martin Geisler
|
r13834 | committer email to Bugzilla user email. See also ``bugzilla.usermap``. | ||
Martin Geisler
|
r13835 | Contains entries of the form ``committer = Bugzilla user``. | ||
Jim Hague
|
r13802 | |||
John Mulligan
|
r30923 | XMLRPC and REST-API access mode configuration: | ||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | bugzilla.bzurl | ||
Jim Hague
|
r13801 | The base URL for the Bugzilla installation. | ||
Martin Geisler
|
r13841 | Default ``http://localhost/bugzilla``. | ||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | bugzilla.user | ||
Martin Geisler
|
r13841 | The username to use to log into Bugzilla via XMLRPC. Default | ||
``bugs``. | ||||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | bugzilla.password | ||
Jim Hague
|
r13801 | The password for Bugzilla login. | ||
John Mulligan
|
r30923 | REST-API access mode uses the options listed above as well as: | ||
bugzilla.apikey | ||||
An apikey generated on the Bugzilla instance for api access. | ||||
Using an apikey removes the need to store the user and password | ||||
options. | ||||
Jim Hague
|
r13802 | XMLRPC+email access mode uses the XMLRPC access mode configuration items, | ||
and also: | ||||
Martin Geisler
|
r13833 | bugzilla.bzemail | ||
Jim Hague
|
r13802 | The Bugzilla email address. | ||
In addition, the Mercurial email settings must be configured. See the | ||||
Martin Geisler
|
r13837 | documentation in hgrc(5), sections ``[email]`` and ``[smtp]``. | ||
Jim Hague
|
r13802 | |||
Jim Hague
|
r13801 | MySQL access mode configuration: | ||
Martin Geisler
|
r13833 | bugzilla.host | ||
Jim Hague
|
r13801 | Hostname of the MySQL server holding the Bugzilla database. | ||
Martin Geisler
|
r13841 | Default ``localhost``. | ||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | bugzilla.db | ||
Martin Geisler
|
r13841 | Name of the Bugzilla database in MySQL. Default ``bugs``. | ||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | bugzilla.user | ||
Martin Geisler
|
r13841 | Username to use to access MySQL server. Default ``bugs``. | ||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | bugzilla.password | ||
Jim Hague
|
r13801 | Password to use to access MySQL server. | ||
Martin Geisler
|
r13833 | bugzilla.timeout | ||
Jim Hague
|
r13801 | Database connection timeout (seconds). Default 5. | ||
Martin Geisler
|
r13833 | bugzilla.bzuser | ||
Jim Hague
|
r13801 | Fallback Bugzilla user name to record comments with, if changeset | ||
committer cannot be found as a Bugzilla user. | ||||
Martin Geisler
|
r13833 | bugzilla.bzdir | ||
Jim Hague
|
r13801 | Bugzilla install directory. Used by default notify. Default | ||
Martin Geisler
|
r13841 | ``/var/www/html/bugzilla``. | ||
Jim Hague
|
r13801 | |||
Martin Geisler
|
r13833 | bugzilla.notify | ||
Jim Hague
|
r13801 | The command to run to get Bugzilla to send bug change notification | ||
Martin Geisler
|
r13841 | emails. Substitutes from a map with 3 keys, ``bzdir``, ``id`` (bug | ||
id) and ``user`` (committer bugzilla email). Default depends on | ||||
version; from 2.18 it is "cd %(bzdir)s && perl -T | ||||
contrib/sendbugmail.pl %(id)s %(user)s". | ||||
Jim Hague
|
r7504 | |||
Martin Geisler
|
r9203 | Activating the extension:: | ||
Jim Hague
|
r7504 | |||
[extensions] | ||||
Martin Geisler
|
r10112 | bugzilla = | ||
Jim Hague
|
r7504 | |||
[hooks] | ||||
# run bugzilla hook on every change pulled or pushed in here | ||||
incoming.bugzilla = python:hgext.bugzilla.hook | ||||
Jim Hague
|
r13801 | Example configurations: | ||
XMLRPC example configuration. This uses the Bugzilla at | ||||
Martin Geisler
|
r13841 | ``http://my-project.org/bugzilla``, logging in as user | ||
``bugmail@my-project.org`` with password ``plugh``. It is used with a | ||||
Jim Hague
|
r13870 | collection of Mercurial repositories in ``/var/local/hg/repos/``, | ||
with a web interface at ``http://my-project.org/hg``. :: | ||||
Jim Hague
|
r7504 | |||
Jim Hague
|
r13801 | [bugzilla] | ||
bzurl=http://my-project.org/bugzilla | ||||
user=bugmail@my-project.org | ||||
password=plugh | ||||
version=xmlrpc | ||||
Jim Hague
|
r13870 | template=Changeset {node|short} in {root|basename}. | ||
{hgweb}/{webroot}/rev/{node|short}\\n | ||||
{desc}\\n | ||||
strip=5 | ||||
Jim Hague
|
r13801 | |||
[web] | ||||
baseurl=http://my-project.org/hg | ||||
Jim Hague
|
r13802 | XMLRPC+email example configuration. This uses the Bugzilla at | ||
Martin Geisler
|
r13841 | ``http://my-project.org/bugzilla``, logging in as user | ||
Wagner Bruna
|
r14619 | ``bugmail@my-project.org`` with password ``plugh``. It is used with a | ||
Jim Hague
|
r13870 | collection of Mercurial repositories in ``/var/local/hg/repos/``, | ||
with a web interface at ``http://my-project.org/hg``. Bug comments | ||||
are sent to the Bugzilla email address | ||||
Patrick Mezard
|
r13854 | ``bugzilla@my-project.org``. :: | ||
Jim Hague
|
r13802 | |||
[bugzilla] | ||||
Jim Hague
|
r13870 | bzurl=http://my-project.org/bugzilla | ||
Jim Hague
|
r13802 | user=bugmail@my-project.org | ||
password=plugh | ||||
Jim Hague
|
r21842 | version=xmlrpc+email | ||
Jim Hague
|
r13802 | bzemail=bugzilla@my-project.org | ||
Jim Hague
|
r13870 | template=Changeset {node|short} in {root|basename}. | ||
{hgweb}/{webroot}/rev/{node|short}\\n | ||||
{desc}\\n | ||||
strip=5 | ||||
Jim Hague
|
r13802 | |||
[web] | ||||
Jim Hague
|
r13870 | baseurl=http://my-project.org/hg | ||
[usermap] | ||||
user@emaildomain.com=user.name@bugzilladomain.com | ||||
Jim Hague
|
r13802 | |||
Jim Hague
|
r13870 | MySQL example configuration. This has a local Bugzilla 3.2 installation | ||
in ``/opt/bugzilla-3.2``. The MySQL database is on ``localhost``, | ||||
the Bugzilla database name is ``bugs`` and MySQL is | ||||
accessed with MySQL username ``bugs`` password ``XYZZY``. It is used | ||||
with a collection of Mercurial repositories in ``/var/local/hg/repos/``, | ||||
with a web interface at ``http://my-project.org/hg``. :: | ||||
Jim Hague
|
r7504 | |||
[bugzilla] | ||||
host=localhost | ||||
password=XYZZY | ||||
version=3.0 | ||||
bzuser=unknown@domain.com | ||||
Jim Hague
|
r7618 | bzdir=/opt/bugzilla-3.2 | ||
Martin Geisler
|
r9204 | template=Changeset {node|short} in {root|basename}. | ||
{hgweb}/{webroot}/rev/{node|short}\\n | ||||
{desc}\\n | ||||
Jim Hague
|
r7504 | strip=5 | ||
[web] | ||||
Jim Hague
|
r13870 | baseurl=http://my-project.org/hg | ||
Jim Hague
|
r7504 | |||
[usermap] | ||||
user@emaildomain.com=user.name@bugzilladomain.com | ||||
Jim Hague
|
r13802 | All the above add a comment to the Bugzilla bug record of the form:: | ||
Jim Hague
|
r7504 | |||
Changeset 3b16791d6642 in repository-name. | ||||
Jim Hague
|
r13870 | http://my-project.org/hg/repository-name/rev/3b16791d6642 | ||
Jim Hague
|
r7504 | |||
Changeset commit comment. Bug 1234. | ||||
''' | ||||
Vadim Gelfer
|
r2192 | |||
Gregory Szorc
|
r28091 | from __future__ import absolute_import | ||
John Mulligan
|
r30923 | import json | ||
Gregory Szorc
|
r28091 | import re | ||
import time | ||||
Matt Mackall
|
r3891 | from mercurial.i18n import _ | ||
Joel Rosdahl
|
r6211 | from mercurial.node import short | ||
Gregory Szorc
|
r28091 | from mercurial import ( | ||
cmdutil, | ||||
error, | ||||
mail, | ||||
Boris Feld
|
r33393 | registrar, | ||
John Mulligan
|
r30923 | url, | ||
Gregory Szorc
|
r28091 | util, | ||
) | ||||
Vadim Gelfer
|
r2192 | |||
Pulkit Goyal
|
r29432 | xmlrpclib = util.xmlrpclib | ||
Pulkit Goyal
|
r29431 | |||
Augie Fackler
|
r29841 | # Note for extension authors: ONLY specify testedwith = 'ships-with-hg-core' for | ||
Augie Fackler
|
r25186 | # extensions which SHIP WITH MERCURIAL. Non-mainline extensions should | ||
# be specifying the version(s) of Mercurial they are tested with, or | ||||
# leave the attribute unspecified. | ||||
Augie Fackler
|
r29841 | testedwith = 'ships-with-hg-core' | ||
Augie Fackler
|
r16743 | |||
Boris Feld
|
r33393 | configtable = {} | ||
configitem = registrar.configitem(configtable) | ||||
configitem('bugzilla', 'apikey', | ||||
default='', | ||||
) | ||||
Boris Feld
|
r33394 | configitem('bugzilla', 'bzdir', | ||
default='/var/www/html/bugzilla', | ||||
) | ||||
Boris Feld
|
r33395 | configitem('bugzilla', 'bzemail', | ||
default=None, | ||||
) | ||||
Boris Feld
|
r33396 | configitem('bugzilla', 'bzurl', | ||
default='http://localhost/bugzilla/', | ||||
) | ||||
Boris Feld
|
r33397 | configitem('bugzilla', 'bzuser', | ||
default=None, | ||||
) | ||||
Boris Feld
|
r33398 | configitem('bugzilla', 'db', | ||
default='bugs', | ||||
) | ||||
Boris Feld
|
r33399 | configitem('bugzilla', 'fixregexp', | ||
Boris Feld
|
r33470 | default=(r'fix(?:es)?\s*(?:bugs?\s*)?,?\s*' | ||
r'(?:nos?\.?|num(?:ber)?s?)?\s*' | ||||
r'(?P<ids>(?:#?\d+\s*(?:,?\s*(?:and)?)?\s*)+)' | ||||
r'\.?\s*(?:h(?:ours?)?\s*(?P<hours>\d*(?:\.\d+)?))?') | ||||
Boris Feld
|
r33399 | ) | ||
Boris Feld
|
r33400 | configitem('bugzilla', 'fixresolution', | ||
default='FIXED', | ||||
) | ||||
Boris Feld
|
r33401 | configitem('bugzilla', 'fixstatus', | ||
default='RESOLVED', | ||||
) | ||||
Boris Feld
|
r33402 | configitem('bugzilla', 'host', | ||
default='localhost', | ||||
) | ||||
Boris Feld
|
r33524 | configitem('bugzilla', 'notify', | ||
Yuya Nishihara
|
r34918 | default=configitem.dynamicdefault, | ||
Boris Feld
|
r33524 | ) | ||
Boris Feld
|
r33433 | configitem('bugzilla', 'password', | ||
default=None, | ||||
) | ||||
Boris Feld
|
r33462 | configitem('bugzilla', 'regexp', | ||
default=(r'bugs?\s*,?\s*(?:#|nos?\.?|num(?:ber)?s?)?\s*' | ||||
r'(?P<ids>(?:\d+\s*(?:,?\s*(?:and)?)?\s*)+)' | ||||
r'\.?\s*(?:h(?:ours?)?\s*(?P<hours>\d*(?:\.\d+)?))?') | ||||
) | ||||
Boris Feld
|
r33463 | configitem('bugzilla', 'strip', | ||
default=0, | ||||
) | ||||
Boris Feld
|
r33464 | configitem('bugzilla', 'style', | ||
default=None, | ||||
) | ||||
Boris Feld
|
r33465 | configitem('bugzilla', 'template', | ||
default=None, | ||||
) | ||||
Boris Feld
|
r33466 | configitem('bugzilla', 'timeout', | ||
default=5, | ||||
) | ||||
Boris Feld
|
r33467 | configitem('bugzilla', 'user', | ||
default='bugs', | ||||
) | ||||
Boris Feld
|
r33468 | configitem('bugzilla', 'usermap', | ||
default=None, | ||||
) | ||||
Boris Feld
|
r33469 | configitem('bugzilla', 'version', | ||
default=None, | ||||
) | ||||
Boris Feld
|
r33393 | |||
Jim Hague
|
r13800 | class bzaccess(object): | ||
'''Base class for access to Bugzilla.''' | ||||
Vadim Gelfer
|
r2192 | |||
def __init__(self, ui): | ||||
self.ui = ui | ||||
Jim Hague
|
r13800 | usermap = self.ui.config('bugzilla', 'usermap') | ||
if usermap: | ||||
self.ui.readconfig(usermap, sections=['usermap']) | ||||
def map_committer(self, user): | ||||
'''map name of committer to Bugzilla user name.''' | ||||
for committer, bzuser in self.ui.configitems('usermap'): | ||||
if committer.lower() == user.lower(): | ||||
return bzuser | ||||
return user | ||||
# Methods to be implemented by access classes. | ||||
Jim Hague
|
r16221 | # | ||
# 'bugs' is a dict keyed on bug id, where values are a dict holding | ||||
timeless@mozdev.org
|
r17534 | # updates to bug state. Recognized dict keys are: | ||
Jim Hague
|
r16222 | # | ||
# 'hours': Value, float containing work hours to be updated. | ||||
Jim Hague
|
r16223 | # 'fix': If key present, bug is to be marked fixed. Value ignored. | ||
Jim Hague
|
r16222 | |||
Jim Hague
|
r16221 | def filter_real_bug_ids(self, bugs): | ||
'''remove bug IDs that do not exist in Bugzilla from bugs.''' | ||||
Jim Hague
|
r13800 | |||
Jim Hague
|
r16221 | def filter_cset_known_bug_ids(self, node, bugs): | ||
'''remove bug IDs where node occurs in comment text from bugs.''' | ||||
Jim Hague
|
r13800 | |||
Jim Hague
|
r16221 | def updatebug(self, bugid, newstate, text, committer): | ||
'''update the specified bug. Add comment text and set new states. | ||||
Jim Hague
|
r13800 | |||
If possible add the comment as being from the committer of | ||||
the changeset. Otherwise use the default Bugzilla user. | ||||
''' | ||||
Jim Hague
|
r16221 | def notify(self, bugs, committer): | ||
'''Force sending of Bugzilla notification emails. | ||||
Only required if the access method does not trigger notification | ||||
emails automatically. | ||||
''' | ||||
Jim Hague
|
r13800 | |||
# Bugzilla via direct access to MySQL database. | ||||
class bzmysql(bzaccess): | ||||
'''Support for direct MySQL access to Bugzilla. | ||||
The earliest Bugzilla version this is tested with is version 2.16. | ||||
Jim Hague
|
r13801 | |||
Jim Hague
|
r16226 | If your Bugzilla is version 3.4 or above, you are strongly | ||
Jim Hague
|
r13801 | recommended to use the XMLRPC access method instead. | ||
Jim Hague
|
r13800 | ''' | ||
@staticmethod | ||||
def sql_buglist(ids): | ||||
'''return SQL-friendly list of bug ids''' | ||||
return '(' + ','.join(map(str, ids)) + ')' | ||||
_MySQLdb = None | ||||
def __init__(self, ui): | ||||
try: | ||||
import MySQLdb as mysql | ||||
bzmysql._MySQLdb = mysql | ||||
Gregory Szorc
|
r25660 | except ImportError as err: | ||
Pierre-Yves David
|
r26587 | raise error.Abort(_('python mysql support not available: %s') % err) | ||
Jim Hague
|
r13800 | |||
bzaccess.__init__(self, ui) | ||||
Boris Feld
|
r33402 | host = self.ui.config('bugzilla', 'host') | ||
Boris Feld
|
r33467 | user = self.ui.config('bugzilla', 'user') | ||
Vadim Gelfer
|
r2192 | passwd = self.ui.config('bugzilla', 'password') | ||
Boris Feld
|
r33398 | db = self.ui.config('bugzilla', 'db') | ||
Boris Feld
|
r33466 | timeout = int(self.ui.config('bugzilla', 'timeout')) | ||
Vadim Gelfer
|
r2192 | self.ui.note(_('connecting to %s:%s as %s, password %s\n') % | ||
(host, db, user, '*' * len(passwd))) | ||||
Jim Hague
|
r13800 | self.conn = bzmysql._MySQLdb.connect(host=host, | ||
user=user, passwd=passwd, | ||||
db=db, | ||||
connect_timeout=timeout) | ||||
Vadim Gelfer
|
r2192 | self.cursor = self.conn.cursor() | ||
Jim Hague
|
r7019 | self.longdesc_id = self.get_longdesc_id() | ||
Vadim Gelfer
|
r2192 | self.user_ids = {} | ||
Jim Hague
|
r7618 | self.default_notify = "cd %(bzdir)s && ./processmail %(id)s %(user)s" | ||
Vadim Gelfer
|
r2192 | |||
def run(self, *args, **kwargs): | ||||
'''run a query.''' | ||||
self.ui.note(_('query: %s %s\n') % (args, kwargs)) | ||||
try: | ||||
self.cursor.execute(*args, **kwargs) | ||||
Jim Hague
|
r13800 | except bzmysql._MySQLdb.MySQLError: | ||
Vadim Gelfer
|
r2192 | self.ui.note(_('failed query: %s %s\n') % (args, kwargs)) | ||
raise | ||||
Jim Hague
|
r7019 | def get_longdesc_id(self): | ||
'''get identity of longdesc field''' | ||||
self.run('select fieldid from fielddefs where name = "longdesc"') | ||||
ids = self.cursor.fetchall() | ||||
if len(ids) != 1: | ||||
Pierre-Yves David
|
r26587 | raise error.Abort(_('unknown database schema')) | ||
Jim Hague
|
r7019 | return ids[0][0] | ||
Jim Hague
|
r16221 | def filter_real_bug_ids(self, bugs): | ||
'''filter not-existing bugs from set.''' | ||||
Jim Hague
|
r13800 | self.run('select bug_id from bugs where bug_id in %s' % | ||
Jim Hague
|
r16221 | bzmysql.sql_buglist(bugs.keys())) | ||
existing = [id for (id,) in self.cursor.fetchall()] | ||||
for id in bugs.keys(): | ||||
if id not in existing: | ||||
self.ui.status(_('bug %d does not exist\n') % id) | ||||
del bugs[id] | ||||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r16221 | def filter_cset_known_bug_ids(self, node, bugs): | ||
Jim Hague
|
r13799 | '''filter bug ids that already refer to this changeset from set.''' | ||
Vadim Gelfer
|
r2192 | self.run('''select bug_id from longdescs where | ||
bug_id in %s and thetext like "%%%s%%"''' % | ||||
Jim Hague
|
r16221 | (bzmysql.sql_buglist(bugs.keys()), short(node))) | ||
Vadim Gelfer
|
r2192 | for (id,) in self.cursor.fetchall(): | ||
self.ui.status(_('bug %d already knows about changeset %s\n') % | ||||
(id, short(node))) | ||||
Jim Hague
|
r16221 | del bugs[id] | ||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r16221 | def notify(self, bugs, committer): | ||
Vadim Gelfer
|
r2192 | '''tell bugzilla to send mail.''' | ||
self.ui.status(_('telling bugzilla to send mail:\n')) | ||||
Jim Hague
|
r7618 | (user, userid) = self.get_bugzilla_user(committer) | ||
Jim Hague
|
r16221 | for id in bugs.keys(): | ||
Vadim Gelfer
|
r2192 | self.ui.status(_(' bug %s\n') % id) | ||
Jim Hague
|
r7618 | cmdfmt = self.ui.config('bugzilla', 'notify', self.default_notify) | ||
Boris Feld
|
r33394 | bzdir = self.ui.config('bugzilla', 'bzdir') | ||
Jim Hague
|
r7618 | try: | ||
# Backwards-compatible with old notify string, which | ||||
# took one string. This will throw with a new format | ||||
# string. | ||||
cmd = cmdfmt % id | ||||
except TypeError: | ||||
cmd = cmdfmt % {'bzdir': bzdir, 'id': id, 'user': user} | ||||
self.ui.note(_('running notify command %s\n') % cmd) | ||||
Dirkjan Ochtman
|
r6548 | fp = util.popen('(%s) 2>&1' % cmd) | ||
Vadim Gelfer
|
r2192 | out = fp.read() | ||
ret = fp.close() | ||||
if ret: | ||||
self.ui.warn(out) | ||||
Pierre-Yves David
|
r26587 | raise error.Abort(_('bugzilla notify command %s') % | ||
Adrian Buehlmann
|
r14234 | util.explainexit(ret)[0]) | ||
Vadim Gelfer
|
r2192 | self.ui.status(_('done\n')) | ||
def get_user_id(self, user): | ||||
'''look up numeric bugzilla user id.''' | ||||
try: | ||||
return self.user_ids[user] | ||||
except KeyError: | ||||
try: | ||||
userid = int(user) | ||||
except ValueError: | ||||
self.ui.note(_('looking up user %s\n') % user) | ||||
self.run('''select userid from profiles | ||||
where login_name like %s''', user) | ||||
all = self.cursor.fetchall() | ||||
if len(all) != 1: | ||||
raise KeyError(user) | ||||
userid = int(all[0][0]) | ||||
self.user_ids[user] = userid | ||||
return userid | ||||
Jim Hague
|
r7618 | def get_bugzilla_user(self, committer): | ||
Jim Hague
|
r13801 | '''See if committer is a registered bugzilla user. Return | ||
Jim Hague
|
r7618 | bugzilla username and userid if so. If not, return default | ||
bugzilla username and userid.''' | ||||
Vadim Gelfer
|
r2306 | user = self.map_committer(committer) | ||
Vadim Gelfer
|
r2192 | try: | ||
Vadim Gelfer
|
r2306 | userid = self.get_user_id(user) | ||
Vadim Gelfer
|
r2192 | except KeyError: | ||
try: | ||||
defaultuser = self.ui.config('bugzilla', 'bzuser') | ||||
Vadim Gelfer
|
r2306 | if not defaultuser: | ||
Pierre-Yves David
|
r26587 | raise error.Abort(_('cannot find bugzilla user id for %s') % | ||
Vadim Gelfer
|
r2306 | user) | ||
Vadim Gelfer
|
r2192 | userid = self.get_user_id(defaultuser) | ||
Jim Hague
|
r7618 | user = defaultuser | ||
Vadim Gelfer
|
r2192 | except KeyError: | ||
Pierre-Yves David
|
r26587 | raise error.Abort(_('cannot find bugzilla user id for %s or %s') | ||
Brodie Rao
|
r16683 | % (user, defaultuser)) | ||
Jim Hague
|
r7618 | return (user, userid) | ||
Jim Hague
|
r16221 | def updatebug(self, bugid, newstate, text, committer): | ||
'''update bug state with comment text. | ||||
Try adding comment as committer of changeset, otherwise as | ||||
default bugzilla user.''' | ||||
Jim Hague
|
r16222 | if len(newstate) > 0: | ||
self.ui.warn(_("Bugzilla/MySQL cannot update bug state\n")) | ||||
Jim Hague
|
r7618 | (user, userid) = self.get_bugzilla_user(committer) | ||
Vadim Gelfer
|
r2192 | now = time.strftime('%Y-%m-%d %H:%M:%S') | ||
self.run('''insert into longdescs | ||||
(bug_id, who, bug_when, thetext) | ||||
values (%s, %s, %s, %s)''', | ||||
(bugid, userid, now, text)) | ||||
self.run('''insert into bugs_activity (bug_id, who, bug_when, fieldid) | ||||
values (%s, %s, %s, %s)''', | ||||
(bugid, userid, now, self.longdesc_id)) | ||||
Jim Hague
|
r7493 | self.conn.commit() | ||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r13800 | class bzmysql_2_18(bzmysql): | ||
Jim Hague
|
r7618 | '''support for bugzilla 2.18 series.''' | ||
def __init__(self, ui): | ||||
Jim Hague
|
r13800 | bzmysql.__init__(self, ui) | ||
Matt Mackall
|
r10282 | self.default_notify = \ | ||
"cd %(bzdir)s && perl -T contrib/sendbugmail.pl %(id)s %(user)s" | ||||
Jim Hague
|
r7618 | |||
Jim Hague
|
r13800 | class bzmysql_3_0(bzmysql_2_18): | ||
Jim Hague
|
r7019 | '''support for bugzilla 3.0 series.''' | ||
def __init__(self, ui): | ||||
Jim Hague
|
r13800 | bzmysql_2_18.__init__(self, ui) | ||
Jim Hague
|
r7019 | |||
def get_longdesc_id(self): | ||||
'''get identity of longdesc field''' | ||||
self.run('select id from fielddefs where name = "longdesc"') | ||||
ids = self.cursor.fetchall() | ||||
if len(ids) != 1: | ||||
Pierre-Yves David
|
r26587 | raise error.Abort(_('unknown database schema')) | ||
Jim Hague
|
r7019 | return ids[0][0] | ||
Mads Kiilerich
|
r17424 | # Bugzilla via XMLRPC interface. | ||
Jim Hague
|
r13801 | |||
Jim Hague
|
r15870 | class cookietransportrequest(object): | ||
"""A Transport request method that retains cookies over its lifetime. | ||||
Jim Hague
|
r13801 | |||
The regular xmlrpclib transports ignore cookies. Which causes | ||||
a bit of a problem when you need a cookie-based login, as with | ||||
Jim Hague
|
r21542 | the Bugzilla XMLRPC interface prior to 4.4.3. | ||
Jim Hague
|
r13801 | |||
Jim Hague
|
r15870 | So this is a helper for defining a Transport which looks for | ||
cookies being set in responses and saves them to add to all future | ||||
requests. | ||||
Jim Hague
|
r13801 | """ | ||
# Inspiration drawn from | ||||
# http://blog.godson.in/2010/09/how-to-make-python-xmlrpclib-client.html | ||||
# http://www.itkovian.net/base/transport-class-for-pythons-xml-rpc-lib/ | ||||
cookies = [] | ||||
def send_cookies(self, connection): | ||||
if self.cookies: | ||||
for cookie in self.cookies: | ||||
connection.putheader("Cookie", cookie) | ||||
def request(self, host, handler, request_body, verbose=0): | ||||
self.verbose = verbose | ||||
Jim Hague
|
r16193 | self.accept_gzip_encoding = False | ||
Jim Hague
|
r13801 | |||
# issue XML-RPC request | ||||
h = self.make_connection(host) | ||||
if verbose: | ||||
h.set_debuglevel(1) | ||||
self.send_request(h, handler, request_body) | ||||
self.send_host(h, host) | ||||
self.send_cookies(h) | ||||
self.send_user_agent(h) | ||||
self.send_content(h, request_body) | ||||
Augie Fackler
|
r30478 | # Deal with differences between Python 2.6 and 2.7. | ||
Jim Hague
|
r13801 | # In the former h is a HTTP(S). In the latter it's a | ||
Augie Fackler
|
r30478 | # HTTP(S)Connection. Luckily, the 2.6 implementation of | ||
Jim Hague
|
r13801 | # HTTP(S) has an underlying HTTP(S)Connection, so extract | ||
# that and use it. | ||||
try: | ||||
response = h.getresponse() | ||||
except AttributeError: | ||||
response = h._conn.getresponse() | ||||
# Add any cookie definitions to our list. | ||||
for header in response.msg.getallmatchingheaders("Set-Cookie"): | ||||
val = header.split(": ", 1)[1] | ||||
cookie = val.split(";", 1)[0] | ||||
self.cookies.append(cookie) | ||||
if response.status != 200: | ||||
raise xmlrpclib.ProtocolError(host + handler, response.status, | ||||
response.reason, response.msg.headers) | ||||
payload = response.read() | ||||
parser, unmarshaller = self.getparser() | ||||
parser.feed(payload) | ||||
parser.close() | ||||
return unmarshaller.close() | ||||
Jim Hague
|
r15870 | # The explicit calls to the underlying xmlrpclib __init__() methods are | ||
# necessary. The xmlrpclib.Transport classes are old-style classes, and | ||||
# it turns out their __init__() doesn't get called when doing multiple | ||||
# inheritance with a new-style class. | ||||
class cookietransport(cookietransportrequest, xmlrpclib.Transport): | ||||
def __init__(self, use_datetime=0): | ||||
Steven Stallion
|
r16649 | if util.safehasattr(xmlrpclib.Transport, "__init__"): | ||
xmlrpclib.Transport.__init__(self, use_datetime) | ||||
Jim Hague
|
r15870 | |||
class cookiesafetransport(cookietransportrequest, xmlrpclib.SafeTransport): | ||||
def __init__(self, use_datetime=0): | ||||
Steven Stallion
|
r16649 | if util.safehasattr(xmlrpclib.Transport, "__init__"): | ||
xmlrpclib.SafeTransport.__init__(self, use_datetime) | ||||
Jim Hague
|
r15870 | |||
Jim Hague
|
r13801 | class bzxmlrpc(bzaccess): | ||
"""Support for access to Bugzilla via the Bugzilla XMLRPC API. | ||||
Requires a minimum Bugzilla version 3.4. | ||||
""" | ||||
def __init__(self, ui): | ||||
bzaccess.__init__(self, ui) | ||||
Boris Feld
|
r33396 | bzweb = self.ui.config('bugzilla', 'bzurl') | ||
Jim Hague
|
r13801 | bzweb = bzweb.rstrip("/") + "/xmlrpc.cgi" | ||
Boris Feld
|
r33467 | user = self.ui.config('bugzilla', 'user') | ||
Jim Hague
|
r13801 | passwd = self.ui.config('bugzilla', 'password') | ||
Boris Feld
|
r33401 | self.fixstatus = self.ui.config('bugzilla', 'fixstatus') | ||
Boris Feld
|
r33400 | self.fixresolution = self.ui.config('bugzilla', 'fixresolution') | ||
Jim Hague
|
r16223 | |||
Jim Hague
|
r15870 | self.bzproxy = xmlrpclib.ServerProxy(bzweb, self.transport(bzweb)) | ||
Jim Hague
|
r16224 | ver = self.bzproxy.Bugzilla.version()['version'].split('.') | ||
self.bzvermajor = int(ver[0]) | ||||
self.bzverminor = int(ver[1]) | ||||
Jim Hague
|
r21542 | login = self.bzproxy.User.login({'login': user, 'password': passwd, | ||
'restrict_login': True}) | ||||
self.bztoken = login.get('token', '') | ||||
Jim Hague
|
r13801 | |||
Jim Hague
|
r15870 | def transport(self, uri): | ||
Gregory Szorc
|
r31570 | if util.urlreq.urlparse(uri, "http")[0] == "https": | ||
Jim Hague
|
r15870 | return cookiesafetransport() | ||
else: | ||||
return cookietransport() | ||||
Jim Hague
|
r13801 | def get_bug_comments(self, id): | ||
"""Return a string with all comment text for a bug.""" | ||||
Augie Fackler
|
r20673 | c = self.bzproxy.Bug.comments({'ids': [id], | ||
Jim Hague
|
r21542 | 'include_fields': ['text'], | ||
'token': self.bztoken}) | ||||
Jim Hague
|
r13801 | return ''.join([t['text'] for t in c['bugs'][str(id)]['comments']]) | ||
Jim Hague
|
r16221 | def filter_real_bug_ids(self, bugs): | ||
Augie Fackler
|
r20673 | probe = self.bzproxy.Bug.get({'ids': sorted(bugs.keys()), | ||
'include_fields': [], | ||||
'permissive': True, | ||||
Jim Hague
|
r21542 | 'token': self.bztoken, | ||
Augie Fackler
|
r20673 | }) | ||
Jim Hague
|
r16221 | for badbug in probe['faults']: | ||
id = badbug['id'] | ||||
self.ui.status(_('bug %d does not exist\n') % id) | ||||
del bugs[id] | ||||
Jim Hague
|
r13801 | |||
Jim Hague
|
r16221 | def filter_cset_known_bug_ids(self, node, bugs): | ||
for id in sorted(bugs.keys()): | ||||
Jim Hague
|
r13801 | if self.get_bug_comments(id).find(short(node)) != -1: | ||
self.ui.status(_('bug %d already knows about changeset %s\n') % | ||||
(id, short(node))) | ||||
Jim Hague
|
r16221 | del bugs[id] | ||
Jim Hague
|
r13801 | |||
Jim Hague
|
r16221 | def updatebug(self, bugid, newstate, text, committer): | ||
Jim Hague
|
r16223 | args = {} | ||
Jim Hague
|
r16222 | if 'hours' in newstate: | ||
args['work_time'] = newstate['hours'] | ||||
Jim Hague
|
r13801 | |||
Jim Hague
|
r16223 | if self.bzvermajor >= 4: | ||
args['ids'] = [bugid] | ||||
args['comment'] = {'body' : text} | ||||
Jim Hague
|
r16876 | if 'fix' in newstate: | ||
args['status'] = self.fixstatus | ||||
args['resolution'] = self.fixresolution | ||||
Jim Hague
|
r21542 | args['token'] = self.bztoken | ||
Jim Hague
|
r16223 | self.bzproxy.Bug.update(args) | ||
else: | ||||
if 'fix' in newstate: | ||||
self.ui.warn(_("Bugzilla/XMLRPC needs Bugzilla 4.0 or later " | ||||
"to mark bugs fixed\n")) | ||||
args['id'] = bugid | ||||
args['comment'] = text | ||||
self.bzproxy.Bug.add_comment(args) | ||||
Jim Hague
|
r13801 | |||
Jim Hague
|
r13802 | class bzxmlrpcemail(bzxmlrpc): | ||
"""Read data from Bugzilla via XMLRPC, send updates via email. | ||||
Advantages of sending updates via email: | ||||
1. Comments can be added as any user, not just logged in user. | ||||
Jim Hague
|
r16222 | 2. Bug statuses or other fields not accessible via XMLRPC can | ||
potentially be updated. | ||||
Jim Hague
|
r16223 | There is no XMLRPC function to change bug status before Bugzilla | ||
4.0, so bugs cannot be marked fixed via XMLRPC before Bugzilla 4.0. | ||||
But bugs can be marked fixed via email from 3.4 onwards. | ||||
Jim Hague
|
r13802 | """ | ||
Jim Hague
|
r16224 | # The email interface changes subtly between 3.4 and 3.6. In 3.4, | ||
# in-email fields are specified as '@<fieldname> = <value>'. In | ||||
# 3.6 this becomes '@<fieldname> <value>'. And fieldname @bug_id | ||||
# in 3.4 becomes @id in 3.6. 3.6 and 4.0 both maintain backwards | ||||
# compatibility, but rather than rely on this use the new format for | ||||
# 4.0 onwards. | ||||
Jim Hague
|
r13802 | def __init__(self, ui): | ||
bzxmlrpc.__init__(self, ui) | ||||
self.bzemail = self.ui.config('bugzilla', 'bzemail') | ||||
if not self.bzemail: | ||||
Pierre-Yves David
|
r26587 | raise error.Abort(_("configuration 'bzemail' missing")) | ||
Jim Hague
|
r13802 | mail.validateconfig(self.ui) | ||
Jim Hague
|
r16224 | def makecommandline(self, fieldname, value): | ||
if self.bzvermajor >= 4: | ||||
return "@%s %s" % (fieldname, str(value)) | ||||
else: | ||||
if fieldname == "id": | ||||
fieldname = "bug_id" | ||||
return "@%s = %s" % (fieldname, str(value)) | ||||
Jim Hague
|
r13802 | def send_bug_modify_email(self, bugid, commands, comment, committer): | ||
'''send modification message to Bugzilla bug via email. | ||||
The message format is documented in the Bugzilla email_in.pl | ||||
specification. commands is a list of command lines, comment is the | ||||
comment text. | ||||
To stop users from crafting commit comments with | ||||
Bugzilla commands, specify the bug ID via the message body, rather | ||||
than the subject line, and leave a blank line after it. | ||||
''' | ||||
user = self.map_committer(committer) | ||||
Jim Hague
|
r21542 | matches = self.bzproxy.User.get({'match': [user], | ||
'token': self.bztoken}) | ||||
Jim Hague
|
r13802 | if not matches['users']: | ||
Boris Feld
|
r33467 | user = self.ui.config('bugzilla', 'user') | ||
Jim Hague
|
r21542 | matches = self.bzproxy.User.get({'match': [user], | ||
'token': self.bztoken}) | ||||
Jim Hague
|
r13802 | if not matches['users']: | ||
Pierre-Yves David
|
r26587 | raise error.Abort(_("default bugzilla user %s email not found") | ||
% user) | ||||
Jim Hague
|
r13802 | user = matches['users'][0]['email'] | ||
Jim Hague
|
r16224 | commands.append(self.makecommandline("id", bugid)) | ||
Jim Hague
|
r13802 | |||
Jim Hague
|
r16224 | text = "\n".join(commands) + "\n\n" + comment | ||
Jim Hague
|
r13802 | |||
_charsets = mail._charsets(self.ui) | ||||
user = mail.addressencode(self.ui, user, _charsets) | ||||
bzemail = mail.addressencode(self.ui, self.bzemail, _charsets) | ||||
msg = mail.mimeencode(self.ui, text, _charsets) | ||||
msg['From'] = user | ||||
msg['To'] = bzemail | ||||
msg['Subject'] = mail.headencode(self.ui, "Bug modification", _charsets) | ||||
sendmail = mail.connect(self.ui) | ||||
sendmail(user, bzemail, msg.as_string()) | ||||
Jim Hague
|
r16221 | def updatebug(self, bugid, newstate, text, committer): | ||
Jim Hague
|
r16222 | cmds = [] | ||
if 'hours' in newstate: | ||||
cmds.append(self.makecommandline("work_time", newstate['hours'])) | ||||
Jim Hague
|
r16223 | if 'fix' in newstate: | ||
cmds.append(self.makecommandline("bug_status", self.fixstatus)) | ||||
cmds.append(self.makecommandline("resolution", self.fixresolution)) | ||||
Jim Hague
|
r16222 | self.send_bug_modify_email(bugid, cmds, text, committer) | ||
Jim Hague
|
r13802 | |||
John Mulligan
|
r30923 | class NotFound(LookupError): | ||
pass | ||||
class bzrestapi(bzaccess): | ||||
"""Read and write bugzilla data using the REST API available since | ||||
Bugzilla 5.0. | ||||
""" | ||||
def __init__(self, ui): | ||||
bzaccess.__init__(self, ui) | ||||
Boris Feld
|
r33396 | bz = self.ui.config('bugzilla', 'bzurl') | ||
John Mulligan
|
r30923 | self.bzroot = '/'.join([bz, 'rest']) | ||
Boris Feld
|
r33393 | self.apikey = self.ui.config('bugzilla', 'apikey') | ||
Boris Feld
|
r33467 | self.user = self.ui.config('bugzilla', 'user') | ||
John Mulligan
|
r30923 | self.passwd = self.ui.config('bugzilla', 'password') | ||
Boris Feld
|
r33401 | self.fixstatus = self.ui.config('bugzilla', 'fixstatus') | ||
Boris Feld
|
r33400 | self.fixresolution = self.ui.config('bugzilla', 'fixresolution') | ||
John Mulligan
|
r30923 | |||
def apiurl(self, targets, include_fields=None): | ||||
url = '/'.join([self.bzroot] + [str(t) for t in targets]) | ||||
qv = {} | ||||
if self.apikey: | ||||
qv['api_key'] = self.apikey | ||||
elif self.user and self.passwd: | ||||
qv['login'] = self.user | ||||
qv['password'] = self.passwd | ||||
if include_fields: | ||||
qv['include_fields'] = include_fields | ||||
if qv: | ||||
url = '%s?%s' % (url, util.urlreq.urlencode(qv)) | ||||
return url | ||||
def _fetch(self, burl): | ||||
try: | ||||
resp = url.open(self.ui, burl) | ||||
return json.loads(resp.read()) | ||||
except util.urlerr.httperror as inst: | ||||
if inst.code == 401: | ||||
raise error.Abort(_('authorization failed')) | ||||
if inst.code == 404: | ||||
raise NotFound() | ||||
else: | ||||
raise | ||||
def _submit(self, burl, data, method='POST'): | ||||
data = json.dumps(data) | ||||
if method == 'PUT': | ||||
class putrequest(util.urlreq.request): | ||||
def get_method(self): | ||||
return 'PUT' | ||||
request_type = putrequest | ||||
else: | ||||
request_type = util.urlreq.request | ||||
req = request_type(burl, data, | ||||
{'Content-Type': 'application/json'}) | ||||
try: | ||||
resp = url.opener(self.ui).open(req) | ||||
return json.loads(resp.read()) | ||||
except util.urlerr.httperror as inst: | ||||
if inst.code == 401: | ||||
raise error.Abort(_('authorization failed')) | ||||
if inst.code == 404: | ||||
raise NotFound() | ||||
else: | ||||
raise | ||||
def filter_real_bug_ids(self, bugs): | ||||
'''remove bug IDs that do not exist in Bugzilla from bugs.''' | ||||
badbugs = set() | ||||
for bugid in bugs: | ||||
burl = self.apiurl(('bug', bugid), include_fields='status') | ||||
try: | ||||
self._fetch(burl) | ||||
except NotFound: | ||||
badbugs.add(bugid) | ||||
for bugid in badbugs: | ||||
del bugs[bugid] | ||||
def filter_cset_known_bug_ids(self, node, bugs): | ||||
'''remove bug IDs where node occurs in comment text from bugs.''' | ||||
sn = short(node) | ||||
for bugid in bugs.keys(): | ||||
burl = self.apiurl(('bug', bugid, 'comment'), include_fields='text') | ||||
result = self._fetch(burl) | ||||
comments = result['bugs'][str(bugid)]['comments'] | ||||
if any(sn in c['text'] for c in comments): | ||||
self.ui.status(_('bug %d already knows about changeset %s\n') % | ||||
(bugid, sn)) | ||||
del bugs[bugid] | ||||
def updatebug(self, bugid, newstate, text, committer): | ||||
'''update the specified bug. Add comment text and set new states. | ||||
If possible add the comment as being from the committer of | ||||
the changeset. Otherwise use the default Bugzilla user. | ||||
''' | ||||
bugmod = {} | ||||
if 'hours' in newstate: | ||||
bugmod['work_time'] = newstate['hours'] | ||||
if 'fix' in newstate: | ||||
bugmod['status'] = self.fixstatus | ||||
bugmod['resolution'] = self.fixresolution | ||||
if bugmod: | ||||
# if we have to change the bugs state do it here | ||||
bugmod['comment'] = { | ||||
'comment': text, | ||||
'is_private': False, | ||||
'is_markdown': False, | ||||
} | ||||
burl = self.apiurl(('bug', bugid)) | ||||
self._submit(burl, bugmod, method='PUT') | ||||
self.ui.debug('updated bug %s\n' % bugid) | ||||
else: | ||||
burl = self.apiurl(('bug', bugid, 'comment')) | ||||
self._submit(burl, { | ||||
'comment': text, | ||||
'is_private': False, | ||||
'is_markdown': False, | ||||
}) | ||||
self.ui.debug('added comment to bug %s\n' % bugid) | ||||
def notify(self, bugs, committer): | ||||
'''Force sending of Bugzilla notification emails. | ||||
Only required if the access method does not trigger notification | ||||
emails automatically. | ||||
''' | ||||
pass | ||||
Vadim Gelfer
|
r2192 | class bugzilla(object): | ||
# supported versions of bugzilla. different versions have | ||||
# different schemas. | ||||
_versions = { | ||||
Jim Hague
|
r13800 | '2.16': bzmysql, | ||
'2.18': bzmysql_2_18, | ||||
Jim Hague
|
r13801 | '3.0': bzmysql_3_0, | ||
Jim Hague
|
r13802 | 'xmlrpc': bzxmlrpc, | ||
John Mulligan
|
r30923 | 'xmlrpc+email': bzxmlrpcemail, | ||
'restapi': bzrestapi, | ||||
Vadim Gelfer
|
r2192 | } | ||
def __init__(self, ui, repo): | ||||
self.ui = ui | ||||
self.repo = repo | ||||
Jim Hague
|
r21855 | bzversion = self.ui.config('bugzilla', 'version') | ||
try: | ||||
bzclass = bugzilla._versions[bzversion] | ||||
except KeyError: | ||||
Pierre-Yves David
|
r26587 | raise error.Abort(_('bugzilla version %s not supported') % | ||
Jim Hague
|
r21855 | bzversion) | ||
self.bzdriver = bzclass(self.ui) | ||||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r21855 | self.bug_re = re.compile( | ||
Boris Feld
|
r33462 | self.ui.config('bugzilla', 'regexp'), re.IGNORECASE) | ||
Jim Hague
|
r21855 | self.fix_re = re.compile( | ||
Boris Feld
|
r33399 | self.ui.config('bugzilla', 'fixregexp'), re.IGNORECASE) | ||
Jim Hague
|
r21855 | self.split_re = re.compile(r'\D+') | ||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r16221 | def find_bugs(self, ctx): | ||
'''return bugs dictionary created from commit comment. | ||||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r16221 | Extract bug info from changeset comments. Filter out any that are | ||
Jim Hague
|
r13799 | not known to Bugzilla, and any that already have a reference to | ||
the given changeset in their comments. | ||||
''' | ||||
Vadim Gelfer
|
r2192 | start = 0 | ||
Jim Hague
|
r16222 | hours = 0.0 | ||
Jim Hague
|
r16221 | bugs = {} | ||
Jim Hague
|
r21855 | bugmatch = self.bug_re.search(ctx.description(), start) | ||
fixmatch = self.fix_re.search(ctx.description(), start) | ||||
Vadim Gelfer
|
r2192 | while True: | ||
Jim Hague
|
r16222 | bugattribs = {} | ||
Jim Hague
|
r16223 | if not bugmatch and not fixmatch: | ||
Vadim Gelfer
|
r2192 | break | ||
Jim Hague
|
r16223 | if not bugmatch: | ||
m = fixmatch | ||||
elif not fixmatch: | ||||
m = bugmatch | ||||
else: | ||||
if bugmatch.start() < fixmatch.start(): | ||||
m = bugmatch | ||||
else: | ||||
m = fixmatch | ||||
Vadim Gelfer
|
r2192 | start = m.end() | ||
Jim Hague
|
r16223 | if m is bugmatch: | ||
Jim Hague
|
r21855 | bugmatch = self.bug_re.search(ctx.description(), start) | ||
Jim Hague
|
r16223 | if 'fix' in bugattribs: | ||
del bugattribs['fix'] | ||||
else: | ||||
Jim Hague
|
r21855 | fixmatch = self.fix_re.search(ctx.description(), start) | ||
Jim Hague
|
r16223 | bugattribs['fix'] = None | ||
Jim Hague
|
r16222 | try: | ||
ids = m.group('ids') | ||||
except IndexError: | ||||
ids = m.group(1) | ||||
try: | ||||
hours = float(m.group('hours')) | ||||
bugattribs['hours'] = hours | ||||
except IndexError: | ||||
pass | ||||
except TypeError: | ||||
pass | ||||
except ValueError: | ||||
self.ui.status(_("%s: invalid hours\n") % m.group('hours')) | ||||
Jim Hague
|
r21855 | for id in self.split_re.split(ids): | ||
Matt Mackall
|
r10282 | if not id: | ||
continue | ||||
Jim Hague
|
r16222 | bugs[int(id)] = bugattribs | ||
Jim Hague
|
r16221 | if bugs: | ||
Jim Hague
|
r21855 | self.bzdriver.filter_real_bug_ids(bugs) | ||
Jim Hague
|
r16221 | if bugs: | ||
Jim Hague
|
r21855 | self.bzdriver.filter_cset_known_bug_ids(ctx.node(), bugs) | ||
Jim Hague
|
r16221 | return bugs | ||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r16221 | def update(self, bugid, newstate, ctx): | ||
Vadim Gelfer
|
r2192 | '''update bugzilla bug with reference to changeset.''' | ||
def webroot(root): | ||||
'''strip leading prefix of repo root and turn into | ||||
url-safe path.''' | ||||
Boris Feld
|
r33463 | count = int(self.ui.config('bugzilla', 'strip')) | ||
Vadim Gelfer
|
r2192 | root = util.pconvert(root) | ||
while count > 0: | ||||
c = root.find('/') | ||||
if c == -1: | ||||
break | ||||
Matt Mackall
|
r10282 | root = root[c + 1:] | ||
Vadim Gelfer
|
r2192 | count -= 1 | ||
return root | ||||
Yuya Nishihara
|
r28950 | mapfile = None | ||
Vadim Gelfer
|
r2192 | tmpl = self.ui.config('bugzilla', 'template') | ||
Yuya Nishihara
|
r28950 | if not tmpl: | ||
mapfile = self.ui.config('bugzilla', 'style') | ||||
Vadim Gelfer
|
r2192 | if not mapfile and not tmpl: | ||
tmpl = _('changeset {node|short} in repo {root} refers ' | ||||
'to bug {bug}.\ndetails:\n\t{desc|tabindent}') | ||||
Yuya Nishihara
|
r32839 | spec = cmdutil.logtemplatespec(tmpl, mapfile) | ||
t = cmdutil.changeset_templater(self.ui, self.repo, spec, | ||||
False, None, False) | ||||
Matt Mackall
|
r3741 | self.ui.pushbuffer() | ||
Dirkjan Ochtman
|
r7369 | t.show(ctx, changes=ctx.changeset(), | ||
Vadim Gelfer
|
r2192 | bug=str(bugid), | ||
Vadim Gelfer
|
r2197 | hgweb=self.ui.config('web', 'baseurl'), | ||
Vadim Gelfer
|
r2192 | root=self.repo.root, | ||
webroot=webroot(self.repo.root)) | ||||
Matt Mackall
|
r3741 | data = self.ui.popbuffer() | ||
Jim Hague
|
r21855 | self.bzdriver.updatebug(bugid, newstate, data, util.email(ctx.user())) | ||
def notify(self, bugs, committer): | ||||
'''ensure Bugzilla users are notified of bug change.''' | ||||
self.bzdriver.notify(bugs, committer) | ||||
Vadim Gelfer
|
r2192 | |||
def hook(ui, repo, hooktype, node=None, **kwargs): | ||||
'''add comment to bugzilla for each changeset that refers to a | ||||
bugzilla bug id. only add a comment once per bug, so same change | ||||
seen multiple times does not fill bug with duplicate data.''' | ||||
if node is None: | ||||
Pierre-Yves David
|
r26587 | raise error.Abort(_('hook type %s does not pass a changeset id') % | ||
Vadim Gelfer
|
r2192 | hooktype) | ||
try: | ||||
bz = bugzilla(ui, repo) | ||||
Matt Mackall
|
r6747 | ctx = repo[node] | ||
Jim Hague
|
r16221 | bugs = bz.find_bugs(ctx) | ||
if bugs: | ||||
for bug in bugs: | ||||
bz.update(bug, bugs[bug], ctx) | ||||
bz.notify(bugs, util.email(ctx.user())) | ||||
Gregory Szorc
|
r25660 | except Exception as e: | ||
Pierre-Yves David
|
r26587 | raise error.Abort(_('Bugzilla error: %s') % e) | ||