paperclip-storage-aliyun 0.0.3 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 770e2aba77af7013237abcb823fec49c244d15d3
-  data.tar.gz: 199af51f6159ab6b0138eab907b7dd7286b2fd2a
+  metadata.gz: d8531e6601564efe368735f05147b68f4401c495
+  data.tar.gz: e2317360addbbce35acd8bf2947a532cdf11f8c2
 SHA512:
-  metadata.gz: 418741aa19eeb9400e7cae5eb9448c671a99688632cb19a9e7c6c8c43a269297c608c2ef53dc84881ff437833ecdaa826db8f7ece5c0052ac453aa61a9703422
-  data.tar.gz: 27af1cb2c3f924a9f1d736206a57c648d82a3b10b03c3e1ed373dcc8c9d48050f19344c1d7bd22aafe8a021919284c71dbb8fb53170d35142d8a0fddaec3059f
+  metadata.gz: 1f7ce3fda9227a3937a1d04e7acce987c60028187496261c493176f87e016fcaba3083fc7a9977b28af313ac9a92b0ffef6af5ba092a052f40baaac65f7b52e6
+  data.tar.gz: abef51e1bbeb91ac622f74ed61f17bde84026d91eba1ebf423b11833ff770b2a4912d385ec09de6728d16f4d5d0bfafb44517696f0b43e268d4cc6cd3b10753d

data/.editorconfig

@@ -0,0 +1,13 @@
+# EditorConfig is awesome: http://EditorConfig.org
+
+# top-most EditorConfig file
+root = true
+
+# Unix-style newlines with a newline ending every file
+[*]
+charset = utf-8
+indent_style = space
+indent_size = 2
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true

data/.rubocop.yml

@@ -0,0 +1,11 @@
+Style/Documentation:
+  Enabled: false
+
+Metrics/AbcSize:
+  Max: 18
+
+Metrics/LineLength:
+  Max: 120
+
+Metrics/MethodLength:
+  Enabled: false

data/Gemfile

@@ -1,15 +1,16 @@
-source 'http://ruby.taobao.org'
-
-# A database backend that translates database interactions into no-ops. Using
-# NullDB enables you to test your model business logic - including after_save
-# hooks - without ever touching a real database.
-gem "activerecord-nulldb-adapter"
+source 'https://ruby.taobao.org'
 
 gemspec
 
 group :test do
-	gem 'activerecord', '~> 4.0.0'
+  gem 'activerecord', '~> 4.0.0'
+  # A database backend that translates database interactions into no-ops. Using
+  # NullDB enables you to test your model business logic - including after_save
+  # hooks - without ever touching a real database.
+  gem 'activerecord-nulldb-adapter'
   gem 'rspec', '~> 3.3.0'
+  # Use rubocop to lint our Ruby codes
+  gem 'rubocop', '~> 0.34.2'
   gem 'pry'
   gem 'pry-nav'
 end

data/README.md

@@ -21,24 +21,27 @@ Paperclip::Attachment.default_options[:aliyun] = {
   access_id: '3VL9XMho8iCushj8',
   access_key: 'VAUI2q7Tc6yTh1jr3kBsEUzZ84gEa2',
   bucket: 'xx-test',
-  data_centre: 'hangzhou',
+  data_center: 'hangzhou',
   internal: false
 }
 ```
 Then, in the model which defines the attachment, specify your storage and other options, for example:
 ```ruby
 # [rails_root]/app/models/image.rb
