sitemap_generator 7.0.1 → 7.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b26cd553fe4ecc452da26ac6aff3262238f065dc12d9f328959eb3c1edc79f6e
-  data.tar.gz: 877828e867880ed215df8b3e1f6acb130e4e8c5856a349e61e6af17a5c641ac9
+  metadata.gz: 76768fc3383e6ec38752dd3373955432282793e507cc4683cb8ed0f8676d9861
+  data.tar.gz: a40040f7f4d6e7bf362c962bf243285aa6b72d3f20faef9ac031350d1369a9cc
 SHA512:
-  metadata.gz: d957d69f7a4c2fe8e98cecb84045d58fe59f9f8143764c88695a7561a63d54e9d515c80bdc2b2cf496b1704ffb2ca4610b54f5889cb246e2ed45aac88029abdd
-  data.tar.gz: 884fc7be3809f6670cce09afdddfa64cdf6b4c5b25cdc7ad5c7c3b784d08f83bbc96bdb671a1b52de7f0f59fac8618f1bc83122c01664ee574a52c4edd791699
+  metadata.gz: 7252276ae78c1dc17ca8f9ec6605d29ffe8f9cc8aceb03c63240bc50a3f333b79da197465ca0632a8c10de1c02fcf290ff745dc6496bbdb19a5afb3f0e84c65e
+  data.tar.gz: a04d88fc32364ca8eb84e6d33ea960da53c21260ce65e2ac3ab907fcc196a685691ea84416e82813dfc03cb53d84440f9046cd364ffbc63d6fa0baf24ff2d0f1

data/CHANGES.md

