Sitemaps generating unexpected sitemaps path for AwsSdkAdapter

[jfyzhao]

I'm using the following settings with the AwsSdk Adapter for creating my stiemaps:
SitemapGenerator::Sitemap.default_host = "https://www.mysite.com"
SitemapGenerator::Sitemap.create_index = true
SitemapGenerator::Sitemap.sitemaps_host = "https://s3-us-west-2.amazonaws.com/mysite-assets/"
SitemapGenerator::Sitemap.sitemaps_path = "sitemaps/"
SitemapGenerator::Sitemap.adapter = SitemapGenerator::AwsSdkAdapter.new('mysite-assets')

However, this is generating the sitemaps in https://s3-us-west-2.amazonaws.com/mysite-assets/sitemaps/sitemaps/ so that there is an extra sitemaps/ subdirectory created. I can remove the SitemapGenerator::Sitemap.sitemaps_path = "sitemaps/" line, but that causes the search engines to incorrectly ping for the sitemaps in https://s3-us-west-2.amazonaws.com/mysite-assets/ instead of https://s3-us-west-2.amazonaws.com/mysite-assets/sitemaps/

[kjvarga]

You're right, there's a bug in the AwsSdkAdapter code here https://github.com/kjvarga/sitemap_generator/blob/master/lib/sitemap_generator/adapters/aws_sdk_adapter.rb#L25

Can you try creating your adapter like so:
SitemapGenerator::Sitemap.adapter = SitemapGenerator::AwsSdkAdapter.new('mysite-assets', :path => '')

That should avoid the issue until I can fix it. I didn't actually write that code. But I shouldn't have let a hard-coded path through the cracks :/

[jfyzhao]

Yup that fixed it! No worries, thanks for the suggestion.

[bwoodmansee]

👍 -- thanks for posting this

[kjvarga]

This is fixed in v6.0.0 of the gem, you can remove this workaround.