s3_website 1.6.11 → 1.6.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: dc35704ef355b35bcead7f859c794b5108aaad94
-  data.tar.gz: 2a61a4dddec1f8308df6292d01c254cdb713ee18
+  metadata.gz: 44b560c700f0298aa650b5558d4e98adbff4835b
+  data.tar.gz: 252e0d11c51dcf9c750aa6d101497ef806583d73
 SHA512:
-  metadata.gz: b7af6c832b233c9a817fd3994d54664d549e660fbafba2584a6058113a06018fce38d076ec8aedc09f1821ec01aee535279d5b6597d70cb4352bef02c5f30038
-  data.tar.gz: 115f4e95a4f801032900b8a97742441e69551f1eb8cce7fa849a108688b245474c4ad1a53d1530888182de08e98d35626f8c2ca62daf1bff1e9e85e5f65ff549
+  metadata.gz: 5d80cbb3d5bd1d0ad83f056816dfc73b24850ecb37dfe95364ab450cd25706e6aee58fa95c66030392f3bbfb39df64cb2c30098e16df13812a1fca09dc7ed358
+  data.tar.gz: 9eb07f87dab4c5a7b7510533dc9b72385bcd47ccdfd41a53b4315a13ccab6860e92bda7e35cb0a0770bc08923c7775092ac8cc3f26c026142292bd608dc21ffd

data/changelog.md

@@ -2,6 +2,10 @@
 
 This project uses [Semantic Versioning](http://semver.org).
 
+## 1.6.12
+
+* Fix bug <https://github.com/laurilehmijoki/s3_website/issues/63>
+
 ## 1.6.11
 
 * Loosen the dependency spec of mime-types (#70)

data/lib/cloudfront/invalidator.rb

@@ -9,13 +9,20 @@ module S3Website
         s3_object_keys << ""
         report = SimpleCloudfrontInvalidator::CloudfrontClient.new(
           aws_key, aws_secret, cloudfront_distribution_id
-        ).invalidate(s3_object_keys)
+        ).invalidate(url_encode_keys s3_object_keys)
         puts report[:text_report]
         report[:invalidated_items_count]
       end
 
       private
 
+      def self.url_encode_keys(keys)
+        require 'uri'
+        keys.map do |key|
+          URI::encode(key, Regexp.union([URI::Parser.new.regexp[:UNSAFE],'~','@', "'"]))
+        end
+      end
+
       def self.apply_config(config, changed_files)
         if config['cloudfront_invalidate_root']
           changed_files.map { |changed_file|

data/s3_website.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "s3_website"
-  s.version     = "1.6.11"
+  s.version     = "1.6.12"
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Lauri Lehmijoki"]
   s.email       = ["lauri.lehmijoki@iki.fi"]

data/spec/lib/cloudfront/invalidator_spec.rb

@@ -1,13 +1,13 @@
 require 'spec_helper'
 
 describe S3Website::Cloudfront::Invalidator do
-  describe 'default behaviour' do
-    let(:config) {{
-      's3_id' => 'aws id',
-      's3_secret' => 'aws secret',
-      'cloudfront_distribution_id' => 'EFXX'
-    }}
+  let(:config) {{
+    's3_id' => 'aws id',
+    's3_secret' => 'aws secret',
+    'cloudfront_distribution_id' => 'EFXX'
+  }}
 
+  describe 'default behaviour' do
     it 'invalidates the root resource' do
       invalidator = create_simple_cloudfront_invalidator(config)
       invalidator.
@@ -19,22 +19,33 @@ describe S3Website::Cloudfront::Invalidator do
     end
   end
 
-  describe 'option cloudfront_invalidate_root = true' do
-    let(:config) {{
-      's3_id' => 'aws id',
-      's3_secret' => 'aws secret',
-      'cloudfront_distribution_id' => 'EFXX',
-      'cloudfront_invalidate_root' => true
-    }}
+  context 'option cloudfront_invalidate_root = true' do
+    let(:config_with_root_invalidation) {
+      config.merge( {
+        'cloudfront_invalidate_root' => true
+      })
+    }
 
     it 'invalidates all root resources' do
-      invalidator = create_simple_cloudfront_invalidator(config)
+      invalidator = create_simple_cloudfront_invalidator(config_with_root_invalidation)
       invalidator.
         should_receive(:invalidate).
         with(['article/', '']).
         and_return(:text_report => 'report txt')
 
-      S3Website::Cloudfront::Invalidator.invalidate(config, ['article/index.html'])
+      S3Website::Cloudfront::Invalidator.invalidate(config_with_root_invalidation, ['article/index.html'])
+    end
+  end
+
+  context 'the file name contains special characters' do
+    it 'encodes the file paths according to rfc1738' do
+      invalidator = create_simple_cloudfront_invalidator config
+      invalidator.
+        should_receive(:invalidate).
+        with(['article/arnold%27s%20file.html', '']).
+        and_return(:text_report => 'report txt')
+
+      S3Website::Cloudfront::Invalidator.invalidate(config, ["article/arnold's file.html"])
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: s3_website
 version: !ruby/object:Gem::Version
-  version: 1.6.11
+  version: 1.6.12
 platform: ruby
 authors:
 - Lauri Lehmijoki
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-25 00:00:00.000000000 Z
+date: 2014-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk