right_support 2.8.35 → 2.8.36

data/VERSION

@@ -1 +1 @@
-2.8.35
+2.8.36

data/lib/right_support/deployment/info.rb

@@ -50,17 +50,20 @@ module RightSupport::Deployment
       # commit does not indicate the last commit that would be seen in the Git repo anywhere else,
       # so we should ignore this commit whe logging the current repositories state.
       commit_diff = `git --git-dir #{Shellwords.escape(@dir.to_s)}/.git log origin/HEAD..HEAD \
-                         --pretty=format:'%ae'`.split(/\n+/)
+                         --pretty=format:'%ae' 2> /dev/null`.split(/\n+/)
 
       ops_commit_exists = (commit_diff.count == 1 && commit_diff.first =~ /ops@/)
-      ops_commit_exists ? head_commit = 1 : head_commit = 0
+      ($?.success? && ops_commit_exists) ? head_commit = 1 : head_commit = 0
 
-      @recent_commits = `git --git-dir #{Shellwords.escape(@dir.to_s)}/.git log \
+      recent_commits = `git --git-dir #{Shellwords.escape(@dir.to_s)}/.git log \
                              -n #{Shellwords.escape(@num_commits.to_s)} \
-                             --pretty=format:'%h %an %ad'`.split(/\n+/)[head_commit..-1]
+                             --pretty=format:'%h %an %ad' 2> /dev/null`.split(/\n+/)[head_commit..-1]
+      @recent_commits = $?.success? ? recent_commits : []
 
-      @head_commit = `git --git-dir #{Shellwords.escape(@dir.to_s)}/.git log -n 1 \
-                          --pretty=format:'%H' #{"HEAD~" if ops_commit_exists}`
+      head_commit = `git --git-dir #{Shellwords.escape(@dir.to_s)}/.git log -n 1 \
+                          --pretty=format:'%H' #{"HEAD~" if ops_commit_exists} 2> /dev/null`
+
+      @head_commit = $?.success? ? head_commit : 'unknown'
     end
   end
 end

data/lib/right_support/rack/request_logger.rb

@@ -159,8 +159,10 @@ module RightSupport::Rack
       else
         if body.is_a?(Array)
           body.reduce(0) {|accum, e| accum += e.bytesize}
-        else
+        elsif body.is_a?(String)
           body.bytesize
+        else
+          '-'
         end
       end
 

data/right_support.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "right_support"
-  s.version = "2.8.35"
+  s.version = "2.8.36"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Tony Spataro", "Sergey Sergyenko", "Ryan Williamson", "Lee Kirchhoff", "Alexey Karpik", "Scott Messier"]
-  s.date = "2014-11-12"
+  s.date = "2014-11-18"
   s.description = "A toolkit of useful, reusable foundation code created by RightScale."
   s.email = "support@rightscale.com"
   s.extra_rdoc_files = [

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: right_support
 version: !ruby/object:Gem::Version
-  version: 2.8.35
+  version: 2.8.36
   prerelease: 
 platform: ruby
 authors:
@@ -14,7 +14,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-12 00:00:00.000000000 Z
+date: 2014-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -245,7 +245,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -578187549211322829
+      hash: -3247930919812590420
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: