From 18678263751560a0fb304478059cc2af27c24aa0 Mon Sep 17 00:00:00 2001
From: Bikramjit Singh <bikramjit@live.ca>
Date: Wed, 1 May 2024 09:42:58 -0400
Subject: [PATCH] Make source URL configurable (#668)

* Add source_url attribute to be able to fetch source from local proxy repo
* Remove dead links

---------
Signed-off-by: Bikramjit Singh <bikramjiit.singh@utoronto.ca>
Co-authored-by: Bikramjit Singh <bikramjiit.singh@utoronto.ca>
---
 README.md                | 3 ++-
 attributes/default.rb    | 1 +
 libraries/helpers.rb     | 4 ----
 recipes/server_source.rb | 2 +-
 4 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index 28285c7c..806ba7cb 100644
--- a/README.md
+++ b/README.md
@@ -86,6 +86,7 @@ Example: `default['nagios']['conf']['cfg_dir'] = [ '/etc/nagios/conf.d' , '/usr/
 
 - `node['nagios']['server']['version']` - version of the server source to download
 - `node['nagios']['server']['checksum']` - checksum of the source files
+- `node['nagios']['server']['source_url']` - nagios core source url
 - `node['nagios']['server']['patch_url']` - url to download patches from if installing from source
 - `node['nagios']['server']['patches']` - array of patch filenames to apply if installing from source
 - `node['nagios']['url']` - URL to host Nagios from - defaults to nil and instead uses  FQDN
@@ -157,7 +158,7 @@ Example: `default['nagios']['conf']['cfg_dir'] = [ '/etc/nagios/conf.d' , '/usr/
 - `node['nagios']['server']['web_server']` - web server to use. supports Apache or Nginx, default "apache"
 - `node['nagios']['server']['nginx_dispatch']` - nginx dispatch method. supports cgi or php, default "cgi"
 - `node['nagios']['server']['stop_apache']` - stop apache service if using nginx, default false
-- `node['nagios']['server']['redirect_root']` - if using Apache, should [http://server/](http://server/) redirect to [http://server/nagios3](http://server/nagios3) automatically, default false
+- `node['nagios']['server']['redirect_root']` - if using Apache, should `http://server/` redirect to `http://server/nagios3` automatically, default false
 - `node['nagios']['server']['normalize_hostname']` - If set to true, normalize all hostnames in hosts.cfg to lowercase. Defaults to false.
 
  These are nagios cgi.config options.
diff --git a/attributes/default.rb b/attributes/default.rb
index 1940b66f..a7df9d7c 100644
--- a/attributes/default.rb
+++ b/attributes/default.rb
@@ -84,6 +84,7 @@
 # for server from source installation
 default['nagios']['server']['version']   = '4.4.6'
 default['nagios']['server']['checksum']  = 'ab0d5a52caf01e6f4dcd84252c4eb5df5a24f90bb7f951f03875eef54f5ab0f4'
+default['nagios']['server']['source_url'] = "https://assets.nagios.com/downloads/nagioscore/releases/nagios-#{node['nagios']['server']['version']}.tar.gz"
 default['nagios']['server']['patches']   = []
 default['nagios']['server']['patch_url'] = nil
 default['nagios']['server']['dependencies'] = nagios_server_dependencies
diff --git a/libraries/helpers.rb b/libraries/helpers.rb
index 26bc0a82..fc47cc77 100644
--- a/libraries/helpers.rb
+++ b/libraries/helpers.rb
@@ -218,10 +218,6 @@ def nagios_install_method
       end
     end
 
-    def nagios_source_url
-      "https://assets.nagios.com/downloads/nagioscore/releases/nagios-#{node['nagios']['server']['version']}.tar.gz"
-    end
-
     def nagios_pagerduty_packages
       case node['platform_family']
       when 'rhel'
diff --git a/recipes/server_source.rb b/recipes/server_source.rb
index 6986b829..304deccc 100644
--- a/recipes/server_source.rb
+++ b/recipes/server_source.rb
@@ -56,7 +56,7 @@
 
 remote_file 'nagios source file' do
   path ::File.join(Chef::Config[:file_cache_path], "nagios-#{nagios_version}.tar.gz")
-  source nagios_source_url
+  source node['nagios']['server']['source_url']
   checksum node['nagios']['server']['checksum']
   notifies :run, 'execute[compile-nagios]', :immediately
 end