sdr-client 0.17.2 → 0.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 49708bd983a0e4e3322f7a1449d18bd2836f42020fb0c4fec06cf6ce25ec78a5
-  data.tar.gz: a6aecda31d1791026f321147514f7d1ec36a5080aa0df3e4e7638c10125de1f5
+  metadata.gz: 52457756d6413fcbb3d78c6778e705dd000f73d4bfd9be51b4455c91f861f2cf
+  data.tar.gz: 2e3fbb925f332ddbf6f70a011f708ed659bb768bf3cea618e3b40cea70b7e1a8
 SHA512:
-  metadata.gz: 3238ad6cbb4ade491ff443448b46f9ee78d081b233a07678046421b1de2a80108074428ccb6a8a768d4cbf1af4a60b14dfb37b8336c83ded75dd04c2cb8a743d
-  data.tar.gz: cf082df7b2efe67c88349644545495d3555206efb6aca5a2eca156ab144db72a509c9a0c5aa1b69a7d71504c8c14bca9fa75b57dcc5c7cd37a00902387d0f73d
+  metadata.gz: cdaa5ce7b1c83361d33911f02191d4a34c97c24113ffc01e4594a2300b966ef7b0ff399ea56ee033650be0eb808dd3a363dbf04357c2e124ebfe378d78a69459
+  data.tar.gz: 238c7ff0047034134d1a3c4421bed157e09b97692ec6d57226edfc2e4e5ef0a7be3a88a39b9c10f816ae6e53dc7f44d10dea7d38a1f7424bb9a6a5c9b3ea7fd2

data/.rubocop.yml

@@ -9,6 +9,8 @@ AllCops:
 
 Layout/LineLength:
   Max: 120
+  Exclude:
+    - 'spec/sdr_client/deposit/process_spec.rb'
 
 Metrics/BlockLength:
   Exclude:

data/README.md

@@ -21,8 +21,15 @@ Log in:
 sdr --service-url http://sdr-api-server:3000 login
 ```
 
+Register a new object:
+```
+sdr --service-url https://sdr-api-server:3000 register --label 'hey there' \
+  --admin-policy 'druid:bk123gh4567' \
+  --collection 'druid:gh456kw9876' \
+  --source-id 'googlebooks:stanford_12345' file1.png file2.png
+```
 
-Deposit a new object:
+Deposit (register + accession) a new object:
 ```
 sdr --service-url https://sdr-api-server:3000 deposit --label 'hey there' \
   --admin-policy 'druid:bk123gh4567' \

data/exe/sdr

@@ -27,7 +27,10 @@ global = OptionParser.new do |opts|
 
       COMMANDS
         deposit
-        deposit files to the SDR
+        accession object into the SDR
+
+        register
+        create a draft object in SDR
 
         login
         Will prompt for email & password and exchange it for an login token, which it saves in ~/.sdr/token
@@ -40,77 +43,80 @@ end
 global.order!
 command = ARGV.shift
 
