[MediaWiki-commits] [Gerrit] mediawiki/vagrant[master]: Enable https instead of http for phabricator

2017-09-26 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/377051 )

Change subject: Enable https instead of http for phabricator
..


Enable https instead of http for phabricator

cloud supports https and http, lets use https.

Change-Id: I193223059e60c76b9596d94ba52cedf2afa4a6dd
---
M puppet/modules/phabricator/manifests/init.pp
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  BryanDavis: Looks good to me, approved
  jenkins-bot: Verified
  Dzahn: Looks good to me, but someone else must approve



diff --git a/puppet/modules/phabricator/manifests/init.pp 
b/puppet/modules/phabricator/manifests/init.pp
index bfd3aa7..b582050 100644
--- a/puppet/modules/phabricator/manifests/init.pp
+++ b/puppet/modules/phabricator/manifests/init.pp
@@ -29,6 +29,7 @@
 $vhost_name,
 $remote,
 $branch = undef,
+$protocol = 'http',
 ){
 require ::arcanist
 include ::apache
@@ -82,7 +83,7 @@
 }
 
 phabricator::config { 'phabricator.base-uri':
-value => "http://${vhost_name}${::port_fragment}/";,
+value => "${protocol}://${vhost_name}${::port_fragment}/",
 }
 
 phabricator::config { 'pygments.enabled':

-- 
To view, visit https://gerrit.wikimedia.org/r/377051
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I193223059e60c76b9596d94ba52cedf2afa4a6dd
Gerrit-PatchSet: 7
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Paladox 
Gerrit-Reviewer: BryanDavis 
Gerrit-Reviewer: Dduvall 
Gerrit-Reviewer: Dzahn 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Paladox 
Gerrit-Reviewer: Reedy 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits


[MediaWiki-commits] [Gerrit] mediawiki/vagrant[master]: Enable https instead of http for phabricator

2017-09-10 Thread Paladox (Code Review)
Paladox has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/377051 )

Change subject: Enable https instead of http for phabricator
..

Enable https instead of http for phabricator

cloud supports https and http, lets use https.

Change-Id: I193223059e60c76b9596d94ba52cedf2afa4a6dd
---
0 files changed, 0 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/vagrant 
refs/changes/51/377051/1


-- 
To view, visit https://gerrit.wikimedia.org/r/377051
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I193223059e60c76b9596d94ba52cedf2afa4a6dd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Paladox 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits