Refactor #1909

Add 'rack' http handler for RESTful support

Added by Paul Nasrat over 1 year ago. Updated 4 months ago.

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

0%
Category:-
Target version:0.25.0
Affected version:0.24.7 Branch:
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 over 1 year 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 over 1 year 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 over 1 year ago

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

#4
Updated by Luke Kanies over 1 year 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 over 1 year 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 over 1 year 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 over 1 year ago

  • Status changed from Ready for Checkin to Closed

Pushed in commit:“6e01e7ab403d090f29f13c938ca5b19930c4b408” in branch master.

Also available in: Atom PDF