Question about parse_multipart()

3 views
Skip to first unread message

beam

unread,
Nov 20, 2009, 3:06:05 AM11/20/09
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}

beam

unread,
Nov 20, 2009, 3:27:32 AM11/20/09
to Rack Development

Mark Goris

unread,
Nov 20, 2009, 7:14:42 PM11/20/09
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...
Reply all
Reply to author
Forward
0 new messages