carrierwave-aliyun 0.4.3 → 0.4.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 50edfb8bfae43d6c73918be4d2299199740a9571
-  data.tar.gz: bb1a6b302d2bd7d0798ad66986316a6bde633885
+  metadata.gz: 32069f77772a0096a898c9edb0449f2626623911
+  data.tar.gz: 24e3c0c5cda7292b9083443b9c907887b370c01f
 SHA512:
-  metadata.gz: 9cf7e002a1ac03c94d11d5e0634df187c22ff0d5b3e7700b25485225790cc28feae223d7613413a8b1e6b5cb875536a6cfd087fa9fc023a81b7e37b78a352b47
-  data.tar.gz: d0227c03bd3d037bbe81bff512464a1b1e52f30948a722a2fe62a25513fc3414c909c42b736d332aaf1064ab30fe8ba951634c302d562ea68c243c2ad9896a95
+  metadata.gz: 3073b74446c7f29d3e8cd9973f85235f46e3df074a6028d97f5c99bc1ea8672d87589ad0af9232dd4c134511639e0b8a9d02ad6bc2b889cd9b35296d9a12bac2
+  data.tar.gz: dc8d9f2809f437090939a91344b8588d281c513a6910e7b0f52954254ec23d60ef97aa6f995cb2b4acfba1ef03deaeb23681916b92380020a75554825777b243

data/.gitignore

