bugzilla.py
773 lines
| 27.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
|
r13801 | # Copyright 2011 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
|
r13801 | The hook does not change bug status. | ||
Jim Hague
|
r7504 | |||
Jim Hague
|
r13802 | Three basic modes of access to Bugzilla are provided: | ||
Jim Hague
|
r7504 | |||
Jim Hague
|
r13802 | 1. Access via the Bugzilla XMLRPC interface. Requires Bugzilla 3.4 or later. | ||
2. Check data via the Bugzilla XMLRPC interface and submit bug change | ||||
via email to Bugzilla email interface. Requires Bugzilla 3.4 or later. | ||||
Martin Geisler
|
r9203 | |||
Jim Hague
|
r13896 | 3. 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 | ||||
forward. | ||||
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 | ||||
necessary to add comments. | ||||
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 | ||||
that the Mercurial user email is not recognised by Bugzilla as a Bugzilla | ||||
Jim Hague
|
r13871 | user, the email associated with the Bugzilla username used to log into | ||
Bugzilla is used instead as the source of the comment. | ||||
Jim Hague
|
r13802 | |||
Configuration items common to all access modes: | ||||
Martin Geisler
|
r9203 | |||
Martin Geisler
|
r13833 | bugzilla.version | ||
Martin Geisler
|
r13883 | This access type to use. Values recognised are: | ||
Jim Hague
|
r13871 | |||
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 | ||
Martin Geisler
|
r9252 | Regular expression to match bug IDs in changeset commit message. | ||
Martin Geisler
|
r13841 | Must contain one "()" group. The default expression matches ``Bug | ||
1234``, ``Bug no. 1234``, ``Bug number 1234``, ``Bugs 1234,5678``, | ||||
``Bug 1234 and 5678`` and variations thereof. Matching is case | ||||
insensitive. | ||||
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 | |||
Jim Hague
|
r13801 | XMLRPC access mode configuration: | ||
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. | ||
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 | ||||
version=xmlrpc | ||||
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 | |||
Matt Mackall
|
r3891 | from mercurial.i18n import _ | ||
Joel Rosdahl
|
r6211 | from mercurial.node import short | ||
Jim Hague
|
r13802 | from mercurial import cmdutil, mail, templater, util | ||
Jim Hague
|
r15870 | import re, time, urlparse, xmlrpclib | ||
Vadim Gelfer
|
r2192 | |||
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. | ||||
def filter_real_bug_ids(self, ids): | ||||
'''remove bug IDs that do not exist in Bugzilla from set.''' | ||||
pass | ||||
def filter_cset_known_bug_ids(self, node, ids): | ||||
'''remove bug IDs where node occurs in comment text from set.''' | ||||
pass | ||||
def add_comment(self, bugid, text, committer): | ||||
'''add comment to bug. | ||||
If possible add the comment as being from the committer of | ||||
the changeset. Otherwise use the default Bugzilla user. | ||||
''' | ||||
pass | ||||
def notify(self, ids, committer): | ||||
'''Force sending of Bugzilla notification emails.''' | ||||
pass | ||||
# 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 | |||
If your Bugzilla is version 3.2 or above, you are strongly | ||||
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 | ||||
except ImportError, err: | ||||
raise util.Abort(_('python mysql support not available: %s') % err) | ||||
bzaccess.__init__(self, ui) | ||||
Vadim Gelfer
|
r2192 | host = self.ui.config('bugzilla', 'host', 'localhost') | ||
user = self.ui.config('bugzilla', 'user', 'bugs') | ||||
passwd = self.ui.config('bugzilla', 'password') | ||||
db = self.ui.config('bugzilla', 'db', 'bugs') | ||||
timeout = int(self.ui.config('bugzilla', 'timeout', 5)) | ||||
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: | ||||
raise util.Abort(_('unknown database schema')) | ||||
return ids[0][0] | ||||
Vadim Gelfer
|
r2192 | def filter_real_bug_ids(self, ids): | ||
Jim Hague
|
r13799 | '''filter not-existing bug ids from set.''' | ||
Jim Hague
|
r13800 | self.run('select bug_id from bugs where bug_id in %s' % | ||
bzmysql.sql_buglist(ids)) | ||||
Jim Hague
|
r13799 | return set([c[0] for c in self.cursor.fetchall()]) | ||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r13798 | def filter_cset_known_bug_ids(self, node, ids): | ||
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
|
r13800 | (bzmysql.sql_buglist(ids), 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
|
r13799 | ids.discard(id) | ||
return ids | ||||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r7618 | def notify(self, ids, 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) | ||
Vadim Gelfer
|
r2192 | for id in ids: | ||
self.ui.status(_(' bug %s\n') % id) | ||||
Jim Hague
|
r7618 | cmdfmt = self.ui.config('bugzilla', 'notify', self.default_notify) | ||
bzdir = self.ui.config('bugzilla', 'bzdir', '/var/www/html/bugzilla') | ||||
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) | ||||
raise util.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: | ||
raise util.Abort(_('cannot find bugzilla user id for %s') % | ||||
user) | ||||
Vadim Gelfer
|
r2192 | userid = self.get_user_id(defaultuser) | ||
Jim Hague
|
r7618 | user = defaultuser | ||
Vadim Gelfer
|
r2192 | except KeyError: | ||
Vadim Gelfer
|
r2306 | raise util.Abort(_('cannot find bugzilla user id for %s or %s') % | ||
(user, defaultuser)) | ||||
Jim Hague
|
r7618 | return (user, userid) | ||
def add_comment(self, bugid, text, committer): | ||||
'''add comment to bug. try adding comment as committer of | ||||
changeset, otherwise as default bugzilla user.''' | ||||
(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: | ||||
raise util.Abort(_('unknown database schema')) | ||||
return ids[0][0] | ||||
Jim Hague
|
r13801 | # Buzgilla via XMLRPC interface. | ||
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 | ||||
the Bugzilla XMLRPC interface. | ||||
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 | ||||
# 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) | ||||
# Deal with differences between Python 2.4-2.6 and 2.7. | ||||
# In the former h is a HTTP(S). In the latter it's a | ||||
# HTTP(S)Connection. Luckily, the 2.4-2.6 implementation of | ||||
# 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): | ||||
xmlrpclib.Transport.__init__(self, use_datetime) | ||||
class cookiesafetransport(cookietransportrequest, xmlrpclib.SafeTransport): | ||||
def __init__(self, use_datetime=0): | ||||
xmlrpclib.SafeTransport.__init__(self, use_datetime) | ||||
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) | ||||
bzweb = self.ui.config('bugzilla', 'bzurl', | ||||
'http://localhost/bugzilla/') | ||||
bzweb = bzweb.rstrip("/") + "/xmlrpc.cgi" | ||||
user = self.ui.config('bugzilla', 'user', 'bugs') | ||||
passwd = self.ui.config('bugzilla', 'password') | ||||
Jim Hague
|
r15870 | self.bzproxy = xmlrpclib.ServerProxy(bzweb, self.transport(bzweb)) | ||
Jim Hague
|
r13801 | self.bzproxy.User.login(dict(login=user, password=passwd)) | ||
Jim Hague
|
r15870 | def transport(self, uri): | ||
if urlparse.urlparse(uri, "http")[0] == "https": | ||||
return cookiesafetransport() | ||||
else: | ||||
return cookietransport() | ||||
Jim Hague
|
r13801 | def get_bug_comments(self, id): | ||
"""Return a string with all comment text for a bug.""" | ||||
c = self.bzproxy.Bug.comments(dict(ids=[id])) | ||||
return ''.join([t['text'] for t in c['bugs'][str(id)]['comments']]) | ||||
def filter_real_bug_ids(self, ids): | ||||
res = set() | ||||
bugs = self.bzproxy.Bug.get(dict(ids=sorted(ids), permissive=True)) | ||||
for bug in bugs['bugs']: | ||||
res.add(bug['id']) | ||||
return res | ||||
def filter_cset_known_bug_ids(self, node, ids): | ||||
for id in sorted(ids): | ||||
if self.get_bug_comments(id).find(short(node)) != -1: | ||||
self.ui.status(_('bug %d already knows about changeset %s\n') % | ||||
(id, short(node))) | ||||
ids.discard(id) | ||||
return ids | ||||
def add_comment(self, bugid, text, committer): | ||||
self.bzproxy.Bug.add_comment(dict(id=bugid, comment=text)) | ||||
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. | ||||
2. Bug statuses and other fields not accessible via XMLRPC can | ||||
be updated. This is not currently used. | ||||
""" | ||||
def __init__(self, ui): | ||||
bzxmlrpc.__init__(self, ui) | ||||
self.bzemail = self.ui.config('bugzilla', 'bzemail') | ||||
if not self.bzemail: | ||||
raise util.Abort(_("configuration 'bzemail' missing")) | ||||
mail.validateconfig(self.ui) | ||||
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) | ||||
matches = self.bzproxy.User.get(dict(match=[user])) | ||||
if not matches['users']: | ||||
user = self.ui.config('bugzilla', 'user', 'bugs') | ||||
matches = self.bzproxy.User.get(dict(match=[user])) | ||||
if not matches['users']: | ||||
raise util.Abort(_("default bugzilla user %s email not found") % | ||||
user) | ||||
user = matches['users'][0]['email'] | ||||
text = "\n".join(commands) + "\n@bug_id = %d\n\n" % bugid + comment | ||||
_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()) | ||||
def add_comment(self, bugid, text, committer): | ||||
self.send_bug_modify_email(bugid, [], text, committer) | ||||
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, | ||
'xmlrpc+email': bzxmlrpcemail | ||||
Vadim Gelfer
|
r2192 | } | ||
_default_bug_re = (r'bugs?\s*,?\s*(?:#|nos?\.?|num(?:ber)?s?)?\s*' | ||||
r'((?:\d+\s*(?:,?\s*(?:and)?)?\s*)+)') | ||||
_bz = None | ||||
def __init__(self, ui, repo): | ||||
self.ui = ui | ||||
self.repo = repo | ||||
def bz(self): | ||||
'''return object that knows how to talk to bugzilla version in | ||||
use.''' | ||||
if bugzilla._bz is None: | ||||
bzversion = self.ui.config('bugzilla', 'version') | ||||
try: | ||||
bzclass = bugzilla._versions[bzversion] | ||||
except KeyError: | ||||
raise util.Abort(_('bugzilla version %s not supported') % | ||||
bzversion) | ||||
bugzilla._bz = bzclass(self.ui) | ||||
return bugzilla._bz | ||||
def __getattr__(self, key): | ||||
return getattr(self.bz(), key) | ||||
_bug_re = None | ||||
_split_re = None | ||||
Benoit Boissinot
|
r3976 | def find_bug_ids(self, ctx): | ||
Jim Hague
|
r13799 | '''return set of integer bug IDs from commit comment. | ||
Vadim Gelfer
|
r2192 | |||
Jim Hague
|
r13799 | Extract bug IDs from changeset comments. Filter out any that are | ||
not known to Bugzilla, and any that already have a reference to | ||||
the given changeset in their comments. | ||||
''' | ||||
Vadim Gelfer
|
r2192 | if bugzilla._bug_re is None: | ||
bugzilla._bug_re = re.compile( | ||||
self.ui.config('bugzilla', 'regexp', bugzilla._default_bug_re), | ||||
re.IGNORECASE) | ||||
bugzilla._split_re = re.compile(r'\D+') | ||||
start = 0 | ||||
Benoit Boissinot
|
r8455 | ids = set() | ||
Vadim Gelfer
|
r2192 | while True: | ||
Benoit Boissinot
|
r3976 | m = bugzilla._bug_re.search(ctx.description(), start) | ||
Vadim Gelfer
|
r2192 | if not m: | ||
break | ||||
start = m.end() | ||||
for id in bugzilla._split_re.split(m.group(1)): | ||||
Matt Mackall
|
r10282 | if not id: | ||
continue | ||||
Benoit Boissinot
|
r8455 | ids.add(int(id)) | ||
Vadim Gelfer
|
r2192 | if ids: | ||
ids = self.filter_real_bug_ids(ids) | ||||
if ids: | ||||
Jim Hague
|
r13798 | ids = self.filter_cset_known_bug_ids(ctx.node(), ids) | ||
Vadim Gelfer
|
r2192 | return ids | ||
Benoit Boissinot
|
r3976 | def update(self, bugid, 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.''' | ||||
count = int(self.ui.config('bugzilla', 'strip', 0)) | ||||
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 | ||||
mapfile = self.ui.config('bugzilla', 'style') | ||||
tmpl = self.ui.config('bugzilla', 'template') | ||||
Matt Mackall
|
r3741 | t = cmdutil.changeset_templater(self.ui, self.repo, | ||
Jim Correia
|
r7762 | False, None, mapfile, False) | ||
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}') | ||||
if tmpl: | ||||
tmpl = templater.parsestring(tmpl, quoted=False) | ||||
t.use_template(tmpl) | ||||
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() | ||
Matt Mackall
|
r5975 | self.add_comment(bugid, data, util.email(ctx.user())) | ||
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: | ||||
raise util.Abort(_('hook type %s does not pass a changeset id') % | ||||
hooktype) | ||||
try: | ||||
bz = bugzilla(ui, repo) | ||||
Matt Mackall
|
r6747 | ctx = repo[node] | ||
Benoit Boissinot
|
r3976 | ids = bz.find_bug_ids(ctx) | ||
Vadim Gelfer
|
r2192 | if ids: | ||
for id in ids: | ||||
Benoit Boissinot
|
r3976 | bz.update(id, ctx) | ||
Jim Hague
|
r7618 | bz.notify(ids, util.email(ctx.user())) | ||
Jim Hague
|
r13800 | except Exception, e: | ||
raise util.Abort(_('Bugzilla error: %s') % e) | ||||
Vadim Gelfer
|
r2192 | |||