It would be nice to remove the document_type from the PSON hashes for Puppet 4. This would be a modification of the API though, so it is a backwards incompatible change. But it is data that AFAIK the agent throws away anyway when it receives it.
The benefit of doing this change is that the API would look the same for all serialisation formats, and a bunch of code for the PSON special handling could be removed. It should be completely backwards compatible for Puppet, but other consumers of the API might be affected. Here is a example of code that adds this information to the API reply (there's a bunch of places that does this): https://github.com/puppetlabs/puppet/blob/puppet-4/lib/puppet/resource/catalog.rb#L390-L403 And here is the code that throws away that info: https://github.com/puppetlabs/puppet/blob/puppet-4/lib/puppet/network/formats.rb#L157-L167 What do you others think about doing that change? -- Erik Dalén -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to puppet-dev+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/puppet-dev/CAAAzDLfKczVcG3LPLhFRSJhxB1hhxu%2B6jOQDT3w0uf3vD9p04w%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
