Skip to content

Add additional multipart headers as an optional parameter #183

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
deathgrindfreak wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add additional multipart headers as an optional parameter #183

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
fb86698
Add additional multipart headers as an optional parameter
deathgrindfreak Oct 15, 2018
17e5af4
Remove includes method for older node.js versions
deathgrindfreak Oct 15, 2018
b4e29f2
Always pass in additional headers object
deathgrindfreak Oct 15, 2018
e67f154
Iterate over Object.keys of header, add all header values not just th…
deathgrindfreak Oct 17, 2018
99d6436
Node 0.10 doesn't support "for of", replace with forEach
deathgrindfreak Oct 17, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 16 additions & 2 deletions lib/types/multipart.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,7 @@ function Multipart(boy, cfg) {
field.removeAllListeners('end');
}

var additionalHeaders = {};
part.on('header', function(header) {
var contype,
fieldname,
Expand Down Expand Up @@ -169,6 +170,10 @@ function Multipart(boy, cfg) {
else
encoding = '7bit';

Object.keys(header).forEach(function(k) {
if (!isRequiredHeader(k)) additionalHeaders[k] = header[k];
});

var onData,
onEnd;
if (contype === 'application/octet-stream' || filename !== undefined) {
Expand Down Expand Up @@ -210,7 +215,8 @@ function Multipart(boy, cfg) {
cb();
}
};
boy.emit('file', fieldname, file, filename, encoding, contype);

boy.emit('file', fieldname, file, filename, encoding, contype, additionalHeaders);

onData = function(data) {
if ((nsize += data.length) > fileSizeLimit) {
Expand Down Expand Up @@ -258,7 +264,8 @@ function Multipart(boy, cfg) {
curField = undefined;
if (buffer.length)
buffer = decodeText(buffer, 'binary', charset);
boy.emit('field', fieldname, buffer, false, truncated, encoding, contype);

boy.emit('field', fieldname, buffer, false, truncated, encoding, contype, additionalHeaders);
--nends;
checkFinished();
};
Expand Down Expand Up @@ -317,6 +324,13 @@ function FileStream(opts) {

this.truncated = false;
}

function isRequiredHeader(h) {
return h === 'content-type'
|| h === 'content-disposition'
|| h === 'content-transfer-encoding';
}

inherits(FileStream, ReadableStream);

FileStream.prototype._read = function(n) {};
Expand Down