-
-
Notifications
You must be signed in to change notification settings - Fork 45
Description
I'd love to use this library to generate other content-dispositions headers in multipart messages. One of the use cases is form-data, where the individual fields have a content-disposition of form-data; name=fieldNameGoesHere
.
This module has no problem parsing that, but it blows up when you try to generate a header without a filename.
To maintain backwards-compatibility as much as possible, I'd suggest making the filename
argument optional if an options
argument is provided instead, i.e.:
contentDisposition({type: 'form-data', {parameters: name: 'fieldNameGoesHere'}})
(Side note: I'd love to have the API actually be type, parameters
rather than filename, options
, but that would not only break backwards-compatibility, it'd also make the module less convenient for its original purpose (i.e. generating content-disposition headers for file attachments). I can only imagine this as a major version update with the old API as an additional method.)