-subcommands = {
-  'deposit' => OptionParser.new do |opts|
-    opts.on('--label LABEL', 'The object label') do |label|
-      options[:label] = label
-    end
+deposit_options = OptionParser.new do |opts|
+  opts.on('--label LABEL', 'The object label') do |label|
+    options[:label] = label
+  end
 
-    opts.on('--admin-policy ADMIN_POLICY', 'The druid identifier of the admin policy object') do |apo|
-      options[:apo] = apo
-    end
+  opts.on('--admin-policy ADMIN_POLICY', 'The druid identifier of the admin policy object') do |apo|
+    options[:apo] = apo
+  end
 
-    opts.on('--type TYPE', 'The object type to create. ' \
-            'One of: "image", "book", "document", "map", "manuscript", "media", ' \
-            '"three_dimensional", "collection", or "admin_policy"') do |type|
-      if %w[image book document map manuscript media three_dimensional collection admin_policy].include?(type)
-        options[:type] = "http://cocina.sul.stanford.edu/models/#{type}.jsonld"
-      end
+  opts.on('--type TYPE', 'The object type to create. ' \
+          'One of: "image", "book", "document", "map", "manuscript", "media", ' \
+          '"three_dimensional", "collection", or "admin_policy"') do |type|
+    if %w[image book document map manuscript media three_dimensional collection admin_policy].include?(type)
+      options[:type] = "http://cocina.sul.stanford.edu/models/#{type}.jsonld"
     end
+  end
 
-    opts.on('--collection COLLECTION', 'The druid identifier of the collection object') do |collection|
-      options[:collection] = collection
-    end
+  opts.on('--collection COLLECTION', 'The druid identifier of the collection object') do |collection|
+    options[:collection] = collection
+  end
 
-    opts.on('--catkey CATKEY', 'The catkey for this item') do |catkey|
-      options[:catkey] = catkey
-    end
+  opts.on('--catkey CATKEY', 'The catkey for this item') do |catkey|
+    options[:catkey] = catkey
+  end
 
-    opts.on('--source-id SOURCE_ID', 'The source id for this object') do |source_id|
-      options[:source_id] = source_id
-    end
+  opts.on('--source-id SOURCE_ID', 'The source id for this object') do |source_id|
+    options[:source_id] = source_id
+  end
 
-    opts.on('--copyright COPYRIGHT', 'The copyright statement') do |copyright|
-      options[:copyright] = copyright
-    end
+  opts.on('--copyright COPYRIGHT', 'The copyright statement') do |copyright|
+    options[:copyright] = copyright
+  end
 
-    opts.on('--use-statement STATEMENT', 'The use and reproduction statement') do |use_statement|
-      options[:use_statement] = use_statement
-    end
+  opts.on('--use-statement STATEMENT', 'The use and reproduction statement') do |use_statement|
+    options[:use_statement] = use_statement
+  end
 
-    opts.on('--viewing-direction DIRECTION', 'The viewing direction (if a book). ' \
-            'Either "left-to-right" or "right-to-left"') do |viewing_direction|
-      options[:viewing_direction] = viewing_direction if %w[left-to-right right-to-left].include?(viewing_direction)
-    end
+  opts.on('--viewing-direction DIRECTION', 'The viewing direction (if a book). ' \
+          'Either "left-to-right" or "right-to-left"') do |viewing_direction|
+    options[:viewing_direction] = viewing_direction if %w[left-to-right right-to-left].include?(viewing_direction)
+  end
 
-    opts.on('--access LEVEL', 'The access level for this object. ' \
-            'Either "world", "stanford", "location-based", "citation-only" or "dark"') do |level|
-      options[:access] = level if %w[world stanford location-based citation-only dark].include?(level)
-    end
+  opts.on('--access LEVEL', 'The access level for this object. ' \
+          'Either "world", "stanford", "location-based", "citation-only" or "dark"') do |level|
+    options[:access] = level if %w[world stanford location-based citation-only dark].include?(level)
+  end
 
-    opts.on('--files-metadata FILES_METADATA', 'A JSON object representing per-file metadata') do |files_metadata|
-      options[:files_metadata] = JSON.parse(files_metadata)
-    end
+  opts.on('--files-metadata FILES_METADATA', 'A JSON object representing per-file metadata') do |files_metadata|
+    options[:files_metadata] = JSON.parse(files_metadata)
+  end
 
-    opts.on('--strategy STRATEGY',
-            'The strategy to use for distributing files into filesets. Either "default" or "filename"') do |strategy|
-      strategy_class = case strategy
-                       when 'filename'
-                         SdrClient::Deposit::MatchingFileGroupingStrategy
-                       when 'default'
-                         SdrClient::Deposit::SingleFileGroupingStrategy
-                       else
-                         warn "Unknown strategy #{strategy}"
-                         exit(1)
-                       end
-      options[:grouping_strategy] = strategy_class
-    end
+  opts.on('--strategy STRATEGY',
+          'The strategy to use for distributing files into filesets. Either "default" or "filename"') do |strategy|
+    strategy_class = case strategy
+                     when 'filename'
+                       SdrClient::Deposit::MatchingFileGroupingStrategy
+                     when 'default'
+                       SdrClient::Deposit::SingleFileGroupingStrategy
+                     else
+                       warn "Unknown strategy #{strategy}"
+                       exit(1)
+                     end
+    options[:grouping_strategy] = strategy_class
+  end
 
-    opts.on('-h', '--help', 'Display this screen') do
-      puts opts
-      exit
-    end
-  end,
+  opts.on('-h', '--help', 'Display this screen') do
+    puts opts
+    exit
+  end
+end
+
+subcommands = {
+  'deposit' => deposit_options,
+  'register' => deposit_options,
   'login' => OptionParser.new
 }
 

data/lib/sdr_client.rb

@@ -4,6 +4,8 @@ require 'dry/monads'
 require 'faraday'
 require 'active_support'
 require 'active_support/core_ext/object/json'
+require 'active_support/core_ext/hash/indifferent_access'
+require 'cocina/models'
 
 require 'sdr_client/version'
 require 'sdr_client/deposit'
@@ -12,6 +14,7 @@ require 'sdr_client/credentials'
 require 'sdr_client/login'
 require 'sdr_client/login_prompt'
 require 'sdr_client/cli'
+require 'sdr_client/connection'
 
 module SdrClient
   class Error < StandardError; end

data/lib/sdr_client/cli.rb

@@ -6,7 +6,9 @@ module SdrClient
     def self.start(command, options)
       case command
       when 'deposit'
-        SdrClient::Deposit.run(options)
+        SdrClient::Deposit.run(accession: true, **options)
+      when 'register'
+        SdrClient::Deposit.run(accession: false, **options)
       when 'login'
         status = SdrClient::Login.run(options)
         puts status.value if status.failure?

data/lib/sdr_client/connection.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module SdrClient
+  # The connection to the server
+  class Connection
+    include Dry::Monads[:result]
+
+    def initialize(url:, token: Credentials.read)
+      @url = url
+      @token = token
+    end
+
+    def connection
+      @connection ||= Faraday.new(url: url) do |conn|
+        conn.authorization :Bearer, token
+        conn.adapter :net_http
+      end
+    end
+
+    # This is only available to certain blessed accounts (argo) as it gives the
+    # token that allows you to act as any other user. Thus the caller must authenticate
+    # the user (e.g. using Shibboleth) before calling this method with their email address.
+    # @param [String] the email address of the person to proxy to.
+    # @return [Result] the token for the account
+    def proxy(to)
+      response = connection.post("/v1/auth/proxy?to=#{to}")
+      case response.status
+      when 200
+        Success(response.body)
+      else
+        Failure("Status: #{response.status}\n#{response.body}")
+      end
+    end
+
+    delegate :put, :post, to: :connection
+
+    private
+
+    attr_reader :url, :token
+  end
+end

data/lib/sdr_client/deposit.rb

@@ -23,10 +23,9 @@ module SdrClient
                  url:,
                  files: [],
                  files_metadata: {},
+                 accession: false,
                  grouping_strategy: SingleFileGroupingStrategy,
                  logger: Logger.new(STDOUT))
