fog-aliyun 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f7c3f28cba2b2acb17e34f8f00a8ed5577bf6c1e
-  data.tar.gz: 465522578a4c15f6a1c4b043ee48f89c62dd78bd
+  metadata.gz: cffd78ea3a1c223ae9e0b45b3ec996c3e8baa55d
+  data.tar.gz: 54de6ae02cece427a30c90aeb3fc98b055a59cbb
 SHA512:
-  metadata.gz: fb70b9d6a7b355632beb07a470c782e2ce53dffd93a9668446648053d24b57b95cd88d5ec262d7dfe34b23be4ed335e08627d9ec9c90292c1dd37b9bd791d71a
-  data.tar.gz: 3f2257c4ecbf8842c547e446d3841c048a7ad057c52c39dd1924c6a52f3f754a9edfe2cc373ca5b2ecdcbbe83d92948e932237c57c91aabd030888e6848e6685
+  metadata.gz: ca783c81990b44dbd811460e10a72bdc749e0bb5c862fc29ab1a84e6c1403e4e9c34f8f33ec23d3c5cac1e044016cacdaa388ff51092f2566a5fcf383d26521c
+  data.tar.gz: b6a0937a7e9bb735638645e2b040c0ddb6b660d81b8703eab3288e442ffdb39e8cbed73e582ac8bee5a08af1c337d38ef937a4103e160cc42f746e7fcee92f25

data/README.md

@@ -32,9 +32,7 @@ Since it's a bad practice to have your credentials in source code, you should lo
 default:
   :aliyun_accesskey_id:     <YOUR_ACCESS_KEY_ID>,
   :aliyun_accesskey_secret: <YOUR_SECRET_ACCESS_KEY>,
-  :aliyun_oss_endpoint:     <YOUR_OSS_ENDPOINT>,
-  :aliyun_oss_location:     <YOUR_OSS_LOACTION>,
-  :aliyun_oss_bucket:       <YOUR_OSS_BUCKET>
+  :aliyun_region_id:        <YOUR_TARGET_REGION>
 ```
 
 ### Connecting to OSS
@@ -48,12 +46,17 @@ opt = {
   :provider                => 'aliyun',
   :aliyun_accesskey_id     => <YOUR_ACCESS_KEY_ID>,
   :aliyun_accesskey_secret => <YOUR_SECRET_ACCESS_KEY>,
-  :aliyun_oss_endpoint     => <YOUR_OSS_ENDPOINT>,
-  :aliyun_oss_location     => <YOUR_OSS_LOACTION>,
   :aliyun_oss_bucket       => <YOUR_OSS_BUCKET>,
+  :aliyun_region_id        => <YOUR_TARGET_REGION>,
+  :aliyun_oss_endpoint     => <YOUR_OSS_ENDPOINT>,
 }
 conn = Fog::Storage.new(opt)
 ```
+**-> Note:** `:aliyun_region_id` is optional and  default to "cn-hangzhou".
+**-> Note:** `:aliyun_oss_endpoint` is optional. If it is not specified, it will be generated automatically by `:aliyun_region_id`.
+Its basic format is "oss-<region-id>.aliyuncs.com" and with default schema "http" and default port "80".
+If you want to use https or 443 port, you can use a format "<schema>://oss-<region-id>.aliyuncs.com:<port>".
+
 
 ## Fog::Aliyun Abstractions
 

data/lib/fog/aliyun/models/storage/files.rb

@@ -66,6 +66,18 @@ module Fog
                    else
                      directory.key + '/' + key
                    end
+          begin
+          data = service.get_object(object)
+          rescue => error
+            case error.response.body
+              when /<Code>NoSuchKey<\/Code>/
+                nil
+              else
+                raise(error)
+            end
+          end
+
+          contentLen = data[:headers]['Content-Length'].to_i
 
           if block_given?
             pagesNum = (contentLen + Excon::CHUNK_SIZE - 1) / Excon::CHUNK_SIZE
@@ -80,12 +92,9 @@ module Fog
               body = nil
             end
           else
-            data = service.get_object(object)
             body = data[:body]
           end
 
-          contentLen = data[:headers]['Content-Length'].to_i
-          return nil if data[:status] != 200
           lastModified = data[:headers]['Last-Modified']
           last_modified = if !lastModified.nil? && lastModified != ''
                             Time.parse(lastModified).localtime

data/lib/fog/aliyun/storage.rb

@@ -3,11 +3,21 @@ require 'xmlsimple'
 module Fog
   module Storage
     class Aliyun < Fog::Service
+
+      DEFAULT_REGION = 'cn-hangzhou'
+
+      DEFAULT_SCHEME = 'http'
+      DEFAULT_SCHEME_PORT = {
+          'http' => 80,
+          'https' => 443
+      }
+
       recognizes :aliyun_oss_endpoint,
                  :aliyun_oss_location,
-                 :aliyun_oss_bucket
+                 :aliyun_region_id
       requires   :aliyun_accesskey_id,
-                 :aliyun_accesskey_secret
+                 :aliyun_accesskey_secret,
+                 :aliyun_oss_bucket
 
       model_path 'fog/aliyun/models/storage'
       model       :directory
