Issue #10450 has been updated by Josh Cooper.

Category set to serialization
Status changed from Unreviewed to Accepted
Keywords set to yaml

Hi Matthew, 

I agree that the current regex is incorrect, but the problem is that yaml 
allows directives to appear before the separator `---`, so just matching the 
start of string it too restrictive. I think we need some fancier regex-fu to 
make this work. Btw, are you seeing a specific failure as a result of this?
----------------------------------------
Bug #10450: Puppet::Util::ReferenceSerializer.unserialize_value() incorrectly 
tries to YAML decode a value containing '--- ' at the start of _any_ line 
https://projects.puppetlabs.com/issues/10450

Author: Matthew Leather
Status: Accepted
Priority: Normal
Assignee: 
Category: serialization
Target version: 
Affected Puppet version: 2.6.2
Keywords: yaml
Branch: 


I think YAML applies only when the first line is '--- ' so the re should use \A 
instead of ^

I see it in 2.6.2 but I got a clone of git://github.com/puppetlabs/puppet today 
and reference_serializer.rb still has the problem.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to