end
end
-class ArvadosClient < Google::APIClient
- def execute(*args)
- if args.last.is_a? Hash
- args.last[:headers] ||= {}
- args.last[:headers]['Accept'] ||= 'application/json'
- end
- super(*args)
- end
-end
-
def init_config
# read authentication data from arvados configuration file if present
lineno = 0
end
begin
- client = ArvadosClient.new(:host => ENV['ARVADOS_API_HOST'], :application_name => 'arvados-cli', :application_version => '1.0')
+ client = Google::APIClient.new(:host => ENV['ARVADOS_API_HOST'], :application_name => 'arvados-cli', :application_version => '1.0')
arvados = client.discovered_api('arvados', ENV['ARVADOS_API_VERSION'])
rescue Exception => e
puts "Failed to connect to Arvados API server: #{e}"
})
end
+request_id = result.headers[:x_request_id]
begin
results = JSON.parse result.body
rescue JSON::ParserError => e
- abort "Failed to parse server response:\n" + e.to_s
+ err_msg = "Failed to parse server response:\n" + e.to_s
+ if request_id
+ err_msg += "\nRequest ID: #{request_id or client.request_id}"
+ end
+ abort err_msg
end
if results["errors"] then
- abort "Error: #{results["errors"][0]}"
+ err_message = results["errors"][0]
+ if request_id and !err_message.match(/.*req-[0-9a-zA-Z]{20}.*/)
+ err_message += " (#{request_id})"
+ end
+ abort "Error: #{err_message}"
end
case global_opts[:format]
gemspec
gem 'rake'
gem 'minitest', '>= 5.0.0'
+gem 'mocha', require: false
gem 'signet', '<= 0.11'
end
class Arvados
+ class ArvadosClient < Google::APIClient
+ attr_reader :request_id
+
+ def execute(*args)
+ @request_id = "req-" + Random::DEFAULT.rand(2**128).to_s(36)[0..19]
+ if args.last.is_a? Hash
+ args.last[:headers] ||= {}
+ args.last[:headers]['X-Request-Id'] = @request_id
+ end
+ begin
+ super(*args)
+ rescue => e
+ if !e.message.match(/.*req-[0-9a-zA-Z]{20}.*/)
+ raise $!, "#{$!} (Request ID: #{@request_id})", $!.backtrace
+ end
+ raise e
+ end
+ end
+ end
+
class TransactionFailedError < StandardError
end
end
def client
- @client ||= Google::APIClient.
+ @client ||= ArvadosClient.
new(:host => config["ARVADOS_API_HOST"],
:application_name => @application_name,
:application_version => @application_version.to_s)
})
resp = JSON.parse result.body, :symbolize_names => true
if resp[:errors]
+ if !resp[:errors][0].match(/.*req-[0-9a-zA-Z]{20}.*/)
+ resp[:errors][0] += " (#{result.headers['X-Request-Id'] or client.request_id})"
+ end
raise Arvados::TransactionFailedError.new(resp[:errors])
elsif resp[:uuid] and resp[:etag]
self.new(resp)
--- /dev/null
+# Copyright (C) The Arvados Authors. All rights reserved.
+#
+# SPDX-License-Identifier: Apache-2.0
+
+require "arvados"
+require "mocha/minitest"
+
+class FakeError < StandardError; end
+class RequestIdTest < Minitest::Test
+ def test_raise_exception_with_request_id
+ arv = Arvados.new
+ clnt = arv.client
+ assert_nil clnt.request_id
+
+ Google::APIClient.any_instance.stubs(:execute).raises(FakeError.new("Uh-oh..."))
+ err = assert_raises(FakeError) do
+ arv.collection.get(uuid: "zzzzz-4zz18-zzzzzzzzzzzzzzz")
+ end
+ assert clnt.request_id != nil
+ assert_match /Uh-oh.*\(Request ID: req-[0-9a-zA-Z]{20}\)/, err.message
+ end
+end
\ No newline at end of file
}
end
- def client_accepts_plain_text_stream
- (request.headers['Accept'].split(' ') &
- ['text/plain', '*/*']).count > 0
- end
-
def render *opts
if opts.first
response = opts.first[:json]