@@ -1,3 +1,9 @@
-uploads
-*.gem
-Gemfile.lock
+.bundle/
+log/*.log
+pkg/
+test/dummy/db/*.sqlite3
+test/dummy/db/*.sqlite3-journal
+test/dummy/log/*.log
+test/dummy/tmp/
+coverage/
+uploads/

data/.travis.yml

@@ -3,6 +3,6 @@ bundler_args: --without development
 
 rvm:
   - 2.3.0
-  - 2.1.0
+  - 2.1.8
 
 script: rspec spec

data/CHANGELOG.md

@@ -1,77 +1,81 @@
+## 0.4.4
+
+- 修正对 Carrierwave master 版本的支持，它们[移除了](https://github.com/carrierwaveuploader/carrierwave/pull/1813) `carrierwave/processing/mime_types`；
+
 ## 0.4.3
 
-* 修正私密空间下载地址算法的问题，导致偶尔会签名错误无法下载的问题；
+- 修正私密空间下载地址算法的问题，导致偶尔会签名错误无法下载的问题；
 
 ## 0.4.2
 
-* `config.aliyun_host` 现在支持配置 //you-host.com，以便同时支持 http 和 https。
+- `config.aliyun_host` 现在支持配置 //you-host.com，以便同时支持 http 和 https。
 
 ## 0.4.1
 
-* 由于 aliyun-oss-sdk 目前不支持 internal 上传，暂时去掉，以免签名错误。
+- 由于 aliyun-oss-sdk 目前不支持 internal 上传，暂时去掉，以免签名错误。
 
 ## 0.4.0
 
-* 采用 aliyun-oss-sdk 来作为上传后端，不再依赖 rest-client，不再内部实现上传逻辑；
-* 增加 `config.aliyun_private_read` 配置项，开启以后，返回的 @user.avatar.url 将会是带 Token 和有效期的 URL，可以用于访问私有读取空间的文件；
-* 去掉 `config.aliyun_upload_host` 配置项，删除了阿里内部的支持，以后请用 0.3.x 版本；
+- 采用 aliyun-oss-sdk 来作为上传后端，不再依赖 rest-client，不再内部实现上传逻辑；
+- 增加 `config.aliyun_private_read` 配置项，开启以后，返回的 @user.avatar.url 将会是带 Token 和有效期的 URL，可以用于访问私有读取空间的文件；
+- 去掉 `config.aliyun_upload_host` 配置项，删除了阿里内部的支持，以后请用 0.3.x 版本；
 
 ## 0.3.6
 
-* 修正上传中文文件名无法成功的问题；
+- 修正上传中文文件名无法成功的问题；
 
 ## 0.3.5
 
-* CarrierWave::Storage::Aliyun::File 继承 CarrierWave::SanitizedFile 以实现一些方法；
+- CarrierWave::Storage::Aliyun::File 继承 CarrierWave::SanitizedFile 以实现一些方法；
 
 ## 0.3.4
 
-* Use OpenSSL::HMAC with Ruby 2.2.0.
+- Use OpenSSL::HMAC with Ruby 2.2.0.
 
 ## 0.3.3
 
-* 增加 `config.aliyun_upload_host`, 以便有需要的时候，可以自由修改上传的 host.
+- 增加 `config.aliyun_upload_host`, 以便有需要的时候，可以自由修改上传的 host.
 
 ## 0.3.2
 
-* 请注意 `config.aliyun_host` 要求修改带 HTTP 协议，以便支持设置 http:// 或 https://.
+- 请注意 `config.aliyun_host` 要求修改带 HTTP 协议，以便支持设置 http:// 或 https://.
 
 ## 0.3.1
 
-* 修复当文件名中包含了 "+"，在 OSS 中上传会遇到签名不对应的问题；
+- 修复当文件名中包含了 "+"，在 OSS 中上传会遇到签名不对应的问题；
 
 ## 0.3.0
 
-* 新增 `aliyun_area` 参数，用于配置 OSS 所在地区数据中心；
+- 新增 `aliyun_area` 参数，用于配置 OSS 所在地区数据中心；
 
 ## 0.2.1
 
-* 避免计算上传文件的时候读取所有内容到内存，之前的做法对于大文件会耗费过多的内存；
-* Carrierwave::Storage::Aliyum::Connection 的 put 方法接口变化，file 现在应该传一个 File 的实例。
+- 避免计算上传文件的时候读取所有内容到内存，之前的做法对于大文件会耗费过多的内存；
+- Carrierwave::Storage::Aliyum::Connection 的 put 方法接口变化，file 现在应该传一个 File 的实例。
 
 ## 0.2.0
 
-* Aliyun OSS 新的[三级域名规则支持](http://bbs.aliyun.com/read.php?tid=139226) by [chaixl](https://github.com/chaixl)
-* 注意! 如果你之前使用 0.1.5 一下的版本，你可能需要调整一下你的自定义域名的 CNAME 解析，阿里云新的 URL 结构变化(少了 Bucket 一层目录)，当然你也可以选择不要升级，之前 0.1.5 版本是稳定的。
+- Aliyun OSS 新的[三级域名规则支持](http://bbs.aliyun.com/read.php?tid=139226) by [chaixl](https://github.com/chaixl)
+- 注意! 如果你之前使用 0.1.5 一下的版本，你可能需要调整一下你的自定义域名的 CNAME 解析，阿里云新的 URL 结构变化(少了 Bucket 一层目录)，当然你也可以选择不要升级，之前 0.1.5 版本是稳定的。
 
 ## 0.1.5
 
-* 自定义域名支持
+- 自定义域名支持
 
 ## 0.1.3
 
-* delete 接口加入。
-* 支持 Carriewave 自动在更新上传文件的时候删除老文件（比如，用户重新上传头像，老头像图片文件将会被 CarrierWave 删除）。
+- delete 接口加入。
+- 支持 Carriewave 自动在更新上传文件的时候删除老文件（比如，用户重新上传头像，老头像图片文件将会被 CarrierWave 删除）。
 
 ## 0.1.2
 
-* 修正 content_type 的支持，自动用原始文件的 content_type，以免上传 zip 之类的文件以后无法下载.
+- 修正 content_type 的支持，自动用原始文件的 content_type，以免上传 zip 之类的文件以后无法下载.
 
 ## 0.1.1
 
-* 修改 Aliyun OSS 的请求地址.
-* 加入可选项，使用 Aliyun 内部地址调用上传，以提高内部网络使用的速度.
+- 修改 Aliyun OSS 的请求地址.
+- 加入可选项，使用 Aliyun 内部地址调用上传，以提高内部网络使用的速度.
 
 ## 0.1.0
 
-* 功能实现.
+- 功能实现.

data/Gemfile.lock

@@ -0,0 +1,151 @@
+PATH
+  remote: .
+  specs:
+    carrierwave-aliyun (0.4.4)
+      aliyun-oss-sdk (>= 0.1.6)
+      carrierwave (>= 0.5.7)
+
+GEM
+  remote: http://gems.ruby-china.org/
+  specs:
+    actionmailer (4.2.0)
+      actionpack (= 4.2.0)
+      actionview (= 4.2.0)
+      activejob (= 4.2.0)
+      mail (~> 2.5, >= 2.5.4)
+      rails-dom-testing (~> 1.0, >= 1.0.5)
+    actionpack (4.2.0)
+      actionview (= 4.2.0)
+      activesupport (= 4.2.0)
+      rack (~> 1.6.0)
+      rack-test (~> 0.6.2)
+      rails-dom-testing (~> 1.0, >= 1.0.5)
+      rails-html-sanitizer (~> 1.0, >= 1.0.1)
+    actionview (4.2.0)
+      activesupport (= 4.2.0)
+      builder (~> 3.1)
+      erubis (~> 2.7.0)
+      rails-dom-testing (~> 1.0, >= 1.0.5)
+      rails-html-sanitizer (~> 1.0, >= 1.0.1)
+    activejob (4.2.0)
+      activesupport (= 4.2.0)
+      globalid (>= 0.3.0)
+    activemodel (4.2.0)
+      activesupport (= 4.2.0)
+      builder (~> 3.1)
+    activerecord (4.2.0)
+      activemodel (= 4.2.0)
+      activesupport (= 4.2.0)
+      arel (~> 6.0)
+    activesupport (4.2.0)
+      i18n (~> 0.7)
+      json (~> 1.7, >= 1.7.7)
+      minitest (~> 5.1)
+      thread_safe (~> 0.3, >= 0.3.4)
+      tzinfo (~> 1.1)
+    addressable (2.4.0)
+    aliyun-oss-sdk (0.1.6)
+      addressable
+      gyoku
+      httparty
+    arel (6.0.3)
+    builder (3.2.2)
+    carrierwave (0.11.2)
+      activemodel (>= 3.2.0)
+      activesupport (>= 3.2.0)
+      json (>= 1.7)
+      mime-types (>= 1.16)
+      mimemagic (>= 0.3.0)
+    concurrent-ruby (1.0.1)
+    diff-lcs (1.2.5)
+    erubis (2.7.0)
+    globalid (0.3.6)
+      activesupport (>= 4.1.0)
+    gyoku (1.3.1)
+      builder (>= 2.1.2)
+    httparty (0.13.7)
+      json (~> 1.8)
+      multi_xml (>= 0.5.2)
+    i18n (0.7.0)
+    json (1.8.3)
+    loofah (2.0.3)
+      nokogiri (>= 1.5.9)
+    mail (2.6.3)
+      mime-types (>= 1.16, < 3)
+    mime-types (2.99.1)
+    mimemagic (0.3.1)
+    mini_magick (4.4.0)
+    mini_portile2 (2.0.0)
+    minitest (5.8.4)
+    multi_xml (0.5.5)
+    nokogiri (1.6.7.2)
+      mini_portile2 (~> 2.0.0.rc2)
+    rack (1.6.4)
+    rack-test (0.6.3)
+      rack (>= 1.0)
+    rails (4.2.0)
+      actionmailer (= 4.2.0)
+      actionpack (= 4.2.0)
+      actionview (= 4.2.0)
+      activejob (= 4.2.0)
+      activemodel (= 4.2.0)
+      activerecord (= 4.2.0)
+      activesupport (= 4.2.0)
+      bundler (>= 1.3.0, < 2.0)
+      railties (= 4.2.0)
+      sprockets-rails
+    rails-deprecated_sanitizer (1.0.3)
+      activesupport (>= 4.2.0.alpha)
+    rails-dom-testing (1.0.7)
+      activesupport (>= 4.2.0.beta, < 5.0)
+      nokogiri (~> 1.6.0)
+      rails-deprecated_sanitizer (>= 1.0.1)
+    rails-html-sanitizer (1.0.3)
+      loofah (~> 2.0)
+    railties (4.2.0)
+      actionpack (= 4.2.0)
+      activesupport (= 4.2.0)
+      rake (>= 0.8.7)
+      thor (>= 0.18.1, < 2.0)
+    rake (10.5.0)
+    rspec (3.4.0)
+      rspec-core (~> 3.4.0)
+      rspec-expectations (~> 3.4.0)
+      rspec-mocks (~> 3.4.0)
+    rspec-core (3.4.3)
+      rspec-support (~> 3.4.0)
+    rspec-expectations (3.4.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.4.0)
+    rspec-mocks (3.4.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.4.0)
+    rspec-support (3.4.1)
+    sprockets (3.5.2)
+      concurrent-ruby (~> 1.0)
+      rack (> 1, < 3)
+    sprockets-rails (3.0.4)
+      actionpack (>= 4.0)
+      activesupport (>= 4.0)
+      sprockets (>= 3.0.0)
+    sqlite3 (1.3.11)
+    sqlite3-ruby (1.3.3)
+      sqlite3 (>= 1.3.3)
+    thor (0.19.1)
+    thread_safe (0.3.5)
+    tzinfo (1.2.2)
+      thread_safe (~> 0.1)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  carrierwave-aliyun!
+  mini_magick
+  rails (= 4.2.0)
+  rake
+  rspec
+  sqlite3-ruby
+
+BUNDLED WITH
+   1.11.2

data/Rakefile

@@ -0,0 +1,7 @@
+begin
+  require 'bundler/setup'
+rescue LoadError
+  puts 'You must `gem install bundler` and `bundle install` to run rake tasks'
+end
+
+Bundler::GemHelper.install_tasks

data/carrierwave-aliyun.gemspec

@@ -4,7 +4,7 @@ require File.expand_path('lib/carrierwave/aliyun/version')
 
 Gem::Specification.new do |s|
   s.name        = "carrierwave-aliyun"
-  s.version     = CarrierWave::Aliyun::Version.current
+  s.version     = CarrierWave::Aliyun::VERSION
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Jason Lee"]
   s.email       = ["huacnlee@gmail.com"]

data/lib/carrierwave-aliyun.rb

@@ -1,11 +1,13 @@
 require 'carrierwave/storage/aliyun'
-require 'carrierwave/processing/mime_types'
 require 'carrierwave/aliyun/configuration'
 
 CarrierWave.configure do |config|
   config.storage_engines.merge!({ aliyun: 'CarrierWave::Storage::Aliyun' })
 end
-
 CarrierWave::Uploader::Base.send(:include, CarrierWave::Aliyun::Configuration)
-CarrierWave::Uploader::Base.send(:include, CarrierWave::MimeTypes)
-CarrierWave::Uploader::Base.send(:process, :set_content_type)
+
+if CarrierWave::VERSION <= '0.11.0'
+  require 'carrierwave/processing/mime_types'
+  CarrierWave::Uploader::Base.send(:include, CarrierWave::MimeTypes)
+  CarrierWave::Uploader::Base.send(:process, :set_content_type)
+end

data/lib/carrierwave/aliyun/version.rb

@@ -1,32 +1,5 @@
 module CarrierWave
   module Aliyun
-    class Version
-      MAJOR, MINOR, PATCH = 0, 4, 3
-
-      ##
-      # Returns the major version ( big release based off of multiple minor releases )
-      def self.major
-        MAJOR
-      end
-
-      ##
-      # Returns the minor version ( small release based off of multiple patches )
-      def self.minor
-        MINOR
-      end
-
-      ##
-      # Returns the patch version ( updates, features and (crucial) bug fixes )
-      def self.patch
-        PATCH
-      end
-
-      ##
-      # Returns the current version of the Backup gem ( qualified for the gemspec )
-      def self.current
-        "#{major}.#{minor}.#{patch}"
-      end
-
-    end
+    VERSION = '0.4.4'
   end
 end

data/lib/carrierwave/storage/aliyun.rb

@@ -1,4 +1,3 @@
-# encoding: utf-8
 require 'aliyun/oss'
 require 'carrierwave'
 require 'uri'
@@ -6,8 +5,9 @@ require 'uri'
 module CarrierWave
   module Storage
     class Aliyun < Abstract
-
       class Connection
+        PATH_PREFIX = %r{^/}
+
         def initialize(uploader)
           @uploader = uploader
           @aliyun_access_id    = uploader.aliyun_access_id
@@ -19,8 +19,8 @@ module CarrierWave
           # Host for get request
           @aliyun_host = uploader.aliyun_host || "http://#{@aliyun_bucket}.oss-#{@aliyun_area}.aliyuncs.com"
 
-          if not @aliyun_host.include?("//")
-            raise "config.aliyun_host requirement include // http:// or https://, but you give: #{@aliyun_host}"
+          unless @aliyun_host.include?('//')
+            fail "config.aliyun_host requirement include // http:// or https://, but you give: #{@aliyun_host}"
           end
         end
 
@@ -32,17 +32,17 @@ module CarrierWave
         #   - content_type - 上传文件的 MimeType，默认 `image/jpg`
         # returns:
         # 图片的下载地址
-        def put(path, file, options={})
-          path.sub!(/^\//, '')
+        def put(path, file, options = {})
+          path.sub!(PATH_PREFIX, '')
           opts = {
-            'Content-Type' => options[:content_type] || "image/jpg"
+            'Content-Type' => options[:content_type] || 'image/jpg'
           }
 
           res = oss_upload_client.bucket_create_object(path, file, opts)
           if res.success?
             path_to_url(path)
           else
-            raise "Put file failed"
+            fail 'Put file failed'
           end
         end
 
@@ -52,12 +52,12 @@ module CarrierWave
         # returns:
         # file data
         def get(path)
-          path.sub!(/^\//, '')
+          path.sub!(PATH_PREFIX, '')
           res = oss_upload_client.bucket_get_object(path)
           if res.success?
             return res.parsed_response
           else
-            raise "Get content faild"
+            fail 'Get content faild'
           end
         end
 
@@ -69,29 +69,30 @@ module CarrierWave
         # returns:
         # 图片的下载地址
         def delete(path)
-          path.sub!(/^\//, '')
+          path.sub!(PATH_PREFIX, '')
           res = oss_upload_client.bucket_delete_object(path)
           if res.success?
             return path_to_url(path)
           else
-            raise "Delete failed"
+            fail 'Delete failed'
           end
         end
 
         ##
         # 根据配置返回完整的上传文件的访问地址
         def path_to_url(path)
-          [@aliyun_host, path].join("/")
+          [@aliyun_host, path].join('/')
         end
 
         # 私有空间访问地址，会带上实时算出的 token 信息
         # 有效期 3600s
         def private_get_url(path)
-          path.sub!(/^\//, '')
+          path.sub!(PATH_PREFIX, '')
           oss_client.bucket_get_object_share_link(path, 3600)
         end
 
         private
+
         def oss_client
           return @oss_client if defined?(@oss_client)
           opts = {
@@ -117,12 +118,6 @@ module CarrierWave
       end
 
       class File < CarrierWave::SanitizedFile
-        def initialize(uploader, base, path)
-          @uploader = uploader
-          @path     = URI.encode(path)
-          @base     = base
-        end
-
         ##
         # Returns the current path/filename of the file on Cloud Files.
         #
@@ -130,8 +125,12 @@ module CarrierWave
         #
         # [String] A path
         #
-        def path
-          @path
+        attr_reader :path
+
+        def initialize(uploader, base, path)
+          @uploader = uploader
+          @path     = URI.encode(path)
+          @base     = base
         end
 
         ##
@@ -151,14 +150,12 @@ module CarrierWave
         # Remove the file from Cloud Files
         #
         def delete
-          begin
-            oss_connection.delete(@path)
-            true
-          rescue Exception => e
-            # If the file's not there, don't panic
-            puts "carrierwave-aliyun delete file failed: #{e}"
-            nil
-          end
+          oss_connection.delete(@path)
+          true
+        rescue => e
+          # If the file's not there, don't panic
+          puts "carrierwave-aliyun delete file failed: #{e}"
+          nil
         end
 
         def url
@@ -183,25 +180,24 @@ module CarrierWave
 
         private
 
-          def headers
-            @headers ||= {}
-          end
-
-          def connection
-            @base.connection
-          end
+        def headers
+          @headers ||= {}
+        end
 
-          def oss_connection
-            return @oss_connection if defined? @oss_connection
+        def connection
+          @base.connection
+        end
 
-            @oss_connection = CarrierWave::Storage::Aliyun::Connection.new(@uploader)
-          end
+        def oss_connection
+          return @oss_connection if defined? @oss_connection
 
+          @oss_connection = CarrierWave::Storage::Aliyun::Connection.new(@uploader)
+        end
       end
 
       def store!(file)
         f = CarrierWave::Storage::Aliyun::File.new(uploader, self, uploader.store_path)
-        f.store(::File.open(file.file), :content_type => file.content_type)
+        f.store(::File.open(file.file), content_type: file.content_type)
         f
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: carrierwave-aliyun
 version: !ruby/object:Gem::Version
-  version: 0.4.3
+  version: 0.4.4
 platform: ruby
 authors:
 - Jason Lee
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-07 00:00:00.000000000 Z
+date: 2016-05-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: carrierwave
@@ -50,7 +50,9 @@ files:
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
+- Gemfile.lock
 - README.md
+- Rakefile
 - carrierwave-aliyun.gemspec
 - lib/carrierwave-aliyun.rb
 - lib/carrierwave/aliyun/configuration.rb