brick_ftp 0.6.1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: d1c8f9d8a8b8938d7081b50d6f5f319799c1a7b8
-  data.tar.gz: 304b3b10bf2f7c50aafe09cbd158784a45d94940
+SHA256:
+  metadata.gz: bff0644f7601a0b205787a383d7e404ea52575f6e10d2e4ec92f2494b0b2e207
+  data.tar.gz: 89b62baafda83fc5a354a3b869bdb9779ec0ff82def20627f40412ef02f2818e
 SHA512:
-  metadata.gz: a0a11120dc634a2b93166bb753291a2b2094fee6701140d6a9be4511dacb318e1b6e0f0d6b6d7a973cb558801047be0ec541db700ca8972f4f709d842ae8bebc
-  data.tar.gz: baeb7298119d8458664ce3c2006ca6481a379fd4992df2baebb8da63d2cb5cf80401433f1289c4b45a6e9f4c606973d7f43a004749880c8f635a14f93db7ed74
+  metadata.gz: '08e939656344e02093ac1df11a0fd9f93194706eac4de105880f90177c7d67876af69cfd42694fe9260e38b4599f89a61ceb6b4ed106ef1d903c68848b20f961'
+  data.tar.gz: f441cbcf2c4e2bb5c64fecb88c2e44ec5bd3293ee6d9fc2bd77b3715635fcb707bd61e3e601e38d048c32d2c6706bcff711d418eab1610c0dbebe7264464c562

data/.circleci/config.yml

@@ -0,0 +1,48 @@
+version: 2
+jobs:
+  build: &build
+    docker:
+      - image: circleci/ruby:latest
+    working_directory: /home/circleci/app
+    steps:
+      - checkout
+      - run:
+          name: Install gems
+          command: bundle install
+      - run:
+          name: Rubocop
+          command: rubocop
+      - run:
+          name: YARDoc
+          command: yardoc --fail-on-warning lib
+      - run:
+          name: RSpec
+          command: rspec
+      - store_artifacts:
+          path: coverage
+          destination: coverage
+  ruby-2.2:
+    <<: *build
+    docker:
+      - image: circleci/ruby:2.2
+  ruby-2.3:
+    <<: *build
+    docker:
+      - image: circleci/ruby:2.3
+  ruby-2.4:
+    <<: *build
+    docker:
+      - image: circleci/ruby:2.4
+  ruby-2.5:
+    <<: *build
+    docker:
+      - image: circleci/ruby:2.5
+
+workflows:
+  version: 2
+  build-using-multi-rubies:
+    jobs:
+      - ruby-2.2
+      - ruby-2.3
+      - ruby-2.4
+      - ruby-2.5

data/.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,31 @@
+<!--
+1. Please check if an issue already exists so there are no duplicates
+2. Check out and follow our Guidelines: https://github.com/serverless/serverless/blob/master/CONTRIBUTING.md
+3. Fill out the whole template so we have a good overview on the issue
+4. Do not remove any section of the template. If something is not applicable leave it empty but leave it in the Issue
+5. Please follow the template, otherwise we'll have to ask you to update it
+-->
+
+# This is a (Bug Report / Feature Proposal)
+
+## Description
+
+For bug reports:
+* What went wrong?
+* What did you expect should have happened?
+* What was the config you used?
+* What stacktrace or error message from your provider did you see?
+
+For feature proposals:
+* What is the use case that should be solved. The more detail you describe this in the easier it is to understand for us.
+* If there is additional config how would it look
+
+Similar or dependent issues:
+* #12345
+
+## Additional Data
+
+* ***Serverless Framework Version you're using***:
+* ***Operating System***:
+* ***Stack Trace***:
+* ***Provider Error messages***:

data/.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,46 @@
+<!--
+1. Please check out and follow our Contributing Guidelines: https://github.com/koshigoe/brick_ftp#contributing
+2. Do not remove any section of the template. If something is not applicable leave it empty but leave it in the PR
+3. Please follow the template, otherwise we'll have to ask you to update it and it will take longer until your PR is merged
+-->
+
+## What did you implement:
+
+Closes #XXXXX
+
+<!--
+Briefly describe the feature if no issue exists for this PR
+-->
+
+## How did you implement it:
+
+<!--
+If this is a nontrivial change please briefly describe your implementation so its easy for us to understand and review your code.
+-->
+
+## How can we verify it:
+
+<!--
+Add any applicable config, commands, screenshots or other resources
+to make it easy for us to verify this works. The easier you make it for us
+to review a PR, the faster we can review and merge it.
+
+Examples:
+* serverless.yml - Fully functioning to easily deploy changes
+* Screenshots - Showing the difference between your output and the master
+* Cloud Configuration - List cloud resources and show that the correct configuration is in place (e.g. AWS CLI commands)
+* Other - Anything else that comes to mind to help us evaluate
+-->
+
+## Todos:
+
+- [ ] Write tests
+- [ ] Write documentation
+- [ ] Fix linting errors
+- [ ] Make sure code coverage hasn't dropped
+- [ ] Provide verification config / commands / resources
+- [ ] Enable "Allow edits from maintainers" for this PR
+- [ ] Update the messages below
+
+***Is this ready for review?:*** NO
+***Is it a breaking change?:*** NO

data/.rubocop.yml

@@ -0,0 +1,24 @@
+inherit_from: .rubocop_todo.yml
+
+AllCops:
+  TargetRubyVersion: 2.2
+
+Style/EmptyCaseCondition:
+  Enabled: false
+
+Style/FormatStringToken:
+  EnforcedStyle: template
+
+Style/TrailingCommaInLiteral:
+  EnforcedStyleForMultiline: consistent_comma
+
+Metrics/LineLength:
+  Max: 124
+  AllowURI: true
+  URISchemes:
+    - http
+    - https
+
+Metrics/BlockLength:
+  Exclude:
+    - 'spec/**/*'

data/.rubocop_todo.yml

@@ -0,0 +1,42 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2018-01-16 23:25:20 +0900 using RuboCop version 0.52.1.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 5
+Metrics/AbcSize:
+  Max: 26
+
+# Offense count: 107
+# Configuration parameters: CountComments, ExcludedMethods.
+Metrics/BlockLength:
+  Max: 446
+
+# Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/ClassLength:
+  Max: 170
+
+# Offense count: 1
+Metrics/CyclomaticComplexity:
+  Max: 7
+
+# Offense count: 3
+# Configuration parameters: CountComments.
+Metrics/MethodLength:
+  Max: 15
+
+# Offense count: 2
+# Configuration parameters: CountKeywordArgs.
+Metrics/ParameterLists:
+  Max: 7
+
+# Offense count: 41
+Style/Documentation:
+  Enabled: false
+
+Style/Lambda:
+  EnforcedStyle: literal

data/CHANGELOG.md

@@ -2,10 +2,10 @@ Changelog
 ====
 
 
-[unreleased](https://github.com/koshigoe/brick_ftp/compare/v0.6.1...master)
+[unreleased](https://github.com/koshigoe/brick_ftp/compare/v0.7.0...master)
 ----
 
-[Full Changelog](https://github.com/koshigoe/brick_ftp/compare/v0.6.1...master)
+[Full Changelog](https://github.com/koshigoe/brick_ftp/compare/v0.7.0...master)
 
 ### Enhancements:
 
@@ -14,6 +14,23 @@ Changelog
 ### Breaking Changes:
 
 
+[v0.7.0](https://github.com/koshigoe/brick_ftp/compare/v0.6.1...v0.7.0)
+----
+
+[Full Changelog](https://github.com/koshigoe/brick_ftp/compare/v0.6.1....v0.7.0)
+
+### Enhancements:
+
+- [#80](https://github.com/koshigoe/brick_ftp/pull/80) Use Rubocop.
+- [#85](https://github.com/koshigoe/brick_ftp/pull/85) Allow `BrickFTP::Client#upload_file` to multi part uploading.
+
+### Fixed Bugs:
+
+### Breaking Changes:
+
+- [#87](https://github.com/koshigoe/brick_ftp/pull/87) Support Ruby >= 2.2.0
+
+
 [v0.6.1](https://github.com/koshigoe/brick_ftp/compare/v0.6.0...v0.6.1)
 ----
 

data/Rakefile

@@ -1,6 +1,8 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
+RuboCop::RakeTask.new
 
-task :default => :spec
+task default: %i[rubocop spec]

data/bin/console

@@ -1,10 +1,10 @@
 #!/usr/bin/env ruby
 
-require "bundler/setup"
-require "brick_ftp"
+require 'bundler/setup'
+require 'brick_ftp'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
 
-require "pry"
+require 'pry'
 Pry.start

data/brick_ftp.gemspec

@@ -1,32 +1,35 @@
-# coding: utf-8
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'brick_ftp/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "brick_ftp"
+  spec.name          = 'brick_ftp'
   spec.version       = BrickFTP::VERSION
-  spec.license       = "MIT"
-  spec.authors       = ["koshigoe"]
-  spec.email         = ["koshigoeb@gmail.com"]
+  spec.license       = 'MIT'
+  spec.authors       = ['koshigoe']
+  spec.email         = ['koshigoeb@gmail.com']
 
-  spec.summary       = %q{BrickFTP's REST API client.}
-  spec.description   = %q{BrickFTP's REST API client.}
-  spec.homepage      = "https://github.com/koshigoe/brick_ftp"
+  spec.summary       = "BrickFTP's REST API client."
+  spec.description   = "BrickFTP's REST API client."
+  spec.homepage      = 'https://github.com/koshigoe/brick_ftp'
 
   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_development_dependency "bundler", "~> 1.12"
-  spec.add_development_dependency "rake", "~> 10.0"
-  spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "webmock", "~> 2.1"
-  spec.add_development_dependency "yard", "~> 0.9"
-  spec.add_development_dependency "pry", "~> 0.10"
+  spec.required_ruby_version = '>= 2.2.0'
+
+  spec.add_development_dependency 'bundler', '~> 1.12'
+  spec.add_development_dependency 'pry', '~> 0.10'
+  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'rubocop', '~> 0.52'
+  spec.add_development_dependency 'simplecov', '~> 0.15'
+  spec.add_development_dependency 'webmock', '~> 2.1'
+  spec.add_development_dependency 'yard', '~> 0.9'
 
   spec.add_dependency 'deep_hash_transform', '~> 1.0'
-  spec.add_dependency 'thor', '~> 0.19.1'
   spec.add_dependency 'inifile', '~> 3.0.0'
+  spec.add_dependency 'thor', '~> 0.19.1'
 end

data/exe/brick_ftp

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 
 root = File.dirname(File.dirname(__FILE__))
-$:.unshift(File.join(root, 'lib'))
+$LOAD_PATH.unshift(File.join(root, 'lib'))
 
 require 'brick_ftp'
 

data/lib/brick_ftp.rb

@@ -1,6 +1,6 @@
 require 'deep_hash_transform'
 require 'thor'
-require "brick_ftp/version"
+require 'brick_ftp/version'
 require 'brick_ftp/configuration'
 require 'brick_ftp/log_formatter'
 require 'brick_ftp/http_client'
@@ -40,10 +40,12 @@ require 'brick_ftp/cli'
 require 'brick_ftp/cli/config'
 require 'brick_ftp/cli/site'
 require 'brick_ftp/cli/main'
+require 'brick_ftp/utils'
+require 'brick_ftp/utils/chunk_io'
 
 module BrickFTP
   # https://brickftp.com/redundancy/
-  IP_ADDRESSES = %w(
+  IP_ADDRESSES = %w[
     54.193.69.72
     54.193.69.200
     54.193.65.189
@@ -72,7 +74,7 @@ module BrickFTP
     52.29.176.178
     54.207.27.239
     52.68.4.44
-  ).freeze
+  ].freeze
 
   # Return configuration.
   # If it has not been configured yet, initialize configuration.

data/lib/brick_ftp/api/base.rb

@@ -67,7 +67,7 @@ module BrickFTP
         BrickFTP::HTTPClient.new.send(
           self.class.endpoints[:delete][:http_method],
           self.class.api_path_for(:delete, self),
-          headers: headers,
+          headers: headers
         )
         true
       end

data/lib/brick_ftp/api/bundle_content.rb

@@ -1,9 +1,9 @@
 module BrickFTP
   module API
     class BundleContent < Base
-      endpoint :post, :index, ->(params) {
+      endpoint :post, :index, ->(params) do
         params.key?(:path) ? '/api/rest/v1/bundles/folders/%{path}' : '/api/rest/v1/bundles/folders'
-      }
+      end
 
       attribute :id
       attribute :path

data/lib/brick_ftp/api/file_operation/upload.rb

@@ -31,17 +31,34 @@ module BrickFTP
         attribute :part, writable: true
         attribute :restart, writable: true
 
-        def self.create(path:, source:)
+        # Upload file.
+        #
+        # @see https://brickftp.com/docs/rest-api/file-uploading/
+        # @param [String] path Remote file path.
+        # @param [IO] source Source `data` (not `path`) to upload.
+        # @param [Integer] chunk_size Size of chunk to multi-part upload.
+        # @return [BrickFTP::API::FileOperation::Upload] An instance of BrickFTP::API::FileOperation::Upload.
+        #
+        def self.create(path:, source:, chunk_size: nil)
           api_client = BrickFTP::HTTPClient.new
-          step1 = api_client.post(api_path_for(:create, path: path), params: { action: 'put' })
+          chunk_io = BrickFTP::Utils::ChunkIO.new(source, chunk_size: chunk_size)
 
-          upload_uri = URI.parse(step1['upload_uri'])
-          upload_client = BrickFTP::HTTPClient.new(upload_uri.host)
-          upload_client.put(step1['upload_uri'], params: source)
+          ref = nil
+          params_for_request_upload_url = { action: 'put' }
+          upload_info = {}
+          chunk_io.each.with_index(1) do |chunk, part|
+            params_for_request_upload_url.update(part: part, ref: ref) if part > 1
+            upload_info = api_client.post(api_path_for(:create, path: path), params: params_for_request_upload_url)
+            ref = upload_info['ref']
 
-          step3 = api_client.post(api_path_for(:create, path: path), params: { action: 'end', ref: step1['ref'] })
+            upload_uri = URI.parse(upload_info['upload_uri'])
+            upload_client = BrickFTP::HTTPClient.new(upload_uri.host)
+            upload_client.put(upload_info['upload_uri'], params: chunk)
+          end
 
-          new(step1.merge(step3).symbolize_keys)
+          uploaded_info = api_client.post(api_path_for(:create, path: path), params: { action: 'end', ref: ref })
+
+          new(upload_info.merge(uploaded_info).symbolize_keys)
         end
       end
     end

data/lib/brick_ftp/api/folder.rb

@@ -1,7 +1,9 @@
 module BrickFTP
   module API
     class Folder < Base
+      # rubocop:disable Metrics/LineLength
       endpoint :get,  :index,  '/api/rest/v1/folders/%{path}', :page, :per_page, :search, :'sort_by[path]', :'sort_by[size]', :'sort_by[modified_at_datetime]'
+      # rubocop:enable Metrics/LineLength
       endpoint :post, :create, '/api/rest/v1/folders/%{path}'
 
       attribute :id

data/lib/brick_ftp/client.rb

@@ -136,7 +136,8 @@ module BrickFTP
     # Show the entire history for the current site.
     # @see https://brickftp.com/ja/docs/rest-api/history/
     # @param page [Integer] Page number of items to return in this request.
-    # @param per_page [Integer] Requested number of items returned per request. Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
+    # @param per_page [Integer] Requested number of items returned per request. Default: 1000, maximum: 10000.
+    #   Leave blank for default (strongly recommended).
     # @param start_at [String] Date and time in the history to start from.
     # @return [Array] array of `BrickFTP::API::History::Site`
     def list_site_history(page: nil, per_page: nil, start_at: nil)
@@ -147,7 +148,8 @@ module BrickFTP
     # Show login history only.
     # @see https://brickftp.com/ja/docs/rest-api/history/
     # @param page [Integer] Page number of items to return in this request.
-    # @param per_page [Integer] Requested number of items returned per request. Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
+    # @param per_page [Integer] Requested number of items returned per request. Default: 1000, maximum: 10000.
+    #   Leave blank for default (strongly recommended).
     # @param start_at [String] Date and time in the history to start from.
     # @return [Array] array of `BrickFTP::API::History::Login`
     def list_login_history(page: nil, per_page: nil, start_at: nil)
@@ -159,7 +161,8 @@ module BrickFTP
     # @see https://brickftp.com/ja/docs/rest-api/history/
     # @param user_id [Integer] User ID.
     # @param page [Integer] Page number of items to return in this request.
-    # @param per_page [Integer] Requested number of items returned per request. Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
+    # @param per_page [Integer] Requested number of items returned per request.
+    #   Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
     # @param start_at [String] Date and time in the history to start from.
     # @return [Array] array of `BrickFTP::API::History::User`
     def list_user_history(user_id:, page: nil, per_page: nil, start_at: nil)
@@ -171,7 +174,8 @@ module BrickFTP
     # @see https://brickftp.com/ja/docs/rest-api/history/
     # @param path [String] path of folder.
     # @param page [Integer] Page number of items to return in this request.
-    # @param per_page [Integer] Requested number of items returned per request. Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
+    # @param per_page [Integer] Requested number of items returned per request.
+    #   Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
     # @param start_at [String] Date and time in the history to start from.
     # @return [Array] array of `BrickFTP::API::History::Folder`
     def list_folder_history(path:, page: nil, per_page: nil, start_at: nil)
@@ -183,7 +187,8 @@ module BrickFTP
     # @see https://brickftp.com/ja/docs/rest-api/history/
     # @param path [String] path of file.
     # @param page [Integer] Page number of items to return in this request.
-    # @param per_page [Integer] Requested number of items returned per request. Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
+    # @param per_page [Integer] Requested number of items returned per request.
+    #   Default: 1000, maximum: 10000. Leave blank for default (strongly recommended).
     # @param start_at [String] Date and time in the history to start from.
     # @return [Array] array of `BrickFTP::API::History::File`
     def list_file_history(path:, page: nil, per_page: nil, start_at: nil)
@@ -291,13 +296,17 @@ module BrickFTP
     # @see https://brickftp.com/ja/docs/rest-api/file-operations/
     # @param path [String]
     # @param page [Integer] Page number of items to return in this request.
-    # @param per_page [Integer] Requested number of items returned per request. Maximum: 5000, leave blank for default (strongly recommended).
+    # @param per_page [Integer] Requested number of items returned per request.
+    #   Maximum: 5000, leave blank for default (strongly recommended).
     # @param search [String] Only return items matching the given search text.
-    # @param sort_by_path [String] Sort by file name, and value is either asc or desc to indicate normal or reverse sort. (Note that sort_by[path] = asc is the default.)
-    # @param sort_by_size [String] Sort by file size, and value is either asc or desc to indicate smaller files first or larger files first, respectively.
-    # @param sort_by_modified_at_datetime [String] Sort by modification time, and value is either asc or desc to indicate older files first or newer files first, respectively.
+    # @param sort_by_path [String] Sort by file name, and value is either asc or desc to indicate normal or reverse sort.
+    #   (Note that sort_by[path] = asc is the default.)
+    # @param sort_by_size [String] Sort by file size, and value is either asc or desc to indicate smaller files
+    #   first or larger files first, respectively.
+    # @param sort_by_modified_at_datetime [String] Sort by modification time, and value is either asc or desc to
+    #   indicate older files first or newer files first, respectively.
     # @return [Array] array of BrickFTP::API::Folder.
-    def list_folders(path:, page: nil, per_page: nil, search: nil, sort_by_path: nil, sort_by_size: nil, sort_by_modified_at_datetime: nil)
+    def list_folders(path:, page: nil, per_page: nil, search: nil, sort_by_path: nil, sort_by_size: nil, sort_by_modified_at_datetime: nil) # rubocop:disable Metrics/LineLength
       query = { path: path, page: page, per_page: per_page, search: search }.reject { |_, v| v.nil? }
       query[:'sort_by[path]'] = sort_by_path if sort_by_path
       query[:'sort_by[size]'] = sort_by_size if sort_by_size
@@ -330,7 +339,7 @@ module BrickFTP
     # @param move_destination [String]
     # @return [BrickFTP::API::FileMove]
     def move_file(path:, move_destination:)
-      BrickFTP::API::FileOperation::Move.create(path: path, :'move-destination' => move_destination)
+      BrickFTP::API::FileOperation::Move.create(path: path, 'move-destination': move_destination)
     end
 
     # Copy a file or folder to the destination provided in the copy_destination parameter.
@@ -339,13 +348,13 @@ module BrickFTP
     # @param copy_destination [String]
     # @return [BrickFTP::API::FileCopy]
     def copy_file(path:, copy_destination:)
-      BrickFTP::API::FileOperation::Copy.create(path: path, :'copy-destination' => copy_destination)
+      BrickFTP::API::FileOperation::Copy.create(path: path, 'copy-destination': copy_destination)
     end
 
     # Delete a file.
     # @see https://brickftp.com/ja/docs/rest-api/file-operations/
     # @param file_or_path [BrickFTP::API::File, String] file object or file(folder) path.
-    # @param recursive: [Boolean]
+    # @param recursive [Boolean]
     # @return [Boolean] return true.
     def delete_file(file_or_path, recursive: false)
       instantize_file(file_or_path).destroy(recursive: recursive)
@@ -355,9 +364,10 @@ module BrickFTP
     # @see https://brickftp.com/ja/docs/rest-api/file-uploading/
     # @param path [String]
     # @param source [IO] source `data` (not `path`) to upload
+    # @param chunk_size [Integer] Size of chunk to multi-part upload.
     # @return [BrickFTP::API::FileUpload]
-    def upload_file(path:, source:)
-      BrickFTP::API::FileOperation::Upload.create(path: path, source: source)
+    def upload_file(path:, source:, chunk_size: nil)
+      BrickFTP::API::FileOperation::Upload.create(path: path, source: source, chunk_size: chunk_size)
     end
 
     # Get usage of site.

data/lib/brick_ftp/configuration.rb

@@ -23,10 +23,10 @@ module BrickFTP
     attr_accessor :logger
     # Log level
     # @return [Integer]
-    attr_accessor :log_level
+    attr_reader :log_level
     # Log formatter
     # @return [Logger::Formatter]
-    attr_accessor :log_formatter
+    attr_reader :log_formatter
     # Open timeout
     # @return [Integer]
     attr_accessor :open_timeout
@@ -39,7 +39,7 @@ module BrickFTP
 
     DEFAULT_PROFILE = 'global'.freeze
     # Name of storable configurations. (TODO: log_path, log_level, log_formatter)
-    STORABLE_CONFIGURATION_KEYS = %w(subdomain api_key open_timeout read_timeout).freeze
+    STORABLE_CONFIGURATION_KEYS = %w[subdomain api_key open_timeout read_timeout].freeze
 
     STORABLE_CONFIGURATION_KEYS.each do |name|
       define_method("#{name}=") do |value|
@@ -104,7 +104,7 @@ module BrickFTP
 
     private
 
-    attr_reader :inifile, :dirty_attributes
+    attr_reader :inifile
 
     def load_config_file(config_file_path)
       @inifile = if config_file_path && File.exist?(config_file_path)

data/lib/brick_ftp/http_client.rb

@@ -8,7 +8,7 @@ module BrickFTP
       def initialize(response)
         begin
           error = JSON.parse(response.body)
-        rescue
+        rescue StandardError
           error = { 'http-code' => response.code, 'error' => "#{response.message}, #{response.body}" }
         end
 
@@ -80,6 +80,8 @@ module BrickFTP
 
     private
 
+    # FIXME: Metrics/AbcSize, Metrics/MethodLength
+    # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
     def request(method, path, params: {}, headers: {})
       req = Net::HTTP.const_get(method.to_s.capitalize).new(path, headers)
       req['User-Agent'] = USER_AGENT
@@ -99,21 +101,28 @@ module BrickFTP
         req.body = params.to_json unless params.empty?
       when params.is_a?(IO)
         req.body_stream = params
-        req["Content-Length"] = params.size
+        req['Content-Length'] = params.size
       end
 
       start = Time.now
       begin
-        logger.debug 'Request headers: %{headers}' % { headers: req.each_capitalized.map { |k, v| "#{k}: #{v}" } }
-        logger.debug 'Request body: %{body}' % { body: req.body }
+        logger.debug format('Request headers: %{headers}', headers: req.each_capitalized.map { |k, v| "#{k}: #{v}" })
+        logger.debug format('Request body: %{body}', body: req.body)
         @conn.request(req).tap do |res|
-          logger.debug 'Response headers: %{headers}' % { headers: res.each_capitalized.map { |k, v| "#{k}: #{v}" } }
-          logger.debug 'Response body: %{body}' % { body: res.body }
+          logger.debug format('Response headers: %{headers}', headers: res.each_capitalized.map { |k, v| "#{k}: #{v}" })
+          logger.debug format('Response body: %{body}', body: res.body)
         end
       ensure
-        logger.info 'Complete %{method} %{path} (%{time} ms)' % { method: method.upcase, path: path, time: (Time.now - start) * 1000 }
+        message = format(
+          'Complete %{method} %{path} (%{time} ms)',
+          method: method.upcase,
+          path: path,
+          time: (Time.now - start) * 1000
+        )
+        logger.info message
       end
     end
+    # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
 
     def logger
       BrickFTP.logger

data/lib/brick_ftp/log_formatter.rb

@@ -2,7 +2,7 @@ module BrickFTP
   class LogFormatter < Logger::Formatter
     FORMAT = "severity:%{severity}\tpid:%{pid}\ttime:%{time}\tmessage:%{message}\n".freeze
 
-    def call(severity, time, program_name, message)
+    def call(severity, time, _program_name, message)
       params = {
         severity: severity,
         pid: Process.pid,

data/lib/brick_ftp/utils.rb

@@ -0,0 +1,4 @@
+module BrickFTP
+  module Utils
+  end
+end

data/lib/brick_ftp/utils/chunk_io.rb

@@ -0,0 +1,59 @@
+require 'tempfile'
+
+module BrickFTP
+  module Utils
+    class ChunkIO
+      include Enumerable
+
+      attr_reader :io, :chunk_size
+
+      # Wrap IO object.
+      #
+      # @param [IO] io an IO object.
+      # @param [Integer] chunk_size Size of chunk.
+      #
+      def initialize(io, chunk_size: nil)
+        @io = io
+        @chunk_size = chunk_size
+      end
+
+      # Iterate with chunked IO object.
+      #
+      # @yield [chunk] Give a chunk IO object to block.
+      # @yieldparam [StringIO] chunk a chunked IO object.
+      #
+      def each(&block)
+        return enum_for(__method__) unless block
+
+        if chunk_size
+          each_chunk(&block)
+        else
+          whole(&block)
+        end
+      end
+
+      private
+
+      def whole
+        yield io
+      end
+
+      def each_chunk
+        eof = false
+        offset = 0
+        until eof
+          Tempfile.create('chunk-io') do |chunk|
+            copied = IO.copy_stream(io, chunk, chunk_size, offset)
+            eof = copied.zero?
+            next if eof
+
+            offset += copied
+            chunk.rewind
+
+            yield chunk
+          end
+        end
+      end
+    end
+  end
+end

data/lib/brick_ftp/version.rb

@@ -1,3 +1,3 @@
 module BrickFTP
-  VERSION = '0.6.1'.freeze
+  VERSION = '0.7.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick_ftp
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.7.0
 platform: ruby
 authors:
 - koshigoe
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-01-10 00:00:00.000000000 Z
+date: 2018-01-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -24,6 +24,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.12'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -53,47 +67,61 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: webmock
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '0.52'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '0.52'
 - !ruby/object:Gem::Dependency
-  name: yard
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.15'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.15'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '2.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '2.1'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
 - !ruby/object:Gem::Dependency
   name: deep_hash_transform
   requirement: !ruby/object:Gem::Requirement
@@ -109,33 +137,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: thor
+  name: inifile
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.19.1
+        version: 3.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.19.1
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
-  name: inifile
+  name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: 0.19.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: 0.19.1
 description: BrickFTP's REST API client.
 email:
 - koshigoeb@gmail.com
@@ -144,9 +172,13 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
+- ".github/ISSUE_TEMPLATE.md"
+- ".github/PULL_REQUEST_TEMPLATE.md"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
+- ".rubocop.yml"
+- ".rubocop_todo.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE.txt
@@ -194,6 +226,8 @@ files:
 - lib/brick_ftp/configuration.rb
 - lib/brick_ftp/http_client.rb
 - lib/brick_ftp/log_formatter.rb
+- lib/brick_ftp/utils.rb
+- lib/brick_ftp/utils/chunk_io.rb
 - lib/brick_ftp/version.rb
 - lib/brick_ftp/webhook.rb
 - lib/brick_ftp/webhook/request.rb
@@ -209,7 +243,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.2.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -217,7 +251,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 2.7.3
 signing_key: 
 specification_version: 4
 summary: BrickFTP's REST API client.

data/.travis.yml

@@ -1,5 +0,0 @@
-sudo: false
-language: ruby
-rvm:
-  - 2.3.1
-before_install: gem install bundler -v 1.12.5