-      token = Credentials.read
-
       augmented_metadata = FileMetadataBuilder.build(files: files, files_metadata: files_metadata)
       metadata = Request.new(label: label,
                              type: type,
@@ -41,8 +40,11 @@ module SdrClient
                              embargo_access: embargo_access,
                              viewing_direction: viewing_direction,
                              files_metadata: augmented_metadata)
-      Process.new(metadata: metadata, url: url, token: token, files: files,
-                  grouping_strategy: grouping_strategy, logger: logger).run
+      connection = Connection.new(url: url)
+      Process.new(metadata: metadata, connection: connection, files: files,
+                  grouping_strategy: grouping_strategy,
+                  accession: accession,
+                  logger: logger).run
     end
     # rubocop:enable Metrics/MethodLength
     # rubocop:enable Metrics/ParameterLists

data/lib/sdr_client/deposit/file.rb

@@ -34,9 +34,10 @@ module SdrClient
           administrative: {
             sdrPreserve: @preserve,
             shelve: @shelve
-          }
+          },
+          version: 1,
+          hasMessageDigests: message_digests
         }.tap do |json|
-          json['hasMessageDigests'] = message_digests unless message_digests.empty?
           json['hasMimeType'] = @mime_type if @mime_type
           json['use'] = @use if @use
         end

data/lib/sdr_client/deposit/file_set.rb

@@ -25,7 +25,8 @@ module SdrClient
           "label": label,
           structural: {
             contains: files.map(&:as_json)
-          }
+          },
+          version: 1
         }
       end
 

data/lib/sdr_client/deposit/model_process.rb