-include Paperclip::Storage::Aliyun
-
 class Image < ActiveRecord::Base
   has_attached_file :attachment, {
     storage: :aliyun,
     styles: { thumbnail: "60x60#"},
     path: 'public/system/:class/:attachment/:id_partition/:style/:filename',
-    url: "http://#{oss_connection.fetch_file_host}/public/system/:class/:attachment/:id_partition/:style/:filename"
+    url: ':aliyun_upload_url'
   }
 end
 ```
 
-### Thanks
-这个gem是在参考[Jason Lee](https://github.com/huacnlee)先生写的gem [carrierwave-aliyun](https://github.com/huacnlee/carrierwave-aliyun)的基础上写出来的，其中主要直接用了阿里云接口的代码以及对应的测试代码，在此基础上自行实现Paperclip必要的`get`方法以及`exists?`方法。在此特别感谢**Jason Lee**先生的开源代码。
+Similar to Paperclip::Storage::S3, there are four options for the url by now:
+- `:aliyun_upload_url` : the url based on the options you give
+- `:aliyun_internal_url` : the internal url, no matter what `options[:aliyun][:internal]` is
+- `:aliyun_external_url` : the external url, no matter what `options[:aliyun][:internal]` is
+- `:aliyun_alias_url` : the alias url based on the `host_alias` you give, typically used together with CDN
+
+Please note the values above are all strings, not symbols. You could still make your own url if only you know what you are doing.

data/lib/aliyun/connection.rb

@@ -1,15 +1,23 @@
-# encoding: utf-8
-
 require 'openssl'
 require 'digest/md5'
-require "rest-client"
-require "base64"
+require 'rest-client'
+require 'base64'
 require 'uri'
 require 'aliyun/data_center'
 
 module Aliyun
   class Connection
     include DataCenter
+
+    # The upload host according to the connection configurations
+    attr_reader :aliyun_upload_host
+    # The internal host
+    attr_reader :aliyun_internal_host
+    # The external host
+    attr_reader :aliyun_external_host
+    # The alias host
+    attr_reader :aliyun_alias_host
+
     # Initialize the OSS connection
     #
     # @param [Hash] An options to specify connection details
@@ -18,7 +26,7 @@ module Aliyun
     # @option bucket [String] bucket used to access
     # @option data_center [String] available data center name, e.g. 'hangzhou'
     # @option internal [true, false] if the service should be accessed through internal network
-    # @option host [String] force the host to a given value, only use it when this gem can not work expectly
+    # @option host_alias [String] the alias of the host, such as the CDN domain name
     # @note both access_id and acces_key are related to authorization algorithm:
     #   https://docs.aliyun.com/#/pub/oss/api-reference/access-control&signature-header
     def initialize(options = {})
@@ -26,18 +34,10 @@ module Aliyun
       @aliyun_access_key = options[:access_key]
       @aliyun_bucket = options[:bucket]
 
-      @endpoint = get_endpoint(options)
-
-      @aliyun_upload_host = "#{@aliyun_bucket}.#{@endpoint}"
-
-      @aliyun_host = options[:host] || @aliyun_upload_host
-    end
-
-    # the file host according to the connection configurations
-    #
-    # @return [String] the host value 
-    def fetch_file_host
-      @aliyun_host
+      @aliyun_upload_host = "#{@aliyun_bucket}.#{get_endpoint(options)}"
+      @aliyun_internal_host = "#{@aliyun_bucket}.#{get_endpoint(options.merge(internal: true))}"
+      @aliyun_external_host = "#{@aliyun_bucket}.#{get_endpoint(options.merge(internal: false))}"
+      @aliyun_alias_host = options[:host_alias] || @aliyun_upload_host
     end
 
     # Return the meta informations for the a file specified by the path
@@ -68,29 +68,30 @@ module Aliyun
 
     # Upload File to Aliyun OSS
     # https://docs.aliyun.com/#/pub/oss/api-reference/object&PutObject
-    # 
+    #
     # @param path [String] the target storing path on the oss
     # @param file [File] an instance of File represents a file to be uploaded
     # @param options [Hash]
     #   - content_type - MimeType value for the file, default is "image/jpg"
-    # 
+    #
     # @return [String] The downloadable url of the uploaded file
     # @return [nil] if the uploading failed
-    def put(path, file, options={})
+    def put(path, file, options = {})
       path = format_path(path)
       bucket_path = get_bucket_path(path)
-      content_md5 = Digest::MD5.file(file)
-      content_type = options[:content_type] || "image/jpg"
+      content_md5 = Digest::MD5.file(file).base64digest
+      content_type = options[:content_type] || 'image/jpg'
       date = gmtdate
       url = path_to_url(path)
-      auth_sign = sign("PUT", bucket_path, content_md5, content_type, date)
+      auth_sign = sign('PUT', bucket_path, content_md5, content_type, date)
       headers = {
-        "Authorization" => auth_sign,
-        "Content-Type" => content_type,
-        "Content-Length" => file.size,
-        "Date" => date,
-        "Host" => @aliyun_upload_host,
-        "Expect" => "100-Continue"
+        'Authorization' => auth_sign,
+        'Content-Md5' => content_md5,
+        'Content-Type' => content_type,
+        'Content-Length' => file.size,
+        'Date' => date,
+        'Host' => @aliyun_upload_host,
+        'Expect' => '100-Continue'
       }
       response = RestClient.put(URI.encode(url), file, headers)
       response.code == 200 ? path_to_url(path) : nil
@@ -107,9 +108,9 @@ module Aliyun
       bucket_path = get_bucket_path(path)
       date = gmtdate
       headers = {
-        "Host" => @aliyun_upload_host,
-        "Date" => date,
-        "Authorization" => sign("DELETE", bucket_path, "", "" ,date)
+        'Host' => @aliyun_upload_host,
+        'Date' => date,
+        'Authorization' => sign('DELETE', bucket_path, '', '', date)
       }
       url = path_to_url(path)
       response = RestClient.delete(URI.encode(url), headers)
@@ -126,9 +127,9 @@ module Aliyun
       bucket_path = get_bucket_path(path)
       date = gmtdate
       headers = {
-        "Host" => @aliyun_upload_host,
-        "Date" => date,
-        "Authorization" => sign("GET", bucket_path, "", "" ,date)
+        'Host' => @aliyun_upload_host,
+        'Date' => date,
+        'Authorization' => sign('GET', bucket_path, '', '', date)
       }
       url = path_to_url(path)
       response = RestClient.get(URI.encode(url), headers)
@@ -150,7 +151,7 @@ module Aliyun
     #
     # @return [String] a string represents the formated time, e.g. "Wed, 05 Sep. 2012 23:00:00 GMT"
     def gmtdate
-      Time.now.gmtime.strftime("%a, %d %b %Y %H:%M:%S GMT")
+      Time.now.gmtime.strftime('%a, %d %b %Y %H:%M:%S GMT')
     end
 
     # remove leading slashes in the path
@@ -158,10 +159,7 @@ module Aliyun
     # @param path [String] the path to retrieve the file on remote storage
     # @return [String] the new string after removing leading slashed
     def format_path(path)
-      return "" if path.blank?
-      path.gsub!(/^\/+/,"")
-
-      path
+      path.blank? ? '' : path.gsub(%r{^/+}, '')
     end
 
     # A path consis of the bucket name and file name
@@ -170,7 +168,7 @@ module Aliyun
     # @param path [String] the path to retrieve the file on remote storage
     # @return [String] the expected bucket path, e.g. "test-bucket/oss-api.pdf"
     def get_bucket_path(path)
-      [@aliyun_bucket,path].join("/")
+      [@aliyun_bucket, path].join('/')
     end
 
     # The full path contains host name to the file
@@ -178,11 +176,11 @@ module Aliyun
     # @param path [String] the path to retrieve the file on remote storage
     # @return [String] the expected full path, e.g. "http://martin-test.oss-cn-hangzhou.aliyuncs.com/oss-api.pdf"
     def path_to_url(path)
-      return path if path =~ /^https?:\/{2}/  # 已经是全路径
-      "http://#{fetch_file_host}/#{path}"
+      path =~ %r{^https?://} ? path : "http://#{aliyun_upload_host}/#{path}"
     end
 
     private
+
     # The signature algorithm
     # https://docs.aliyun.com/#/pub/oss/api-reference/access-control&signature-header
     #
@@ -190,14 +188,16 @@ module Aliyun
     # @param content_md5 [String] the md5 value for the content to be uploaded
     # @param content_type [String] the content type of the file, e.g. "application/pdf"
     # @param date [String] the GMT formatted date string
-    def sign(verb, path, content_md5 = '', content_type = '', date)
+    def sign(verb, path, content_md5, content_type, date)
       canonicalized_oss_headers = ''
       canonicalized_resource = "/#{path}"
-      string_to_sign = "#{verb}\n\n#{content_type}\n#{date}\n#{canonicalized_oss_headers}#{canonicalized_resource}"
+      string_to_sign = [
+        verb, content_md5, content_type, date,
+        canonicalized_oss_headers + canonicalized_resource
+      ].join("\n")
       digest = OpenSSL::Digest.new('sha1')
       h = OpenSSL::HMAC.digest(digest, @aliyun_access_key, string_to_sign)
-      h = Base64.encode64(h)
-      "OSS #{@aliyun_access_id}:#{h}"
+      "OSS #{@aliyun_access_id}:#{Base64.encode64(h)}"
     end
   end
 end

data/lib/aliyun/data_center.rb

@@ -1,20 +1,34 @@
 require 'aliyun/errors'
 
 module Aliyun
-	module DataCenter
-		# https://docs.aliyun.com/#/pub/oss/product-documentation/domain-region
-    AVAILABLE_CHINA_DATA_CENTERS = %w(hangzhou qingdao beijing hongkong shenzhen shanghai)
-    AVAILABLE_US_DATA_CENTERS = %w(us-west-1)
+  module DataCenter
+    # https://docs.aliyun.com/#/pub/oss/product-documentation/domain-region
+    AVAILABLE_DATA_CENTERS = %w(
+      oss-cn-hangzhou
+      oss-cn-qingdao
+      oss-cn-beijing
+      oss-cn-hongkong
+      oss-cn-shenzhen
+      oss-cn-shanghai
+      oss-us-west-1
+      oss-ap-southeast-1
+    )
 
     def get_endpoint(options)
-    	data_center = options[:data_center]
-    	if (AVAILABLE_CHINA_DATA_CENTERS + AVAILABLE_US_DATA_CENTERS).exclude?(data_center)
-    		raise InvalildDataCenter, "Unsupported Data Center #{data_center} Detected"
-    	end
+      data_center = find_center(options[:data_center])
 
-    	internal = options[:internal] ? "-internal" : ''
-    	country = AVAILABLE_CHINA_DATA_CENTERS.include?(data_center) ? 'cn' : 'us'
-    	"oss-#{country}-#{data_center}#{internal}.aliyuncs.com"
+      unless data_center && AVAILABLE_DATA_CENTERS.include?(data_center)
+        fail InvalildDataCenter, "Unsupported Data Center #{options[:data_center]} Detected"
+      end
+
+      "#{data_center}#{options[:internal] ? '-internal' : ''}.aliyuncs.com"
+    end
+
+    def find_center(data_center)
+      return if /^(oss|cn|us|ap|oss-cn)$/.match(data_center)
+
+      regexp = Regexp.new(data_center)
+      AVAILABLE_DATA_CENTERS.find { |center| regexp.match(center) }
     end
-	end
-end
+  end
+end

data/lib/aliyun/errors.rb

@@ -1,4 +1,4 @@
 module Aliyun
-	class InvalildDataCenter < StandardError
-	end
-end
+  class InvalildDataCenter < StandardError
+  end
+end

data/lib/paperclip/storage/aliyun.rb

@@ -2,12 +2,38 @@ module Paperclip
   module Storage
     module Aliyun
       def self.extended(base)
+        base.instance_eval do
+          @aliyun_options = @options[:aliyun]
+        end
+
+        [
+          :aliyun_upload_url,   :aliyun_internal_url,
+          :aliyun_external_url, :aliyun_alias_url
+        ].each do |url_style|
+          Paperclip.interpolates(url_style) do |attachment, style|
+            attachment.send(url_style, style)
+          end unless Paperclip::Interpolations.respond_to? url_style
+        end
       end
 
-      def exists?(style = default_style)
-        return false unless path(style)
+      def aliyun_upload_url(style = default_style)
+        "http://#{oss_connection.aliyun_upload_host}/#{path(style).sub(%r{\A/}, '')}"
+      end
+
+      def aliyun_internal_url(style = default_style)
+        "http://#{oss_connection.aliyun_internal_host}/#{path(style).sub(%r{\A/}, '')}"
+      end
 
-        oss_connection.exists? path(style)
+      def aliyun_external_url(style = default_style)
+        "http://#{oss_connection.aliyun_external_host}/#{path(style).sub(%r{\A/}, '')}"
+      end
+
+      def aliyun_alias_url(style = default_style)
+        "http://#{oss_connection.aliyun_alias_host}/#{path(style).sub(%r{\A/}, '')}"
+      end
+
+      def exists?(style = default_style)
+        path(style) ? oss_connection.exists?(path(style)) : false
       end
 
       def flush_writes #:nodoc:
@@ -28,7 +54,7 @@ module Paperclip
         @queued_for_delete = []
       end
 
-      def copy_to_local_file(style = default_style, local_dest_path)
+      def copy_to_local_file(style, local_dest_path)
         log("copying #{path(style)} to local file #{local_dest_path}")
         local_file = ::File.open(local_dest_path, 'wb')
         remote_file_str = oss_connection.get path(style)
@@ -37,9 +63,9 @@ module Paperclip
       end
 
       def oss_connection
-        return @oss_connection if @oss_connection
-
-        @oss_connection ||= ::Aliyun::Connection.new Paperclip::Attachment.default_options[:aliyun]
+        @oss_connection ||= ::Aliyun::Connection.new(
+          Paperclip::Attachment.default_options[:aliyun].merge(@aliyun_options)
+        )
       end
     end
   end

data/paperclip-storage-aliyun.gemspec

@@ -4,9 +4,9 @@ Gem::Specification.new do |s|
   s.require_path = 'lib'
   s.summary      = 'Extend a Aliyun OSS storage for paperclip'
   s.description  = 'Extend a Aliyun OSS storage for paperclip'
-  s.version      = '0.0.3'
+  s.version      = '0.1.0'
   s.files        = `git ls-files`.split("\n")
-  s.authors      = ['Martin Hong']
+  s.authors      = ['Martin Hong', 'Aidi Stan']
   s.email        = 'hongzeqin@gmail.com'
   s.homepage     = 'https://github.com/Martin91/paperclip-storage-aliyun'
   s.license      = 'MIT'

data/spec/aliyun_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-require "net/http"
+require 'net/http'
 
 describe Aliyun::Connection do
   before :all do
@@ -9,7 +9,7 @@ describe Aliyun::Connection do
   end
 
   describe '#initialize' do
-    it "raise error when use invalid data center" do
+    it 'raise error when use invalid data center' do
       expect do
         ::Aliyun::Connection.new data_center: 'guangzhou'
       end.to raise_error(Aliyun::InvalildDataCenter)
@@ -17,16 +17,16 @@ describe Aliyun::Connection do
   end
 
   describe '#put' do
-    it "upload the attachment" do
-      url = @connection.put @path, load_attachment("girl.jpg")
+    it 'upload the attachment' do
+      url = @connection.put @path, load_attachment('girl.jpg')
       response_code = Net::HTTP.get_response(URI.parse(url)).code
-      expect(response_code).to eq("200")
+      expect(response_code).to eq('200')
     end
 
-    it "support setting content type" do
-      content_type = "application/pdf"
+    it 'support setting content type' do
+      content_type = 'application/pdf'
       path = 'pdfs/masu.pdf'
-      @connection.put path, load_attachment("masu.pdf"), content_type: content_type
+      @connection.put path, load_attachment('masu.pdf'), content_type: content_type
       file_meta = @connection.head(path)
       expect(file_meta[:content_type]).to eq(content_type)
 
@@ -35,19 +35,19 @@ describe Aliyun::Connection do
   end
 
   describe '#delete' do
-    it "delete the attachment" do
+    it 'delete the attachment' do
       url = @connection.delete @path
       response_code = Net::HTTP.get_response(URI.parse(url)).code
-      expect(response_code).to eq("404")
+      expect(response_code).to eq('404')
     end
   end
 
-  describe "#exists?" do
+  describe '#exists?' do
     before :all do
-      @connection.put @path, load_attachment("girl.jpg")
+      @connection.put @path, load_attachment('girl.jpg')
     end
 
-    it "return true if the file has been uploaded" do
+    it 'return true if the file has been uploaded' do
       expect(@connection.exists?(@path)).to be_truthy
     end
 

data/spec/fixtures/schema.rb

@@ -1,5 +1,5 @@
-require "active_record"
-require "paperclip"
+require 'active_record'
+require 'paperclip'
 
 ActiveRecord::Schema.define do
   create_table :posts, force: true do |t|

data/spec/lib/paperclip/storage/aliyun_spec.rb

@@ -1,5 +1,5 @@
 require 'spec_helper'
-require "open-uri"
+require 'open-uri'
 require 'net/http'
 require 'support/post'
 
@@ -10,25 +10,22 @@ describe Paperclip::Storage::Aliyun do
   end
 
   after :all do
-    if @post && @post.respond_to?(:id)
-      @post.destroy!
-    end
-
+    @post.destroy! if @post && @post.respond_to?(:id)
     @file.close
   end
 
-  describe "#flush_writes" do
-    it "uploads the attachment to Aliyun" do
+  describe '#flush_writes' do
+    it 'uploads the attachment to Aliyun' do
       response = open(@post.attachment.url)
       expect(response).to be_truthy
     end
 
-    it "get uploaded file from Aliyun" do
+    it 'get uploaded file from Aliyun' do
       attachment = open @post.attachment.url
       expect(attachment.size).to eq(@file.size)
     end
 
-    it "set content type according to the original file" do
+    it 'set content type according to the original file' do
       attachment = load_attachment('masu.pdf')
       post = Post.create attachment: attachment
       headers = RestClient.head(post.attachment.url).headers
@@ -38,8 +35,8 @@ describe Paperclip::Storage::Aliyun do
     end
   end
 
-  describe "#exists?" do
-    it "returns true if the file exists on Aliyun" do
+  describe '#exists?' do
+    it 'returns true if the file exists on Aliyun' do
       expect(@post.attachment).to exist
     end
 
@@ -48,29 +45,29 @@ describe Paperclip::Storage::Aliyun do
       expect(post.attachment).not_to exist
     end
 
-    it "not raise exception when attachment not saved" do
+    it 'not raise exception when attachment not saved' do
       post = Post.create
-      expect{post.attachment.exists?}.not_to raise_error
+      expect { post.attachment.exists? }.not_to raise_error
     end
   end
 
-  describe "#copy_to_local_file" do
-    it "copies file from Aliyun to a local file" do
-      destination = File.join(Bundler.root, "tmp/photo.jpg")
+  describe '#copy_to_local_file' do
+    it 'copies file from Aliyun to a local file' do
+      destination = File.join(Bundler.root, 'tmp/photo.jpg')
       @post.attachment.copy_to_local_file(:original, destination)
-      expect(File.exists?(destination)).to be_truthy
-      
+      expect(File.exist?(destination)).to be_truthy
+
       File.delete destination
     end
   end
 
-  describe "#flush_deletes" do
-    it "deletes the attachment from Aliyun" do
+  describe '#flush_deletes' do
+    it 'deletes the attachment from Aliyun' do
       attachment_url = @post.attachment.url
       @post.destroy
 
       response_code = Net::HTTP.get_response(URI.parse(attachment_url)).code
-      expect(response_code).to eq("404")
+      expect(response_code).to eq('404')
     end
   end
 end

data/spec/spec_helper.rb

@@ -2,28 +2,26 @@ require 'pry'
 require 'pry-nav'
 require 'paperclip-storage-aliyun'
 
-Dir[File.join(Bundler.root, "spec/support/**/*.rb")].each &method(:require)
-
-include Paperclip::Storage::Aliyun
-def file_host
-  oss_connection.fetch_file_host
-end
+Paperclip.logger.level = ::Logger::UNKNOWN
+Dir[Bundler.root.join('spec/support/**/*.rb')].each(&method(:require))
 
+# Aliyun defaults
 OSS_CONNECTION_OPTIONS = {
   access_id: '3VL9XMho8iCuslj8',
   access_key: 'VAUI2q7Tc6yTf1jr3kBsEUzZ84gEa2',
   bucket: 'martin-test',
   data_center: 'hangzhou',
   internal: false
-  # host: nil
+  # host_alias: nil
 }
 
-# paperclip初始化设置
+# Paperclip defaults
 Paperclip::Attachment.default_options[:storage] = :aliyun
-Paperclip::Attachment.default_options[:path] = 'public/system/:class/:attachment/:id_partition/:style/:filename'
 Paperclip::Attachment.default_options[:aliyun] = OSS_CONNECTION_OPTIONS
-Paperclip::Attachment.default_options[:url] = "http://#{file_host}/public/system/:class/:attachment/:id_partition/:style/:filename"
+Paperclip::Attachment.default_options[:path] = 'public/system/:class/:attachment/:id_partition/:style/:filename'
+Paperclip::Attachment.default_options[:url] = ':aliyun_upload_url'
 
+# Utility methods
 def load_attachment(file_name)
-  File.open (File.expand_path "attachments/#{file_name}", File.dirname(__FILE__)), 'rb'
+  File.open(Bundler.root.join("spec/attachments/#{file_name}"), 'rb')
 end

data/spec/support/post.rb

@@ -1,6 +1,6 @@
-require "active_record"
-require "nulldb"
-require "paperclip"
+require 'active_record'
+require 'nulldb'
+require 'paperclip'
 
 class Post < ActiveRecord::Base
   include Paperclip::Glue
@@ -11,10 +11,10 @@ end
 
 RSpec.configure do |config|
   config.before(:all) do
-    FileUtils.mkdir_p File.join(Bundler.root, "tmp")
+    FileUtils.mkdir_p File.join(Bundler.root, 'tmp')
     ActiveRecord::Base.establish_connection(
-      adapter: "nulldb",
-      schema: File.join(Bundler.root, "spec/fixtures/schema.rb"),
+      adapter: 'nulldb',
+      schema: File.join(Bundler.root, 'spec/fixtures/schema.rb')
     )
   end
 

data/spec/support/rails.rb

@@ -1,4 +1,4 @@
-require "paperclip"
+require 'paperclip'
 
 module Paperclip
   module Interpolations

metadata

@@ -1,14 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: paperclip-storage-aliyun
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors:
 - Martin Hong
+- Aidi Stan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-02 00:00:00.000000000 Z
+date: 2015-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: paperclip
@@ -44,8 +45,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".editorconfig"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - Gemfile
 - README.md
 - lib/aliyun/connection.rb