@@ -37,9 +47,9 @@ module Fog
         # Initialize connection to OSS
         #
         # ==== Notes
-        # options parameter must include values for :aliyun_oss_endpoint, :aliyun_accesskey_id,
-        # :aliyun_secret_access_key, :aliyun_oss_location and :aliyun_oss_bucket in order to create a connection.
-        # if you haven't set these values in the configuration file.
+        # options parameter must include values for :aliyun_accesskey_id, :aliyun_secret_access_key and :aliyun_oss_bucket in order to create a connection.
+        # :aliyun_oss_location will be replaced by :aliyun_region_id, and it has a default value cn-hangzhou
+        # if :aliyun_oss_endpoint is not specified, it will be generated by method region_to_endpoint
         #
         # ==== Examples
         #   sdb = Fog::Storage.new(:provider=>'aliyun',
@@ -55,32 +65,37 @@ module Fog
         attr_reader :aliyun_accesskey_id
         attr_reader :aliyun_accesskey_secret
         attr_reader :aliyun_oss_endpoint
-        attr_reader :aliyun_oss_location
+        attr_reader :aliyun_region_id
         attr_reader :aliyun_oss_bucket
 
         def initialize(options = {})
           # initialize the parameters
-          @aliyun_oss_endpoint     = options[:aliyun_oss_endpoint]
-          @aliyun_oss_location     = options[:aliyun_oss_location]
+          @aliyun_region_id        = options[:aliyun_region_id] || options[:aliyun_oss_location] || DEFAULT_REGION
+          @aliyun_oss_endpoint     = options[:aliyun_oss_endpoint] || region_to_endpoint(@aliyun_region_id)
           @aliyun_accesskey_id     = options[:aliyun_accesskey_id]
           @aliyun_accesskey_secret = options[:aliyun_accesskey_secret]
           @aliyun_oss_bucket       = options[:aliyun_oss_bucket]
 
           # check for the parameters
           missing_credentials = []
-          missing_credentials << :aliyun_oss_endpoint unless @aliyun_oss_endpoint
-          missing_credentials << :aliyun_oss_location unless @aliyun_oss_location
+          missing_credentials << :aliyun_oss_bucket unless @aliyun_oss_bucket
           missing_credentials << :aliyun_accesskey_id unless @aliyun_accesskey_id
           missing_credentials << :aliyun_accesskey_secret unless @aliyun_accesskey_secret
           raise ArgumentError, "Missing required arguments: #{missing_credentials.join(', ')}" unless missing_credentials.empty?
 
           @connection_options = options[:connection_options] || {}
 
+          endpoint = @aliyun_oss_endpoint
+
+          if !endpoint.start_with?(DEFAULT_SCHEME)
+            @aliyun_oss_endpoint = "#{DEFAULT_SCHEME}://#{endpoint}"
+          end
+
           uri = URI.parse(@aliyun_oss_endpoint)
           @host   = uri.host
           @path   = uri.path
-          @port   = uri.port
-          @scheme = uri.scheme
+          @scheme = uri.scheme || DEFAULT_SCHEME
+          @port   = uri.port || DEFAULT_SCHEME_PORT[@scheme]
 
           @persistent = options[:persistent] || false
         end
@@ -89,6 +104,15 @@ module Fog
           @connection.reset
         end
 
+        def region_to_endpoint(region=nil)
+          case region.to_s
+            when ''
+              "oss-#{DEFAULT_REGION}.aliyuncs.com"
+            else
+              "oss-#{region}.aliyuncs.com"
+          end
+        end
+
         def request(params)
           method = params[:method]
           time = Time.new.utc
@@ -181,14 +205,14 @@ module Fog
       class Mock
         def initialize(options = {})
           @aliyun_oss_endpoint     = options[:aliyun_oss_endpoint]
-          @aliyun_oss_location     = options[:aliyun_oss_location]
+          @aliyun_region_id        = options[:aliyun_region_id]
           @aliyun_accesskey_id     = options[:aliyun_accesskey_id]
           @aliyun_accesskey_secret = options[:aliyun_accesskey_secret]
           @aliyun_oss_bucket       = options[:aliyun_oss_bucket]
 
           # missing_credentials = Array.new
           # missing_credentials << :aliyun_oss_endpoint unless @aliyun_oss_endpoint
-          # missing_credentials << :aliyun_oss_location unless @aliyun_oss_location
+          # missing_credentials << :aliyun_region_id unless @aliyun_region_id
           # missing_credentials << :aliyun_accesskey_id  unless @aliyun_accesskey_id
           # missing_credentials << :aliyun_accesskey_secret unless @aliyun_accesskey_secret
           # raise ArgumentError, "Missing required arguments: #{missing_credentials.join(', ')}" unless missing_credentials.empty?

data/lib/fog/aliyun/version.rb

@@ -1,5 +1,5 @@
 module Fog
   module Aliyun
-    VERSION = '0.2.1'.freeze
+    VERSION = '0.2.2'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fog-aliyun
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Qinsi Deng, Jianxun Li, Jane Han
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-04-13 00:00:00.000000000 Z
+date: 2018-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -304,7 +304,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 2.5.2.1
 signing_key: 
 specification_version: 4
 summary: Fog provider for Aliyun Web Services.