@@ -8,15 +8,12 @@ module SdrClient
     class ModelProcess
       DRO_PATH = '/v1/resources'
       # @param [Cocina::Model::RequestDRO] request_dro for depositing
-      # @param [String] url the server to send to
-      # @param [String] token the bearer auth token for the server
+      # @param [Connection] connection the connection to use
       # @param [Array<String>] files a list of file names to upload
       # @param [Logger] logger the logger to use
-      def initialize(request_dro:, url:,
-                     token:, files: [], logger: Logger.new(STDOUT))
+      def initialize(request_dro:, connection:, files: [], logger: Logger.new(STDOUT))
         @files = files
-        @url = url
-        @token = token
+        @connection = connection
         @request_dro = request_dro
         @logger = logger
       end
@@ -35,7 +32,7 @@ module SdrClient
 
       private
 
-      attr_reader :request_dro, :files, :url, :token, :logger
+      attr_reader :request_dro, :files, :logger, :connection
 
       def check_files_exist
         logger.info('checking to see if files exist')
@@ -77,13 +74,6 @@ module SdrClient
         raise "unexpected response: #{response.status} #{response.body}"
       end
 
-      def connection
-        @connection ||= Faraday.new(url: url) do |conn|
-          conn.authorization :Bearer, token
-          conn.adapter :net_http
-        end
-      end
-
       # Map of filenames to mimetypes
       def mime_types
         @mime_types ||=

data/lib/sdr_client/deposit/process.rb

@@ -9,22 +9,24 @@ module SdrClient
       DRO_PATH = '/v1/resources'
       # @param [Request] metadata information about the object
       # @param [Class] grouping_strategy class whose run method groups an array of uploads
-      # @param [String] url the server to send to
-      # @param [String] token the bearer auth token for the server
+      # @param [String] connection the server connection to use
       # @param [Array<String>] files a list of file names to upload
+      # @param [Boolean] accession should the accessionWF be started
       # @param [Logger] logger the logger to use
+      #
       # rubocop:disable Metrics/ParameterLists
-      def initialize(metadata:, grouping_strategy: SingleFileGroupingStrategy, url:,
-                     token:, files: [], logger: Logger.new(STDOUT))
+      def initialize(metadata:, grouping_strategy: SingleFileGroupingStrategy,
+                     connection:, files: [], accession:, logger: Logger.new(STDOUT))
         @files = files
-        @url = url
-        @token = token
+        @connection = connection
         @metadata = metadata
         @logger = logger
         @grouping_strategy = grouping_strategy
+        @accession = accession
       end
       # rubocop:enable Metrics/ParameterLists
 
+      # rubocop:disable Metrics/AbcSize
       def run
         check_files_exist
         upload_responses = UploadFiles.new(files: files,
@@ -35,12 +37,14 @@ module SdrClient
                                                grouping_strategy: grouping_strategy,
                                                logger: logger)
         request = metadata_builder.with_uploads(upload_responses)
-        upload_metadata(request.as_json)
+        model = Cocina::Models.build_request(request.as_json.with_indifferent_access)
+        upload_metadata(model.to_h)
       end
+      # rubocop:enable Metrics/AbcSize
 
       private
 
-      attr_reader :metadata, :files, :url, :token, :logger, :grouping_strategy
+      attr_reader :metadata, :files, :connection, :logger, :grouping_strategy
 
       def check_files_exist
         logger.info('checking to see if files exist')
@@ -49,11 +53,15 @@ module SdrClient
         end
       end
 
+      def accession?
+        @accession
+      end
+
+      # @param [Hash<Symbol,String>] the result of the metadata call
+      # @param [Boolean] accession should the accessionWF be started
       # @return [Hash<Symbol,String>] the result of the metadata call
       def upload_metadata(metadata)
-        logger.info("Starting upload metadata: #{metadata}")
-        request_json = JSON.generate(metadata)
-        response = connection.post(DRO_PATH, request_json, 'Content-Type' => 'application/json')
+        response = metadata_request(metadata)
         unexpected_response(response) unless response.status == 201
 
         logger.info("Response from server: #{response.body}")
@@ -61,6 +69,18 @@ module SdrClient
         { druid: JSON.parse(response.body)['druid'], background_job: response.headers['Location'] }
       end
 
