aliyun-sdk 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8813a13a2575ce30d0853dc4827ec61a10f8093e
-  data.tar.gz: 986f85f79bc3f833dfa1708158cbdf789e3cd7ed
+  metadata.gz: 0a175798738ab2c2b9d5769d915913dbc7e025d4
+  data.tar.gz: af6c15c360539b23b596c93221660d44453d7349
 SHA512:
-  metadata.gz: 8539e0c169ec08a28b4d1dd00cc6d93faba254d4a912d95987457b9dc216353cb922da0ffc04f969af767f44fed44bf27c008509f189c849b31c023af6b1dd87
-  data.tar.gz: 06401e5cd1f01821e9e1c9de8f89871fb76dbcb7aaf7224b4abab7f624675ce59e6414f793e0fed3fc51dcdd724a1ee8515ed90b660425aff52967a54dc2831c
+  metadata.gz: f0ac5dd1199cef681699ef6ba6e7ed24fbc5c4a34e35a056c809a8eadb1cc0b76346b4e206d2cf8b1742d489de206349b6b5d4a5a53c0f0c4da6f05b66ec759f
+  data.tar.gz: 9b33d6da0bb5e48c70a353fa3babb202654d0ea4fd2e498ca16692bf7aad59124693057a5d90623f934f4a8fd7e78bca6aed81c33b91e1dbf27221fbd32c3206

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 ## Change Log
 
+### v0.1.5
+
+- Add open_timeout and read_timeout config
+- Fix a concurrency bug in resumable_upload/download
+
 ### v0.1.4
 
 - Fix object key encoding problem

data/lib/aliyun/oss/bucket.rb

@@ -412,8 +412,8 @@ module Aliyun
       #  义的属性，它们会和object一起存储，在{#get_object}的时候会
       #  返回这些meta。属性的key不区分大小写。例如：{ 'year' => '2015' }
       # @option opts [Integer] :part_size 设置分片上传时每个分片的大小，
-      #  默认为1 MB。断点上传最多允许10000个分片，如果文件大于10000个
-      #  分片的大小，则每个分片的大小会大于1MB。
+      #  默认为10 MB。断点上传最多允许10000个分片，如果文件大于10000个
+      #  分片的大小，则每个分片的大小会大于10MB。
       # @option opts [String] :cpt_file 断点续传的checkpoint文件，如果
       #  指定的cpt文件不存在，则会在file所在目录创建一个默认的cpt文件，
       #  命名方式为：file.cpt，其中file是用户要上传的文件。在上传的过
@@ -459,8 +459,8 @@ module Aliyun
       # @param file [String] 本地文件的路径
       # @param opts [Hash] 下载文件的可选项
       # @option opts [Integer] :part_size 设置分片上传时每个分片的大小，
-      #  默认为1 MB。断点下载最多允许100个分片，如果文件大于100个分片，
-      #  则每个分片的大小会大于1MB
+      #  默认为10 MB。断点下载最多允许100个分片，如果文件大于100个分片，
+      #  则每个分片的大小会大于10 MB
       # @option opts [String] :cpt_file 断点续传的checkpoint文件，如果
       #  指定的cpt文件不存在，则会在file所在目录创建一个默认的cpt文件，
       #  命名方式为：file.cpt，其中file是用户要下载的文件名。在下载的过

data/lib/aliyun/oss/client.rb

@@ -30,6 +30,10 @@ module Aliyun
       #  KEY SECRET，如果不填则会尝试匿名访问
       # @option opts [Boolean] :cname [可选] 指定endpoint是否是用户绑
       #  定的域名
+      # @option opts [Fixnum] :open_timeout [可选] 指定建立连接的超时
+      #  时间，默认为10秒
+      # @option opts [Fixnum] :read_timeout [可选] 指定等待响应的超时
+      #  时间，默认为120秒
       # @example 标准endpoint
       #   oss-cn-hangzhou.aliyuncs.com
       #   oss-cn-beijing.aliyuncs.com

data/lib/aliyun/oss/config.rb

@@ -9,7 +9,8 @@ module Aliyun
     #
     class Config < Struct::Base
 
-      attrs :endpoint, :cname, :access_key_id, :access_key_secret
+      attrs :endpoint, :cname, :access_key_id, :access_key_secret,
+            :open_timeout, :read_timeout
 
       def initialize(opts = {})
         super(opts)

data/lib/aliyun/oss/http.rb

@@ -32,6 +32,8 @@ module Aliyun
     class HTTP
 
       DEFAULT_CONTENT_TYPE = 'application/octet-stream'
+      OPEN_TIMEOUT = 10
+      READ_TIMEOUT = 120
 
       ##
       # A stream implementation
@@ -64,7 +66,13 @@ module Aliyun
             outbuf << ret
           end
 
-          ret.empty? ? nil : ret
+          # Conform to IO#read(length[, outbuf]):
+          # At end of file, it returns nil or "" depend on
+          # length. ios.read() and ios.read(nil) returns
+          # "". ios.read(positive-integer) returns nil.
+          return nil if ret.empty? && !bytes.nil? && bytes > 0
+
+          ret
         end
 
         def write(chunk)
@@ -231,7 +239,9 @@ module Aliyun
           :url => get_request_url(bucket, object),
           :headers => headers,
           :payload => http_options[:body],
-          :block_response =>  block_response
+          :block_response => block_response,
+          :open_timeout => @config.open_timeout || OPEN_TIMEOUT,
+          :timeout => @config.read_timeout || READ_TIMEOUT
         ) do |response, request, result, &blk|
 
           if response.code >= 300

data/lib/aliyun/oss/multipart.rb

@@ -20,16 +20,31 @@ module Aliyun
 
         attrs :id, :object, :bucket, :creation_time, :options
 
+        def initialize(opts = {})
+          super(opts)
+
+          @mutex = Mutex.new
+        end
+
         private
         # Persist transaction states to file
         def write_checkpoint(states, file)
           md5= Util.get_content_md5(states.to_json)
-          File.open(file, 'w') { |f| f.write(states.merge(md5: md5).to_json) }
+
+          @mutex.synchronize {
+            File.open(file, 'w') {
+              |f| f.write(states.merge(md5: md5).to_json)
+            }
+          }
         end
 
         # Load transaction states from file
         def load_checkpoint(file)
-          states = JSON.load(File.read(file))
+          states = {}
+
+          @mutex.synchronize {
+            states = JSON.load(File.read(file))
+          }
           states.symbolize_keys!
           md5 = states.delete(:md5)
 

data/lib/aliyun/oss/upload.rb

@@ -7,7 +7,7 @@ module Aliyun
       # A multipart upload transaction
       #
       class Upload < Transaction
-        PART_SIZE = 4 * 1024 * 1024
+        PART_SIZE = 10 * 1024 * 1024
         READ_SIZE = 16 * 1024
         NUM_THREAD = 10
 

data/lib/aliyun/oss/version.rb

@@ -3,7 +3,7 @@
 module Aliyun
   module OSS
 
-    VERSION = "0.1.4"
+    VERSION = "0.1.5"
 
   end # OSS
 end # Aliyun

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aliyun-sdk
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - Tianlong Wu
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-25 00:00:00.000000000 Z
+date: 2015-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri