Add url helper to internal route links

Author: Chadwick Banning

lib/puppet_forge_server/app/frontend.rb

@@ -48,7 +48,7 @@ def initialize(root, http_client = PuppetForgeServer::Http::HttpClient.new)
       halt(400, haml(:security, :locals => {:query => query})) \
         unless safe_input? query
 
-      modules = get("#{request.base_url}/v3/modules?query=#{query}")['results']
+      modules = get("/v3/modules?query=#{query}")['results']
       haml :modules, :locals => {:query => query, :modules => modules}
     end
 
@@ -57,20 +57,20 @@ def initialize(root, http_client = PuppetForgeServer::Http::HttpClient.new)
       halt(400, haml(:security, :locals => {:query => module_v3_name})) \
         unless safe_input? module_v3_name
 
-      releases = get("#{request.base_url}/v3/modules/#{module_v3_name}")['releases']
+      releases = get("/v3/modules/#{module_v3_name}")['releases']
       if params.has_key? 'version'
         module_uri = releases.find {|r| r['version'] == params['version']}['uri']
-        module_metadata = get("#{request.base_url}#{module_uri}")
+        module_metadata = get("#{module_uri}")
       else
-        module_metadata = get("#{request.base_url}#{releases[0]['uri']}")
+        module_metadata = get("#{releases[0]['uri']}")
       end
       begin
         readme_markdown = markdown(module_metadata['readme'])
       rescue
         readme_markdown = ''
       end
       haml :module, :locals => { :module_metadata => module_metadata,
-                                 :base_url => request.base_url,
+                                 :base_url => url(request.base_url),
                                  :readme_markdown => readme_markdown,
                                  :releases => releases }
     end
@@ -88,7 +88,7 @@ def initialize(root, http_client = PuppetForgeServer::Http::HttpClient.new)
     private
     def get(relative_url)
       begin
-        JSON.parse(@http_client.get(relative_url))
+        JSON.parse(@http_client.get(url(relative_url)))
       rescue
         {'results' => []}
       end

lib/puppet_forge_server/app/views/layout.haml

@@ -21,7 +21,7 @@
     %title Internal Puppet Forge
     -# Latest compiled and minified CSS
     %link{ :href => 'https://maxcdn.bootstrapcdn.com/bootstrap/3.3.2/css/bootstrap.min.css', :type => 'text/css', :rel => 'stylesheet' }
-    %link{ :href => 'css/puppetlabs.css', :type => 'text/css', :rel => 'stylesheet' }
+    %link{ :href => url('css/puppetlabs.css'), :type => 'text/css', :rel => 'stylesheet' }
     -#%link{ :href => 'https://maxcdn.bootstrapcdn.com/bootstrap/3.3.2/css/bootstrap-theme.min.css', :type => 'text/css', :rel => 'stylesheet' }
     /[if lt IE 9]
       %script{:src => 'https://oss.maxcdn.com/libs/html5shiv/3.7.0/html5shiv.js'}
@@ -49,7 +49,7 @@
       .side-width.clearfix
         %a{:href => url('/')}
           #logo
-            %img.logo{ :src => 'img/forge-logo.png', :height => 65}
+            %img.logo{ :src => url('img/forge-logo.png'), :height => 65}
             %span.logo.puppet= 'Puppet Forge'
             %span.logo.forge= 'Server'
         .top-search
@@ -61,4 +61,4 @@
       .side-width.clearfix
         = yield
     %script{ :src => 'https://maxcdn.bootstrapcdn.com/bootstrap/3.3.2/js/bootstrap.min.js' }
-    %script{ :src => 'js/internal.js' }
+    %script{ :src => url('js/internal.js') }

lib/puppet_forge_server/app/views/module.haml

@@ -60,8 +60,8 @@
                 %label= 'Select another release: '
                 %select{:id => 'module-release-selector', :onchange => "location = this.options[this.selectedIndex].value;"}
                   - releases.each do |release|
-                    %option{:value => "/module?name=#{module_metadata['metadata']['name']}&version=#{release['version']}", :selected => module_metadata['metadata']['version'] == release['version']}= release['version']
-                %a{:class => 'module-release-download', :href => module_metadata['file_uri'], :title => 'Download module'}= 'Download'
+                    %option{:value => url("/module?name=#{module_metadata['metadata']['name']}&version=#{release['version']}"), :selected => module_metadata['metadata']['version'] == release['version']}= release['version']
+                %a{:class => 'module-release-download', :href => url(module_metadata['file_uri']), :title => 'Download module'}= 'Download'
             %div{:id => 'release-info-tabs', :class => 'module-meta-info'}
               %div{:class => 'tabbed content'}
                 %div