The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #10722

http/https report processors don't error on bad HTTP response codes.

Added by Nigel Kersten over 2 years ago. Updated almost 2 years ago.

Status:ClosedStart date:11/10/2011
Priority:UrgentDue date:
Assignee:-% Done:

0%

Category:reports
Target version:2.7.14
Affected Puppet version: Branch:
Keywords:devtriage

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

Any Net:: errors are raised correctly, but we fail silently if we get a non-200 HTTP response code.


Related issues

Related to Puppet - Feature #8465: allow SSL on reporturl Closed 07/18/2011
Related to Puppet - Bug #5472: No error message if we fail to push a report to Dashboard Accepted 12/07/2010

History

#1 Updated by Nigel Kersten over 2 years ago

  • Assignee changed from Nigel Kersten to Michael Stahnke
  • Priority changed from Normal to Urgent

#2 Updated by Michael Stahnke over 2 years ago

This was ported (as-is) into PE. This was not merged into Puppet core yet, as tests are pending :)

#3 Updated by Patrick Carlisle over 2 years ago

Tests need to be updated for this, several are broken with this patch.

#4 Updated by Michael Stahnke over 2 years ago

  • Target version changed from 2.6.x to 2.7.x

Moving to 2.7.x since this is still an issue and 2.6.x is done.

#5 Updated by Michael Stahnke over 2 years ago

  • Assignee deleted (Michael Stahnke)

I haven’t merged this becuase there was (maybe is) an outstanding pull request with a more advanced https report processor than we setup for PE.

#6 Updated by Michael Stahnke about 2 years ago

This is related to https://github.com/puppetlabs/puppet/pull/175 (which is the other pull request with similar functionality).

#7 Updated by Nigel Kersten about 2 years ago

  • Keywords set to devtriage

#8 Updated by Patrick Carlisle about 2 years ago

Updated pull request with test fixes: https://github.com/puppetlabs/puppet/pull/561

This version also accepts all 2xx http codes, not just 200.

#9 Updated by Daniel Pittman about 2 years ago

  • Status changed from Accepted to Merged - Pending Release
  • Target version changed from 2.7.x to 2.7.12

#10 Updated by Matthaus Owens almost 2 years ago

  • Status changed from Merged - Pending Release to Closed
  • Target version changed from 2.7.12 to 2.7.14

Released in Puppet 2.7.14rc1

Also available in: Atom PDF