[ARVADOS] updated: 9fea1c7774bd256788ee76385c0eab05d4508796

git at public.curoverse.com git at public.curoverse.com
Mon Jan 12 12:21:32 EST 2015


Summary of changes:
 sdk/cli/bin/arv | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

       via  9fea1c7774bd256788ee76385c0eab05d4508796 (commit)
      from  5fa38c36644bd122b8c31601aa864f05bf4fba73 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 9fea1c7774bd256788ee76385c0eab05d4508796
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Mon Jan 12 12:23:40 2015 -0500

    4924: Update prints uuid from results, not the uuid originally specified.
    Catch Oj::ParseError as well as JSON::ParserError.  Titleize only HTTP status
    messages, not every error message.

diff --git a/sdk/cli/bin/arv b/sdk/cli/bin/arv
index f27711f..8aebc5f 100755
--- a/sdk/cli/bin/arv
+++ b/sdk/cli/bin/arv
@@ -24,6 +24,7 @@ begin
   require 'active_support/inflector'
   require 'yaml'
   require 'tempfile'
+  require 'net/http'
 rescue LoadError
   abort <<-EOS
 
@@ -220,7 +221,7 @@ def edit_and_commit_object initial_obj, tmp_stem, global_opts, &block
         can_retry = true
         if e.is_a? Psych::SyntaxError
           this_error = "YAML error parsing your input: #{e}"
-        elsif e.is_a? JSON::ParserError
+        elsif e.is_a? JSON::ParserError or e.is_a? Oj::ParseError
           this_error = "JSON error parsing your input: #{e}"
         elsif e.is_a? ArvadosAPIError
           this_error = "API responded with error #{e}"
@@ -264,13 +265,14 @@ def check_response result
     results = JSON.parse result.body
   rescue JSON::ParserError => e
     raise "Failed to parse server response:\n" + e.to_s
+  rescue Oj::ParseError => e
+    raise "Failed to parse server response:\n" + e.to_s
   end
 
   if result.response.status != 200
-    require 'net/http'
     raise ArvadosAPIError.new("#{result.response.status}: #{
                               ((results['errors'] && results['errors'].join('\n')) ||
-                                Net::HTTPResponse::CODE_TO_OBJ[status.to_s].to_s.sub(/^Net::HTTP/, '')).titleize}")
+                                Net::HTTPResponse::CODE_TO_OBJ[status.to_s].to_s.sub(/^Net::HTTP/, '').titleize)}")
   end
 
   results
@@ -343,8 +345,8 @@ def arv_edit client, arvados, global_opts, remaining_opts
                      :headers => {
                        authorization: 'OAuth2 '+ENV['ARVADOS_API_TOKEN']
                      })
-      check_response result
-      puts "Updated object #{uuid}"
+      results = check_response result
+      puts "Updated object #{results['uuid']}"
     else
       puts "Object is unchanged, did not update."
     end

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list