From: Lucas Di Pentima <lucas.dipentima@curii.com>
Date: Thu, 27 May 2021 14:28:50 +0000 (-0300)
Subject: 17738: Fixes the issue by strip!()ing a mutable version of OS_VERSION.
X-Git-Tag: 2.2.0~12^2
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/116c96b2c709ed47490ddf05a66000fa3974359b

17738: Fixes the issue by strip!()ing a mutable version of OS_VERSION.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima@curii.com>
---

diff --git a/sdk/ruby/lib/arvados/google_api_client.rb b/sdk/ruby/lib/arvados/google_api_client.rb
index 69383d12f6..5a41b11c4f 100644
--- a/sdk/ruby/lib/arvados/google_api_client.rb
+++ b/sdk/ruby/lib/arvados/google_api_client.rb
@@ -6,7 +6,8 @@ require 'google/api_client'
 # Monkeypatch google-api-client gem to avoid sending newline characters
 # on headers to make ruby-2.3.7+ happy.
 # See: https://dev.arvados.org/issues/13920
-Google::APIClient::ENV::OS_VERSION.strip!
+# Addendum: OS_VERSION sometimes is not mutable, that's why we duplicate it (See: #17738)
+Google::APIClient::ENV::OS_VERSION = +Google::APIClient::ENV::OS_VERSION.strip!
 
 require 'json'
 require 'tempfile'