Puppet: System Administration Automated

Support

Ticket #906 (closed enhancement: fixed)

Opened 1 year ago

Last modified 1 year ago

Mongrel Server working with Pound Proxy

Reported by: mccune Assigned to: mccune
Priority: normal Milestone:
Component: server Version:
Severity: normal Keywords: mongrel, server, puppetmaster, pound
Cc: Triage Stage: Ready for checkin
Attached Patches: Code Complexity: Trivial

Description

Puppetmaster does not work with with the Pound reverse HTTP proxy.

This ticket is for reference, patch to follow shortly.

Attachments

0001-Fixed-puppet-master-parsing-bug-when-mongrel-is-used.patch (1.8 kB) - added by mccune on 11/15/07 20:54:47.
0001-Pound-Single-Line-x509certificate-header.patch (2.0 kB) - added by mccune on 11/15/07 22:05:57.
Patch to Pound-2.3.2 to make it work with Mongrel. The Multi-line certificate is a problem for mongrel's HTTP parser.
0002-Pound-man-page.patch (0.6 kB) - added by mccune on 11/15/07 22:10:17.
Small change to the Pound Man Page.
puppet_pound_for_0.23.2.patch (1.3 kB) - added by mccune on 11/16/07 22:00:29.
Patch for Pound+Mongrel against 0.23.2 release.

Change History

11/15/07 20:41:17 changed by mccune

  • owner changed from community to mccune.
  • complexity changed from Unknown to Trivial.
  • type changed from defect to enhancement.

11/15/07 20:54:47 changed by mccune

  • attachment 0001-Fixed-puppet-master-parsing-bug-when-mongrel-is-used.patch added.

11/15/07 20:55:27 changed by mccune

  • stage changed from Unreviewed to Ready for checkin.
  • patch changed from None to Code.

11/15/07 22:05:57 changed by mccune

  • attachment 0001-Pound-Single-Line-x509certificate-header.patch added.

Patch to Pound-2.3.2 to make it work with Mongrel. The Multi-line certificate is a problem for mongrel's HTTP parser.

11/15/07 22:10:17 changed by mccune

  • attachment 0002-Pound-man-page.patch added.

Small change to the Pound Man Page.

11/16/07 22:00:29 changed by mccune

  • attachment puppet_pound_for_0.23.2.patch added.

Patch for Pound+Mongrel against 0.23.2 release.

(follow-up: ↓ 4 ) 11/19/07 06:37:20 changed by luke

Is there any way you can publish a git repo with all of these patches in them? That would make my life *much* easier.

(in reply to: ↑ 3 ) 11/19/07 15:56:52 changed by mccune

Replying to luke:

Is there any way you can publish a git repo with all of these patches in them? That would make my life *much* easier.

Sure. Would you like them to be in a branch named "master" or in another branch?

-Jeff

11/19/07 17:12:23 changed by luke

DavidS's method of having a per-fix branch has worked really well, although I was skeptical at first.

Just let me know the branch name, whatever it is. I'd prefer you not use 'master', though.

(follow-up: ↓ 7 ) 11/24/07 06:36:32 changed by luke

I'd definitely like to get this into misspiggy.

(in reply to: ↑ 6 ) 11/24/07 17:59:51 changed by mccune

Replying to luke:

I'd definitely like to get this into misspiggy.

Me too...

I still haven't really grokked git, but I'm working at it. I've made the patch to a branch called pending/fix-906, so the patch should be published and accessible in my repository. Please let me know how you pull and merge into the master branch, because I'm still trying to figure out if my repository is published correctly or not, and I'm testing merges as if you'd be performing them, if that makes any sense.

The commit is [a88891a5573785867667f6de0cdc539873455226]

Anyway, this should get you started: 

git remote add mccune http://northstarlabs.net/git/puppet

The branches should be there... 

% git branch -r
  mccune/master
  mccune/pending/fix-896
  mccune/pending/fix-906
  origin/HEAD
  origin/master

Let me know how it goes...

11/25/07 09:08:10 changed by luke

  • status changed from new to closed.
  • resolution set to fixed.

Applied.