Issue #3955 has been updated by donavan m.
Is the `checksum => "{type}value"` form one that really needs to be preserved?
I don't think I've ever seen it in the wild, and it's a newish (under
documented) feature IIRC. I'd rather explicitly use `checksum => "VALUE",
checksum_type => "md5"` if we have the later parameter anyways.
I don't see much benefit to the `TYPE => "VALUE"` sugar. File is a param rich
class already. There will be a sane default checksum_type (I assume). It's
simple to set a resource default in scope. Explicitly setting `checksum_type`
only seems inferior in char count.
PS: Specifying file contents by checksum is awesome. I really hope this gets
some more adoption in 2.7+.
----------------------------------------
Bug #3955: Specifying file content by checksum should not use 'content' property
https://projects.puppetlabs.com/issues/3955
Author: Markus Roberts
Status: Accepted
Priority: High
Assignee:
Category:
Target version:
Affected Puppet version: development
Keywords:
Branch:
Rather than passing around the contents of a file (e.g. in logs) we use the
checksum; to support this the users is also allowed to _specify_ the contents
as a checksum, but this leads to an ambiguity when the actual contents match
the checksum pattern.
Instead of trying to do magic, we should have distinct syntax for referring to
the contents by checksum (which should _always_ expect a valid checksum) and
for specifying the contents literally (which shouldn't do anything different if
the contents happen to resemble a checksum).
--
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.
