FileSaver cannot save as "path?query" on Windows #163

Closed
kitamomonga opened this Issue Nov 6, 2011 · 3 comments

Comments

Projects
None yet
2 participants
@kitamomonga
Contributor

kitamomonga commented Nov 6, 2011

Windows filesystem does not allow to use character "?" for filenames.
"path?query" would be "path@query" or something (like wget for Windows).

As a matter of fact, mechanize should escape/replace more and more characters for saving files to disk (see: wget and cURL and Content-Disposition Security Advisory), but I think it is just another (big) issue.

@drbrain

This comment has been minimized.

Show comment
Hide comment
@drbrain

drbrain Nov 7, 2011

Member

Found this advisory for curl: http://curl.haxx.se/docs/adv_20101013.html

Member

drbrain commented Nov 7, 2011

Found this advisory for curl: http://curl.haxx.se/docs/adv_20101013.html

@drbrain

This comment has been minimized.

Show comment
Hide comment
@drbrain

drbrain Nov 9, 2011

Member

Other than writing a non-existent special dot-file when mechanize is used in the home directory I think I can close this issue. Mechanize already avoids overwriting existing files when saving files (so you can't overwrite .login, etc.)

Member

drbrain commented Nov 9, 2011

Other than writing a non-existent special dot-file when mechanize is used in the home directory I think I can close this issue. Mechanize already avoids overwriting existing files when saving files (so you can't overwrite .login, etc.)

@drbrain drbrain closed this Nov 9, 2011

@kitamomonga

This comment has been minimized.

Show comment
Hide comment
@kitamomonga

kitamomonga Nov 9, 2011

Contributor

ContentDispositionParser!! awesome!
I am anxiously awaiting pre-release/2.1 version.

Contributor

kitamomonga commented Nov 9, 2011

ContentDispositionParser!! awesome!
I am anxiously awaiting pre-release/2.1 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment