return the destination stream, fixes issue#23 #25

Merged
merged 1 commit into from Sep 13, 2012

Conversation

Projects
None yet
3 participants
@substack
Contributor

substack commented Aug 28, 2012

If this gets merged then I can pipe filed to oppressor to do gzip/deflate compression like this:

var oppressor = require('oppressor');
var filed = require('filed');
var http = require('http');

var server = http.createServer(function (req, res) {
    filed(__dirname + '/data.txt').pipe(oppressor(req)).pipe(res);
});
server.listen(8000);

Issue: #23

@jcrugzz

This comment has been minimized.

Show comment
Hide comment
@jcrugzz

jcrugzz Sep 12, 2012

@substack the return dest; line in your fork disappeared somehow. Anyway you could fix that until this is merged?

jcrugzz commented Sep 12, 2012

@substack the return dest; line in your fork disappeared somehow. Anyway you could fix that until this is merged?

mikeal added a commit that referenced this pull request Sep 13, 2012

Merge pull request #25 from substack/return_dest
return the destination stream, fixes issue#23

@mikeal mikeal merged commit d91f706 into mikeal:master Sep 13, 2012

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