Patchwork D359: pushvars: add a coreconfigitem for push.pushvars.server

login
register
mail settings
Submitter phabricator
Date Aug. 15, 2017, 9:59 p.m.
Message ID <e201fa08da2af961419944dd43cb6902@localhost.localdomain>
Download mbox | patch
Permalink /patch/23060/
State Not Applicable
Headers show

Comments

phabricator - Aug. 15, 2017, 9:59 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG057d31ceace3: pushvars: add a coreconfigitem for push.pushvars.server (authored by pulkit).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D359?vs=820&id=963

REVISION DETAIL
  https://phab.mercurial-scm.org/D359

AFFECTED FILES
  mercurial/bundle2.py
  mercurial/configitems.py

CHANGE DETAILS




To: pulkit, #hg-reviewers, lothiraldan
Cc: lothiraldan, mercurial-devel

Patch

diff --git a/mercurial/configitems.py b/mercurial/configitems.py
--- a/mercurial/configitems.py
+++ b/mercurial/configitems.py
@@ -361,6 +361,9 @@ 
 coreconfigitem('progress', 'width',
     default=dynamicdefault,
 )
+coreconfigitem('push', 'pushvars.server',
+    default=False,
+)
 coreconfigitem('server', 'bundle1',
     default=True,
 )
diff --git a/mercurial/bundle2.py b/mercurial/bundle2.py
--- a/mercurial/bundle2.py
+++ b/mercurial/bundle2.py
@@ -1883,7 +1883,7 @@ 
 def bundle2getvars(op, part):
     '''unbundle a bundle2 containing shellvars on the server'''
     # An option to disable unbundling on server-side for security reasons
-    if op.ui.configbool('push', 'pushvars.server', False):
+    if op.ui.configbool('push', 'pushvars.server'):
         hookargs = {}
         for key, value in part.advisoryparams:
             key = key.upper()