From f66095d35bc5f9645eb19bbb8cefa342c0181d2d Mon Sep 17 00:00:00 2001 From: Jesse Wolfe Date: Mon, 26 Apr 2010 18:14:51 -0700 Subject: Fix #3656 JSON serialization of dependencies The pson serialization of resources was behaving incorrectly on parameters that are references to other resources: 1. Dependency parameters (require, subscribe, notify) were getting serialized as anonymous objects that looked like partially constructed resources 2. During de-serialization the pson parser would inflate them into hashes (rather than into resources) 3. The outer resource would try to coerce the hash into a resource by passing it to Resource.new 4. Resource.new would fail with a cryptic message, since it does not accept a hash as its first parameter (but the error is obfuscated by Resource.new's complicated argument handler) This patch solves the problem by explicitly converting dependency parameters into strings in the pson serialization. Signed-off-by: Jesse Wolfe --- lib/puppet/resource.rb | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/puppet/resource.rb b/lib/puppet/resource.rb index f991f539d..d2979f930 100644 --- a/lib/puppet/resource.rb +++ b/lib/puppet/resource.rb @@ -60,7 +60,11 @@ class Puppet::Resource # Don't duplicate the title as the namevar next hash if param == namevar and value == title - hash[param] = value + if value.is_a? Puppet::Resource + hash[param] = value.to_s + else + hash[param] = value + end hash end -- cgit