Refactor #1909

Add 'rack' http handler for RESTful support

Added by Paul Nasrat about 3 years ago. Updated almost 2 years ago.

Status:Closed Start date:01/27/2009
Priority:High Due date:
Assignee:James Turnbull % Done:

0%
Category:-
Target version:0.25.0
Affected Puppet version:0.24.7 Branch:
Keywords:
Votes: 0

Description

Now rack is mature and we already have an implementation for xmlrpc, we should consolidate our network handling and delegate the responsibility to rack. This gives us a thin but clear separation of the application and the point of communication, enabling future development.

History

#1
Updated by Luke Kanies about 3 years ago

  • Status changed from Unreviewed to Accepted
  • Priority changed from Normal to High

Rack support wasn’t ported over to the new system, so we need this for ‘Passenger’ support, and it would make our lives much easier.

#2
Updated by Luke Kanies almost 3 years ago

  • Subject changed from Eliminate multiple http handlers and use rack as abstraction layer to Add 'rack' http handler for RESTful support

Renaming the ticket, since we’ll always at least have webrick, for the default, no-dependency case.

#3
Updated by James Turnbull almost 3 years ago

Status? This is going to be bumped unless someone updates shortly.

#4
Updated by Luke Kanies almost 3 years ago

I’m not comfortable bumping this; people are using Passenger today, and I don’t want to put out a release without it.

I’m quite comfortable putting out a beta without it, though.

#5
Updated by Christian Hofstaedtler almost 3 years ago

I’m sorry, have been quite busy this week. Hopefully I can send updated patches for this today/tomorrow.

#6
Updated by Christian Hofstaedtler almost 3 years ago

  • Status changed from Accepted to Ready For Checkin
  • Assignee changed from Paul Nasrat to James Turnbull

I’ve incorporated all comments and rebased against current master. Please find the updated code on github, git://github.com/zeha/puppet-rack.git branch is ticket/1909 .

#7
Updated by James Turnbull almost 3 years ago

  • Status changed from Ready For Checkin to Closed

Pushed in commit:6e01e7ab403d090f29f13c938ca5b19930c4b408 in branch master.

Also available in: Atom PDF