comparison contrib/phabricator.py @ 33564:91e3dcefc9b7

phabricator: sanity check Differential Revision from commit message

Previously, we trust Differential Revision in commit message blindly. This
patch adds sanity check so a host name change will be detected and the
commit message will be ignored.

Differential Revision: https://phab.mercurial-scm.org/D35

33563:da94a99df96b

    repo.ui.setconfig('phabricator', 'repophid', repophid)
    return repophid

_differentialrevisiontagre = re.compile('\AD([1-9][0-9]*)\Z')
_differentialrevisiondescre = re.compile(
    '^Differential Revision:.*D([1-9][0-9]*)$', re.M)

def getoldnodedrevmap(repo, nodelist):
    """find previous nodes that has been sent to Phabricator

    return {node: (oldnode or None, Differential Revision ID)}

                    m = _differentialrevisiontagre.match(tag)
                    if m:
                        toconfirm[node] = (n, set(precnodes), int(m.group(1)))
                        continue

        # Check commit message
        m = _differentialrevisiondescre.search(ctx.description())
        if m:
            result[node] = (None, int(m.group(1)))

    # Double check if tags are genuine by collecting all old nodes from
    # Phabricator, and expect precursors overlap with it.
    if toconfirm:
        confirmed = {} # {drev: {oldnode}}

33564:91e3dcefc9b7

    repo.ui.setconfig('phabricator', 'repophid', repophid)
    return repophid

_differentialrevisiontagre = re.compile('\AD([1-9][0-9]*)\Z')
_differentialrevisiondescre = re.compile(
    '^Differential Revision:\s*(.*)D([1-9][0-9]*)$', re.M)

def getoldnodedrevmap(repo, nodelist):
    """find previous nodes that has been sent to Phabricator

    return {node: (oldnode or None, Differential Revision ID)}

                    m = _differentialrevisiontagre.match(tag)
                    if m:
                        toconfirm[node] = (n, set(precnodes), int(m.group(1)))
                        continue

        # Check commit message (make sure URL matches)
        m = _differentialrevisiondescre.search(ctx.description())
        if m:
            if m.group(1).rstrip('/') == url.rstrip('/'):
                result[node] = (None, int(m.group(2)))
            else:
                unfi.ui.warn(_('%s: Differential Revision URL ignored - host '
                               'does not match config\n') % ctx)

    # Double check if tags are genuine by collecting all old nodes from
    # Phabricator, and expect precursors overlap with it.
    if toconfirm:
        confirmed = {} # {drev: {oldnode}}