make sure trailing slashes are chomped from the host. Fixes #31 (#35)

spec/sitemapper_builder_spec.cr

@@ -111,4 +111,16 @@ describe Sitemapper::Builder do
       XML
     end
   end
+
+  describe "#generate_index" do
+    it "trims the trailing slash from the host" do
+      Sitemapper.configure { |c| c.sitemap_host = "https://sitemaps.myapp.com/" }
+      builder = Sitemapper::Builder.new(host: "http://food.com", max_urls: 100, use_index: true)
+      builder.add("/burgers")
+      builder.generate
+      data = builder.generate_index
+      data["data"].should contain("https://sitemaps.myapp.com/sitemap.xml")
+      data["data"].should contain("https://sitemaps.myapp.com/sitemap_index.xml")
+    end
+  end
 end

src/sitemapper/builder.cr

@@ -42,7 +42,7 @@ module Sitemapper
           @sitemaps.each do |sm|
             xml.element("sitemap") do
               sitemap_name = sm["name"].to_s + (Sitemapper.config.compress ? ".gz" : "")
-              sitemap_url = [(Sitemapper.config.sitemap_host || @host), sitemap_name].join('/')
+              sitemap_url = [host_for_sitemap, sitemap_name].join('/')
 
               xml.element("loc") { xml.text sitemap_url }
               xml.element("lastmod") { xml.text Time.utc.to_s("%FT%X%:z") }
@@ -54,6 +54,10 @@ module Sitemapper
       {"name" => filename, "data" => doc}
     end
 
+    private def host_for_sitemap : String
+      (Sitemapper.config.sitemap_host || @host).chomp('/')
+    end
+
     private def build_xml_for_page(items)
       XML.build(indent: " ", version: "1.0", encoding: "UTF-8") do |xml|
         xml.element("urlset", xmlns: XMLNS_SCHEMA, "xmlns:video": XMLNS_VIDEO_SCHEMA, "xmlns:image": XMLNS_IMAGE_SCHEMA, "xmlns:xsi": XMLNS_XSI, "xsi:schemaLocation": XSI_SCHEMA_LOCATION) do