@@ -1,3 +1,13 @@
+### 7.0.3
+
+* **Regression fix:** Revert railtie enhancements (#478) that were unintentionally included in 7.0.2. Those changes introduced an `ArgumentError: Missing host to link to!` during `assets:precompile` and other Rails boot contexts where no host is configured (reported in [#488](https://github.com/kjvarga/sitemap_generator/issues/488)). The railtie is restored to its 7.0.1 behaviour: it loads rake tasks only. The enhancements will return in a future release with proper documentation and test coverage. [#489](https://github.com/kjvarga/sitemap_generator/pull/489)
+* **Bugfix:** Restore sitemap generation on Ruby 2.x when `--enable=frozen-string-literal` is set globally. Ruby 2.6/2.7 freezes interpolated heredocs under that flag, causing `gsub!` to raise `FrozenError`. [#486](https://github.com/kjvarga/sitemap_generator/pull/486)
+* Internal: skip duplicate git tag creation in the release task if the tag already points at HEAD.
+
+### 7.0.2
+
+* Reduce string copies and improve compatibility with `frozen_string_literal`. [#456](https://github.com/kjvarga/sitemap_generator/pull/456)
+
 ### 7.0.1
 
 * **Breaking:** Default search engines list is empty. `rake sitemap:refresh` and `ping_search_engines` perform no HTTP pings unless you configure engine URLs on `search_engines` or pass them into `ping_search_engines` (Google’s ping endpoint is deprecated upstream). [#444](https://github.com/kjvarga/sitemap_generator/pull/444)

data/VERSION

@@ -1 +1 @@
-7.0.1
+7.0.3

data/lib/sitemap_generator/adapters/aws_sdk_adapter.rb

@@ -44,12 +44,25 @@ module SitemapGenerator
     # Call with a SitemapLocation and string data
     def write(location, raw_data)
       SitemapGenerator::FileAdapter.new.write(location, raw_data)
-      s3_object = s3_resource.bucket(@bucket).object(location.path_in_public)
-      s3_object.upload_file(location.path, {
-        acl: @acl,
-        cache_control: @cache_control,
-        content_type: location[:compress] ? 'application/x-gzip' : 'application/xml'
-      }.compact)
+
+      if defined?(Aws::S3::TransferManager)
+        client = Aws::S3::Client.new(@options)
+        transfer_manager = Aws::S3::TransferManager.new(client: client)
+        transfer_manager.upload_file(location.path,
+          bucket: @bucket,
+          key: location.path_in_public,
+          acl: @acl,
+          cache_control: @cache_control,
+          content_type: location[:compress] ? 'application/x-gzip' : 'application/xml'
+        )
+      else
+        s3_object = s3_resource.bucket(@bucket).object(location.path_in_public)
+        s3_object.upload_file(location.path, {
+          acl: @acl,
+          cache_control: @cache_control,
+          content_type: location[:compress] ? 'application/x-gzip' : 'application/xml'
+        }.compact)
+      end
     end
 
     private

data/lib/sitemap_generator/builder/sitemap_file.rb

@@ -26,7 +26,9 @@ module SitemapGenerator
         @location = opts.is_a?(Hash) ? SitemapGenerator::SitemapLocation.new(opts) : opts
         @link_count = 0
         @news_count = 0
-        @xml_content = +'' # XML urlset content
+        # The + prefix makes this string explicitly mutable. Ruby 2.x with
+        # --enable=frozen-string-literal freezes interpolated heredocs, and
+        # gsub! below requires a mutable string.
         @xml_wrapper_start = +<<-HTML
           <?xml version="1.0" encoding="UTF-8"?>
             <urlset
@@ -45,6 +47,7 @@ module SitemapGenerator
         @xml_wrapper_start.gsub!(/\s+/, ' ').gsub!(/ *> */, '>').strip!
         @xml_wrapper_end = '</urlset>'
         @filesize = SitemapGenerator::Utilities.bytesize(@xml_wrapper_start) + SitemapGenerator::Utilities.bytesize(@xml_wrapper_end)
+        @xml_content = @xml_wrapper_start
         @written = false
         @reserved_name = nil # holds the name reserved from the namer
         @frozen = false      # rather than actually freeze, use this boolean
@@ -141,8 +144,9 @@ module SitemapGenerator
 
         finalize! unless finalized?
         reserve_name
-        @location.write(@xml_wrapper_start + @xml_content + @xml_wrapper_end, link_count)
-        @xml_content = @xml_wrapper_start = @xml_wrapper_end = ''
+        @xml_content << @xml_wrapper_end
+        @location.write(@xml_content, link_count)
+        @xml_content = @xml_wrapper_end = ''
         @written = true
       end
 

data/lib/sitemap_generator/builder/sitemap_index_file.rb

@@ -12,7 +12,9 @@ module SitemapGenerator
         @location = opts.is_a?(Hash) ? SitemapGenerator::SitemapIndexLocation.new(opts) : opts
         @link_count = 0
         @sitemaps_link_count = 0
-        @xml_content = +'' # XML urlset content
+        # The + prefix makes this string explicitly mutable. Ruby 2.x with
+        # --enable=frozen-string-literal freezes interpolated heredocs, and
+        # gsub! below requires a mutable string.
         @xml_wrapper_start = +<<-HTML
           <?xml version="1.0" encoding="UTF-8"?>
             <sitemapindex
@@ -25,6 +27,7 @@ module SitemapGenerator
         @xml_wrapper_start.gsub!(/\s+/, ' ').gsub!(/ *> */, '>').strip!
         @xml_wrapper_end = '</sitemapindex>'
         @filesize = SitemapGenerator::Utilities.bytesize(@xml_wrapper_start) + SitemapGenerator::Utilities.bytesize(@xml_wrapper_end)
+        @xml_content = @xml_wrapper_start
         @written = false
         @reserved_name = nil # holds the name reserved from the namer
         @frozen = false      # rather than actually freeze, use this boolean

data/lib/sitemap_generator/tasks.rb

@@ -10,46 +10,32 @@ namespace :sitemap do
   # Require sitemap_generator only.  When installed as a plugin the require will fail, so in
   # that case, load the environment first.
   task :require do
-    begin
-      require 'sitemap_generator'
-    rescue LoadError => e
-      if defined?(Rails::VERSION)
-        Rake::Task['sitemap:require_environment'].invoke
-      else
-        raise e
-      end
-    end
-  end
-
-  # Require sitemap_generator after loading the Rails environment.  We still need the require
-  # in case we are installed as a gem and are setup to not automatically be required.
-  task :require_environment do
-    if defined?(Rails::VERSION)
-      Rake::Task['environment'].invoke
-    end
     require 'sitemap_generator'
   end
+  # In a Rails app, we need to boot Rails.
+  # Ensure gem is required in case it wasn't automatically loaded.
+  Rake::Task[:require].enhance([:environment]) if defined?(Rails)
 
   desc 'Install a default config/sitemap.rb file'
-  task install: ['sitemap:require'] do
+  task install: :require do
     SitemapGenerator::Utilities.install_sitemap_rb(verbose)
   end
 
   desc 'Delete all Sitemap files in public/ directory'
-  task clean: ['sitemap:require'] do
+  task clean: :require do
     SitemapGenerator::Utilities.clean_files
   end
 
   desc 'Generate sitemaps and ping search engines.'
-  task refresh: ['sitemap:create'] do
+  task refresh: :create do
     SitemapGenerator::Sitemap.ping_search_engines
   end
 
   desc "Generate sitemaps but don't ping search engines."
-  task 'refresh:no_ping' => ['sitemap:create']
+  task 'refresh:no_ping' => :create
 
   desc "Generate sitemaps but don't ping search engines.  Alias for refresh:no_ping."
-  task create: ['sitemap:require_environment'] do
+  task create: :require do
     SitemapGenerator::Interpreter.run(config_file: ENV['CONFIG_FILE'], verbose: verbose)
   end
 end

data/lib/sitemap_generator/utilities.rb

@@ -142,9 +142,9 @@ module SitemapGenerator
     end
 
     def titleize(string)
-      string = string.dup if string.frozen?
-      string.gsub!(/_/, ' ')
-      string.split(/(\W)/).map(&:capitalize).join
+      result = string.tr('_', ' ')
+      result.gsub!(/\b\w/) { |match| match.upcase }
+      result
     end
 
     def truthy?(value)

data/lib/sitemap_generator.rb

@@ -40,18 +40,21 @@ module SitemapGenerator
     }
 
     # Lazy-initialize the LinkSet instance
-    Sitemap = (Class.new do
-      def method_missing(*args, &block)
-        (@link_set ||= reset!).send(*args, &block)
+    Sitemap = (Config = Class.new do
+      # Use a new LinkSet instance
+      def reset!
+        @link_set = LinkSet.new
       end
 
-      def respond_to?(name, include_private = false)
-        (@link_set ||= reset!).respond_to?(name, include_private)
+      private
+
+      def method_missing(name, *args, &block)
+        @link_set ||= reset!
+        @link_set.respond_to?(name, true) ? @link_set.__send__(name, *args, &block) : super
       end
 
-      # Use a new LinkSet instance
-      def reset!
-        @link_set = LinkSet.new
+      def respond_to_missing?(name, include_private = false)
+        (@link_set ||= reset!).respond_to?(name, include_private) || super
       end
     end).new
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sitemap_generator
 version: !ruby/object:Gem::Version
-  version: 7.0.1
+  version: 7.0.3
 platform: ruby
 authors:
 - Karl Varga
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-04-28 00:00:00.000000000 Z
+date: 2026-07-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: builder
@@ -82,7 +82,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="