+      def metadata_request(metadata)
+        logger.debug("Starting upload metadata: #{metadata}")
+
+        connection.post(path, JSON.generate(metadata), 'Content-Type' => 'application/json')
+      end
+
+      def path
+        path = DRO_PATH
+        path += '?accession=true' if accession?
+        path
+      end
+
       def unexpected_response(response)
         raise "There was an error with your request: #{response.body}" if response.status == 400
         raise 'There was an error with your credentials. Perhaps they have expired?' if response.status == 401
@@ -68,13 +88,6 @@ module SdrClient
         raise "unexpected response: #{response.status} #{response.body}"
       end
 
-      def connection
-        @connection ||= Faraday.new(url: url) do |conn|
-          conn.authorization :Bearer, token
-          conn.adapter :net_http
-        end
-      end
-
       def mime_types
         @mime_types ||=
           Hash[

data/lib/sdr_client/deposit/request.rb

@@ -49,10 +49,10 @@ module SdrClient
           type: type,
           administrative: administrative,
           identification: identification,
-          structural: structural
-        }.tap do |json|
-          json[:label] = label if label
-        end
+          structural: structural,
+          version: 1,
+          label: label.nil? ? ':auto' : label
+        }
       end
 
       # @return [Request] a clone of this request with the file_sets added

data/lib/sdr_client/deposit/upload_files.rb

@@ -9,7 +9,7 @@ module SdrClient
       BLOB_PATH = '/v1/direct_uploads'
       # @param [Array<String>] files a list of filepaths to upload
       # @param [Logger] logger the logger to use
-      # @param [Faraday::Connection] connection
+      # @param [Connection] connection
       # @param [Hash<String,String] mime_types a map of filenames to mime types
       def initialize(files:, mime_types:, logger:, connection:)
         @files = files

data/lib/sdr_client/login.rb

@@ -7,12 +7,12 @@ module SdrClient
     extend Dry::Monads[:result]
 
     # @return [Result] the status of the call
-    def self.run(url:, login_service: LoginPrompt)
+    def self.run(url:, login_service: LoginPrompt, credential_store: Credentials)
       request_json = JSON.generate(login_service.run)
       response = Faraday.post(url + LOGIN_PATH, request_json, 'Content-Type' => 'application/json')
       case response.status
       when 200
-        Credentials.write(response.body)
+        credential_store.write(response.body)
         Success()
       when 400
         Failure('Email address is not a valid email')

data/lib/sdr_client/model_deposit.rb

@@ -9,9 +9,8 @@ module SdrClient
                        files: [],
                        url:,
                        logger: Logger.new(STDOUT))
-      token = Credentials.read
-
-      ModelProcess.new(request_dro: request_dro, url: url, token: token, files: files, logger: logger).run
+      connection = Connection.new(url: url)
+      ModelProcess.new(request_dro: request_dro, connection: connection, files: files, logger: logger).run
     end
   end
 end

data/lib/sdr_client/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SdrClient
-  VERSION = '0.17.2'
+  VERSION = '0.22.0'
 end

data/sdr-client.gemspec

@@ -28,7 +28,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'activesupport'
-  spec.add_dependency 'cocina-models', '~> 0.28.0'
+  spec.add_dependency 'cocina-models', '~> 0.31.0'
   spec.add_dependency 'dry-monads'
   spec.add_dependency 'faraday', '>= 0.16'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sdr-client
 version: !ruby/object:Gem::Version
-  version: 0.17.2
+  version: 0.22.0
 platform: ruby
 authors:
 - Justin Coyne
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-11 00:00:00.000000000 Z
+date: 2020-04-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.28.0
+        version: 0.31.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.28.0
+        version: 0.31.0
 - !ruby/object:Gem::Dependency
   name: dry-monads
   requirement: !ruby/object:Gem::Requirement
@@ -189,6 +189,7 @@ files:
 - lib/sdr-client.rb
 - lib/sdr_client.rb
 - lib/sdr_client/cli.rb
+- lib/sdr_client/connection.rb
 - lib/sdr_client/credentials.rb
 - lib/sdr_client/deposit.rb
 - lib/sdr_client/deposit/file.rb
@@ -232,7 +233,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: The CLI for https://github.com/sul-dlss/sdr-api