osdn-cli 0.1.9 → 0.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 711d2bde0ec7b6b4a631200c6eab8d7b719a8798
-  data.tar.gz: 54f1852e3d5f9deff21a080e377ebfad19410579
+SHA256:
+  metadata.gz: c9458c3957544b3d0486db50098cfb327870d0fc4a93be500699328756ffb8b4
+  data.tar.gz: c60b73446c9feb4216f74e86feb080b74dabfe966d640d70287a53a5e8a1348f
 SHA512:
-  metadata.gz: d82079daa70bccb7f8e8aefea07aeeea5a7896c6bec160f249f7ea0d977c72e3bccb2b49d6a2a0323baf5b25ca24ae0f8414a05ea46832a01861b1ab752059dc
-  data.tar.gz: 1376e42c75a7fb842951dc0cd11a089994566ac52a88c3a9ccd6d47e350911dd0809bcc4db237a5e928996c3a1c111d0948bf3c7b38c73d4deafa7596f92440a
+  metadata.gz: 752467db05db7fcc5685e7f7488f5760181aca25fb88f5ec1835dfff107f7700de30b12050c1331a3cca851ee542dfc1440a455a05d72b032a3bb410ee1df308
+  data.tar.gz: 64e8c11ee3a26cfa04e6f98150a195dda1a3cd45fcfb3716457d8cb636f6f2d0dd4ba33641af511815b1181dea352d792c6a98fcdae080f340684653167561b5

data/ChangeLog

@@ -1,3 +1,8 @@
+2018-08-20  Tatsuki Sugiura  <sugi@nemui.org>
+
+	* Fix to compat with API change; check file size instead of digest on upload
+	* Version 0.1.10
+
 2017-05-01  Tatsuki Sugiura  <sugi@nemui.org>
 
 	* Try to fix launching browser on login at Windows environment.

data/lib/osdn/cli.rb

@@ -165,6 +165,13 @@ module OSDN
               logger.warn "Failed to load variables from #{path}; #{e.message}"
             end
           end
+          # Fix key conflicts
+          if vars['local_file_info']
+            vars['local_file_info'].each do |fname, finfo|
+              finfo.has_key?('size') or next
+              finfo['filesize'] = finfo.delete('size')
+            end
+          end
           logger.debug "Variables: #{vars.inspect}"
           Hashie::Mash.new(vars)
         end

data/lib/osdn/cli/command/frs_upload.rb

@@ -1,5 +1,7 @@
 module OSDN; module CLI; module Command
   class FrsUpload < Base
+    attr_accessor :target_proj, :visibility, :force_digest, :dry_run, :show_progress
+
     def help
       puts "#{$0} frs_upload [opts] [target_dir]"
       puts "Options:"
@@ -164,67 +166,31 @@ module OSDN; module CLI; module Command
         return false
       end
 
-      vars = load_variables(rdir)
-      digests = nil
-      if !@force_digest && vars.local_file_info &&
-         vars.local_file_info[file.basename.to_s]
-        finfo = vars.local_file_info[file.basename.to_s]
-        if finfo[:size] == file.size && finfo.mtime == file.mtime
-          digests = vars.local_file_info[file.basename.to_s].digests
-        end
+      filecmd = Relfile.new logger
+      [:target_proj, :visibility, :force_digest, :show_progress].each do |opt|
+        filecmd.send "#{opt}=", send(opt)
       end
+      filecmd.target_package = target_package(rdir)
+      filecmd.target_release = target_release(rdir)
+      filecmd.calc_file_digest(file)
+
+      vars = load_variables(rdir)
+      digests = vars.local_file_info[file.basename.to_s].digests
 
-      unless digests
-        logger.info "Calculating digest for #{file}..."
-        digests = {
-          sha256: hexdigest(Digest::SHA256, file),
-          sha1:   hexdigest(Digest::SHA1, file),
-          md5:    hexdigest(Digest::MD5, file),
-        }
-        update_variables rdir, {local_file_info: {file.basename.to_s => {digests: digests, mtime: file.mtime, size: file.size}}}
-      end
       if remote_f = rinfo.files.find { |f| f.name == file.basename.to_s }
-        if digests.find { |type, dig| dig != remote_f.send("digest_#{type}") }
+        if remote_f.size != file.size || digests.find { |type, dig| rd = remote_f.send("digest_#{type}"); rd && rd != '' && dig != rd }
           logger.error "#{file} was changed from remote file! Please delete remote file before uploading new one."
         end
         logger.info "Skip already uploaded file '#{file}'"
         return
       end
 
+      finfo = {}
       logger.info "Uploading file #{file} (#{file.size} bytes)"
       if @dry_run
         finfo = Hashie::Mash.new id: '(dry-run)', url: '(dry-run)'
       else
