rack-devel archive mirror (unofficial) https://groups.google.com/group/rack-devel
 help / color / mirror / Atom feed
* Question about parse_multipart()
@ 2009-11-20  8:06 beam
  2009-11-20  8:27 ` beam
  0 siblings, 1 reply; 3+ messages in thread
From: beam @ 2009-11-20  8:06 UTC (permalink / raw)
  To: Rack Development

About commit http://github.com/rack/rack/commit/6674f3652ed19136802a0b84f1923f0b78052442

I read RFC? but not understand why we add this lines in lib/rack/
utils.rb:
330+                if content_type || filename

372+            elsif !filename && content_type
373+              body.rewind
374+
375+              # Generic multipart cases, not coming from a form
376+              data = {:type => content_type,
377+                      :name => name, :tempfile => body, :head =>
head}

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

* Re: Question about parse_multipart()
  2009-11-20  8:06 Question about parse_multipart() beam
@ 2009-11-20  8:27 ` beam
  2009-11-21  0:14   ` Mark Goris
  0 siblings, 1 reply; 3+ messages in thread
From: beam @ 2009-11-20  8:27 UTC (permalink / raw)
  To: Rack Development

Sorry. Already found an answer.
http://groups.google.com/group/rack-devel/browse_thread/thread/24a0fa8ff3dce487/fe887e8ab9cc8c3f?lnk=gst&q=multipart#fe887e8ab9cc8c3f

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

* Re: Question about parse_multipart()
  2009-11-20  8:27 ` beam
@ 2009-11-21  0:14   ` Mark Goris
  0 siblings, 0 replies; 3+ messages in thread
From: Mark Goris @ 2009-11-21  0:14 UTC (permalink / raw)
  To: Rack Development

I also added a ticket on Lighthouse.

http://rack.lighthouseapp.com/projects/22435/tickets/79-multipart-handling-incorrectly-assuming-file-upload

Jonathan was kind enough to suggest an approach to working around this
for now.  We both still believe this issue should be fixed in Rack,
though.

On Nov 20, 12:27 am, beam <ru...@rsl.ru> wrote:
> Sorry. Already found an answer.http://groups.google.com/group/rack-devel/browse_thread/thread/24a0fa...

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

end of thread, other threads:[~2009-11-23  6:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-20  8:06 Question about parse_multipart() beam
2009-11-20  8:27 ` beam
2009-11-21  0:14   ` Mark Goris

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).