Uploaded image for project: 'Puppet'
  1. Puppet
  2. PUP-7257

Accept JSON encoded facts

    XMLWordPrintable

Details

    • Task
    • Status: Closed
    • Normal
    • Resolution: Fixed
    • None
    • PUP 5.0.0
    • None
    • None
    • New Feature
    • The puppet master can now accept JSON encoded facts, but will continue to accept PSON encoded facts from older agents.
    • No Action

    Description

      At https://github.com/puppetlabs/puppet/blob/master/lib/puppet/configurer/fact_handler.rb#L35 we "extra" CGI.escape the facts content we are going to include in the body that gets sent to the master.

      We do not need to do this. We should just url encode the whole body that gets sent.

      Update (or equivalent):

      {:facts_format => :pson, :facts => CGI.escape(text)}
      

      to something like:

      {:facts_format => :json, :facts => facts.to_json}
      

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              moses Moses Mendoza
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Zendesk Support