Bug #14723

http report processor description is wrong

Added by Lauri Tirkkonen about 1 year ago. Updated 10 months ago.

Status:ClosedStart date:05/29/2012
Priority:NormalDue date:
Assignee:Nick Fagerlund% Done:

0%

Category:documentation
Target version:2.7.16
Affected Puppet version:2.7.14 Branch:https://github.com/puppetlabs/puppet/pull/818
Keywords:

Description

http://docs.puppetlabs.com/references/latest/report.html#http reads:

Send report information via HTTP to the reporturl. Each host sends its report as a YAML dump and this sends this YAML to a client via HTTP POST. The YAML is the report parameter of the request.”

However, YAML is actually sent as the request body, so the description is misleading (I thought it meant report=<urlencoded yaml>).

I’ll create a pull request for this once Redmine assigns me a ticket number I can reference in the commit message.

History

#2 Updated by Kelsey Hightower about 1 year ago

  • Status changed from Unreviewed to In Topic Branch Pending Review
  • Assignee set to Nick Fagerlund
  • Branch set to https://github.com/puppetlabs/puppet/pull/818

Nick,

Can you review this pull request? https://github.com/puppetlabs/puppet/pull/818

#3 Updated by Nick Fagerlund about 1 year ago

Reviewed. It looks fine, and I’m going to merge it. The quote-mark removal in the commit is also fine; looks like that was a formatting turd from before the string was a heredoc.

#4 Updated by Nick Fagerlund about 1 year ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release
  • Target version set to 2.7.15

#5 Updated by Matthaus Owens about 1 year ago

  • Status changed from Merged - Pending Release to Closed
  • Target version changed from 2.7.15 to 2.7.16

Released in 2.7.16rc1

#6 Updated by Matthaus Owens 10 months ago

Released in Puppet 3.0.0-rc4

Also available in: Atom PDF