Strip off API end-point file_uri string

Idea taken from #24 (thank you @hickey)

Author: Ilja Bobkevic

lib/puppet_forge_server/backends/directory.rb

@@ -23,6 +23,7 @@ class Directory
 
     # Give highest priority to locally hosted modules
     @@PRIORITY = 0
+    @@FILE_PATH = ''
     attr_reader :PRIORITY
 
     def initialize(url)
@@ -83,7 +84,7 @@ def get_file_metadata(file_name, options)
           file_metadata << {
             :metadata => parse_dependencies(PuppetForgeServer::Models::Metadata.new(normalize_metadata(metadata_raw))),
             :checksum => options[:with_checksum] == true ? Digest::MD5.file(path).hexdigest : nil,
-            :path => "/#{Pathname.new(path).relative_path_from(Pathname.new(@module_dir))}"
+            :path => "/#{Pathname.new(path).relative_path_from(Pathname.new(@module_dir))}".gsub(/^#{@@FILE_PATH}/, '')
           }
         else
           @log.error "Failed reading metadata from #{path}"

lib/puppet_forge_server/backends/proxy.rb

@@ -20,11 +20,12 @@
 module PuppetForgeServer::Backends
   class Proxy
 
-    def initialize(url, cache_dir, http_client)
+    def initialize(url, cache_dir, http_client, file_path)
       @url = url
       @cache_dir = File.join(cache_dir, Digest::SHA1.hexdigest(@url))
       @http_client = http_client
       @log = PuppetForgeServer::Logger.get
+      @file_path = file_path
 
       # Create directory structure for all alphabetic letters
       (10...36).each do |i|
@@ -37,7 +38,7 @@ def get_file_buffer(relative_path)
       File.join(@cache_dir, file_name[0].downcase, file_name)
       path = Dir["#{@cache_dir}/**/#{file_name}"].first
       unless File.exist?("#{path}")
-        buffer = download("/#{relative_path}")
+        buffer = download("#{@file_path.chomp('/')}/#{relative_path}")
         File.open(File.join(@cache_dir, file_name[0].downcase, file_name), 'wb') do |file|
           file.write(buffer.read)
         end

lib/puppet_forge_server/backends/proxy_v1.rb

@@ -22,10 +22,11 @@ class ProxyV1 < PuppetForgeServer::Backends::Proxy
 
     # Priority should be lower than v3 API proxies as v3 requires less API calls
     @@PRIORITY = 15
+    @@FILE_PATH = '/api/v1/files'
     attr_reader :PRIORITY
 
     def initialize(url, cache_dir, http_client = PuppetForgeServer::Http::HttpClient.new)
-      super(url, cache_dir, http_client)
+      super(url, cache_dir, http_client, @@FILE_PATH)
     end
 
     def get_metadata(author, name, options = {})
@@ -78,7 +79,7 @@ def get_module_metadata(modules, options)
           {
               :metadata => parse_dependencies(PuppetForgeServer::Models::Metadata.new(raw_metadata)),
               :checksum => options[:with_checksum] ? Digest::MD5.hexdigest(File.read(get_file_buffer(release['file']))) : nil,
-              :path => "#{release['file']}",
+              :path => "#{release['file']}".gsub(/^#{@@FILE_PATH}/, ''),
               :tags => tags
           }
         end

lib/puppet_forge_server/backends/proxy_v3.rb

@@ -20,10 +20,11 @@ module PuppetForgeServer::Backends
   class ProxyV3 < PuppetForgeServer::Backends::Proxy
 
     @@PRIORITY = 10
+    @@FILE_PATH = '/v3/files'
     attr_reader :PRIORITY
 
     def initialize(url, cache_dir, http_client = PuppetForgeServer::Http::HttpClient.new)
-      super(url, cache_dir, http_client)
+      super(url, cache_dir, http_client, @@FILE_PATH)
     end
 
     def get_metadata(author, name, options = {})
@@ -77,7 +78,7 @@ def get_release_metadata(releases)
         {
             :metadata => parse_dependencies(PuppetForgeServer::Models::Metadata.new(normalize_metadata(element['metadata']))),
             :checksum => element['file_md5'],
-            :path => element['file_uri'],
+            :path => element['file_uri'].gsub(/^#{@@FILE_PATH}/, ''),
             :tags => (element['tags'] + (element['metadata']['tags'] ? element['metadata']['tags'] : [])).flatten.uniq,
             :deleted_at => element['deleted_at']
         }