[ Adding the development list to Cc, to facilitate discussion. ]

"David F. Newman" <[EMAIL PROTECTED]> writes:

>> First of all, I think this new behaviour needs an option to enable
>> it, rather than be on by default. The option could be called
>> "rfc1806", or rather, "rfc2183" now, unless anyone can suggest a
>> friendlier name such as "--obey-content-disposition-filename"!

Even better, I would entirely avoid the rfc numbers in naming either
command-line options *or* functions.  Mentioning rfc-s in comments or
in the manual is fine, of course.

> OK, I'll take these things into account.  My concern with the valid
> characters was that someone doesn't specify an absolute path and
> pass something like /etc/passwd.  So you think that stripping out
> the leading path is enough?  That shouldn't be to tough.

I'm not sure what you mean by "stripping out the leading part," but
what I suggest is to leave only the trailing part.  So if someone
specifies "/etc/passwd", it's exactly the same as if he specified just
"passwd".

> And how about simply --honor-content-disposition

Ian, why do you think this should not be allowed by default?  A
command-line option is easy to miss, and honoring this looks like a
neat idea.  Am I missing something?

Reply via email to