rack-devel archive mirror (unofficial) https://groups.google.com/group/rack-devel
 help / color / mirror / Atom feed
* patch for LSWS handler
@ 2009-10-23  5:00 george
  2009-10-23  8:51 ` Michael Fellinger
  0 siblings, 1 reply; 2+ messages in thread
From: george @ 2009-10-23  5:00 UTC (permalink / raw)
  To: Rack Development


Hi,

The LSWS handler is broken in Rack 1.0.1, the following patch should
fix it.

Thanks,
George Wang



*** lsws.rb     Fri Oct 23 00:49:41 2009
--- lsws.rb.new Fri Oct 23 00:52:10 2009
***************
*** 1,5 ****
--- 1,6 ----
  require 'lsapi'
  require 'rack/content_length'
+ require 'rack/rewindable_input'

  module Rack
    module Handler
***************
*** 38,43 ****
--- 39,46 ----
          ensure
            body.close if body.respond_to? :close
          end
+       ensure
+         rack_input.close
        end
        def self.send_headers(status, headers)
          print "Status: #{status}\r\n"

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: patch for LSWS handler
  2009-10-23  5:00 patch for LSWS handler george
@ 2009-10-23  8:51 ` Michael Fellinger
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Fellinger @ 2009-10-23  8:51 UTC (permalink / raw)
  To: rack-devel


On Fri, Oct 23, 2009 at 2:00 PM, george <gwanglst@gmail.com> wrote:
>
> Hi,
>
> The LSWS handler is broken in Rack 1.0.1, the following patch should
> fix it.

Applied with:
http://github.com/rack/rack/commit/cd401dfd01dbd8276ae817af2089c8c4d9fda6c4

>
> Thanks,
> George Wang
>
>
>
> *** lsws.rb     Fri Oct 23 00:49:41 2009
> --- lsws.rb.new Fri Oct 23 00:52:10 2009
> ***************
> *** 1,5 ****
> --- 1,6 ----
>  require 'lsapi'
>  require 'rack/content_length'
> + require 'rack/rewindable_input'
>
>  module Rack
>    module Handler
> ***************
> *** 38,43 ****
> --- 39,46 ----
>          ensure
>            body.close if body.respond_to? :close
>          end
> +       ensure
> +         rack_input.close
>        end
>        def self.send_headers(status, headers)
>          print "Status: #{status}\r\n"
>



-- 
Michael Fellinger
CTO, The Rubyists, LLC
972-996-5199

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-10-23  8:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-10-23  5:00 patch for LSWS handler george
2009-10-23  8:51 ` Michael Fellinger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).