Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/adblockplus/manifests/web/fileserver.pp

Issue 29795567: #11371 - Password protection for fileserver repositories (Closed)
Patch Set: Uploaded a file by mistake Created June 1, 2018, 5:02 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: modules/adblockplus/manifests/web/fileserver.pp
===================================================================
--- a/modules/adblockplus/manifests/web/fileserver.pp
+++ b/modules/adblockplus/manifests/web/fileserver.pp
@@ -4,10 +4,6 @@
#
# === Parameters:
#
-# [*domain*]
-# A string which is the name of the fileserver domain, under which
-# each repository has a subdomain.
-#
# [*certificate*]
# The name of the SSL certificate file within modules/private/files, if any.
# Requires a private_key as well.
@@ -16,16 +12,12 @@
# The name of the private key file within modules/private/files, if any.
# Requires a certificate as well.
#
-# [*is_default*]
-# Passed on to nginx (whether or not the site config should be default).
-#
# [*repositories*]
# A collection (hash) of repositories to serve.
# The contents of a repository is served on a subdomain of the fileserver.
#
class adblockplus::web::fileserver(
$ensure = 'present',
- $domain,
$certificate = undef,
$private_key = undef,
$repositories={},
@@ -44,13 +36,5 @@
ensure_resources('adblockplus::web::fileserver::repository', $repositories, {
ensure => 'present',
})
-
- nginx::hostconfig{ "$domain":
- source => 'puppet:///modules/adblockplus/nginx/fileserver.conf',
- is_default => true,
- certificate => $certificate,
- private_key => $private_key,
- log => 'access_log_fileserver',
- }
}

Powered by Google App Engine
This is Rietveld