FileSaver replaces LF with CRLF

[agreif]

On windows XP (python 2.4.2) the latest svn checkout produces the following bug: FileSaver converts LF to CRLF automatically, even in binary files!!

I tried to upload files with the following code:

child_uploadfile = static.FileSaver(destination='d:/agreif/tmp',

expectedFields=filename?, maxBytes=10000000,

allowedTypes=[http_headers.MimeType('text', 'x-python'),

http_headers.MimeType('application','x-zip-compressed')])

here is the hexdump of two samples. You see that in front of every occurrance of "0A" a "0D" is inserted.

Sample 1: textfile orig: 68616C6C6F0A uploaded: 68616C6C6F0D0A

Sample 2 binary file (zip) orig: 504B03040A00000000001589FF34D13172B90500000005000000100000004E6575205465787464617465692E707968616C6C6F504B010214000A00000000001589FF34D13172B905000000050000001000000000000000010020000000000000004E6575205465787464617465692E7079504B050600000000010001003E000000330000000000 uploaded: 504B03040D0A00000000001589FF34D13172B90500000005000000100000004E6575205465787464617465692E707968616C6C6F504B010214000D0A00000000001589FF34D13172B905000000050000001000000000000000010020000000000000004E6575205465787464617465692E7079504B050600000000010001003E000000330000000000

[Cory Dodt]

Failing test in test_static to expose this problem

[Cory Dodt]

I thought this would be a simple matter of changing 'w' to 'wb' in static.py, but when I made that change the test still fails, so the stream is producing bad data.

[Glyph]

Since file uploads are such a major new feature of web2 (ostensibly the reason for its existence) I'd say that this is important to its taking over from web...

[therve]

This problem seems to exist under windows only.

[dialtone]

I think I fixed this bug in filesaver-1979 branch. It's now ready for review.

[dialtone]

small changes but still set for review

[jknight]

Looks fine on windows now; I noticed that docstring for writeFile mentions the nonexistent isWriteable instead of isSafeToWrite.

[dialtone]

(In [20546]) static.FileSaver() doesn't add '\r\n' when run on windows anymore.

Author: dialtone

Reviewer: foom

Fixes #1979