All other env vars are to be stringified

[#97563344]
cloudfoundry · Jun 30, 2015 · 4ffa01a · 4ffa01a
1 parent 0e0de33
commit 4ffa01a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/lib/cloud_controller/diego/environment.rb b/lib/cloud_controller/diego/environment.rb
@@ -22,6 +22,8 @@ def as_json(_={})
           case v
           when Array, Hash
             v = MultiJson.dump(v)
+          else
+            v = v.to_s
           end
 
           env << { 'name' => k, 'value' => v }

diff --git a/spec/unit/lib/cloud_controller/diego/environment_spec.rb b/spec/unit/lib/cloud_controller/diego/environment_spec.rb
@@ -8,6 +8,8 @@ module VCAP::CloudController::Diego
         APP_KEY1: 'APP_VAL1',
         APP_KEY2: { nested: 'data' },
         APP_KEY3: [1, 2, 3],
+        APP_KEY4: 1,
+        APP_KEY5: true,
       }
       app
     end
@@ -27,6 +29,8 @@ module VCAP::CloudController::Diego
         { 'name' => 'APP_KEY1', 'value' => 'APP_VAL1' },
         { 'name' => 'APP_KEY2', 'value' => '{"nested":"data"}' },
         { 'name' => 'APP_KEY3', 'value' => '[1,2,3]' },
+        { 'name' => 'APP_KEY4', 'value' => '1' },
+        { 'name' => 'APP_KEY5', 'value' => 'true' },
       ])
     end