-        logger.level <= Logger::INFO && @show_progress != false || @show_progress and
-          OSDN::CLI._show_progress = true
-        fio = file.open
-        logger.info "Starting upload #{file}..."
-        max_upload_tries = 5
-        upload_tries = 0
-        begin
-          upload_tries += 1
-          finfo = api.create_release_file target_proj, target_package(rdir), target_release(rdir), fio, visibility: @visibility
-        rescue OSDNClient::ApiError => e
-          if max_upload_tries - upload_tries <= 0 
-            logger.error "Max upload attempts (#{max_upload_tries}) has been exceeded, give up!"
-            raise e
-          elsif [0, 100, 502].member?(e.code.to_i)
-            fio.rewind
-            logger.error "Upload error (#{e.code} #{e.message}), retrying (#{upload_tries}/#{max_upload_tries})..."
-            sleep 10
-            retry
-          else
-            raise e
-          end
-        ensure
-          OSDN::CLI._show_progress = false
-          fio.close
-        end
-        if digests.find { |type, dig| dig != finfo.send("digest_#{type}") }
-          logger.error "File digests are mismatch! Upload file #{file} may be broken! Please check."
-        else
-          logger.info "Upload complete."
-        end
+        finfo = filecmd.create_one file
       end
       $stdout.puts "New file '#{file}' has been uploaded; #{finfo.url}"
     end

data/lib/osdn/cli/command/relfile.rb

@@ -3,6 +3,8 @@ require 'pathname'
 
 module OSDN; module CLI; module Command
   class Relfile < FrsBase
+    attr_accessor :target_proj, :target_package, :target_release, :visibility, :force_digest, :show_progress
+
     def help
       puts "#{$0} relfile [opts] [list]"
       puts "#{$0} relfile [opts] create <target-file> [target-files...]"
@@ -93,16 +95,14 @@ module OSDN; module CLI; module Command
       end
     end
 
-    def create_one(filename)
+    def calc_file_digest(filename)
       file = Pathname('.') + filename
-      logger.debug "Calculating digest for #{file}..."
-
       vars = load_variables(file.dirname)
       digests = nil
       if !@force_digest && vars.local_file_info &&
-         vars.local_file_info[file.basename.to_s]
+        vars.local_file_info[file.basename.to_s]
         finfo = vars.local_file_info[file.basename.to_s]
-        if finfo[:size] == file.size && finfo.mtime == file.mtime
+        if finfo.filesize == file.size && finfo.mtime == file.mtime
           digests = vars.local_file_info[file.basename.to_s].digests
         end
       end
@@ -114,19 +114,24 @@ module OSDN; module CLI; module Command
           sha1:   hexdigest(Digest::SHA1, file),
           md5:    hexdigest(Digest::MD5, file),
         }
-        update_variables file.dirname, {local_file_info: {file.basename.to_s => {digests: digests, mtime: file.mtime, size: file.size}}}
+        update_variables file.dirname, {local_file_info: {file.basename.to_s => {digests: digests, mtime: file.mtime, filesize: file.size}}}
       end
+    end
 
+    def create_one(filename)
+      file = Pathname('.') + filename
+      calc_file_digest file
+      vars = load_variables(file.dirname)
       fio = file.open
       logger.level <= Logger::INFO && @show_progress != false || @show_progress and
         OSDN::CLI._show_progress = true
-      logger.info "Starting upload #{file}..."
+      logger.info "Starting upload #{file} (#{file.size} bytes)..."
       max_upload_tries = 5
       upload_tries = 0
-      f = nil
+      remote_file = nil
       begin
         upload_tries += 1
-        f = api.create_release_file target_proj, target_package, target_release, fio, visibility: @visibility
+        remote_file = api.create_release_file target_proj, target_package, target_release, fio, visibility: @visibility
       rescue OSDNClient::ApiError => e
         if max_upload_tries - upload_tries <= 0 
           logger.error "Max upload attempts (#{max_upload_tries}) has been exceeded, give up!"
@@ -144,12 +149,15 @@ module OSDN; module CLI; module Command
         OSDN::CLI._show_progress = false
       end
       
-      if digests.find { |type, dig| dig != f.send("digest_#{type}") }
-        logger.error "File digests are mismatch! Upload file #{file} may be broken! Please check."
+      if vars.local_file_info[file.basename.to_s].digests.find { |type, dig| rd = remote_file.send("digest_#{type}"); rd && rd != '' && rd != dig }
+        logger.error "File digest mismatch! Uploaded file #{file} may be broken! Please check."
+      elsif file.size != remote_file.size
+        logger.error "File size mismatch! Uploaded file #{file} may be broken! Please check."
       else
         logger.info "Upload complete."
       end
-      puts format_file(f)
+      puts format_file(remote_file)
+      remote_file
     end
 
     def update

data/lib/osdn/cli/version.rb

@@ -1,5 +1,5 @@
 module OSDN
   module CLI
-    VERSION = "0.1.9"
+    VERSION = "0.1.10"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: osdn-cli
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.1.10
 platform: ruby
 authors:
 - OSDN
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-05-01 00:00:00.000000000 Z
+date: 2018-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -129,7 +129,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.2
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: OSDN Command Line Interface