Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

handle big files #14

Closed
wants to merge 1 commit into from

3 participants

@juliangruber

It might happen that multiple buffers are sent on a push. This fixes the buffered array not being available after the first buffer was received

@juliangruber

this doesn't fix everything, it still breakes on even bigger files...

Writing objects: 17% ...

@distracteddev

Any idea how to fix this when pushing "even bigger files". Trying to push a large existing repo will always fail.

@juliangruber juliangruber referenced this pull request
Closed

Fix upload-pack regex #15

@substack
Owner

The bug that this pull request fixes may have been finally solved in 1.1.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 19, 2012
  1. @juliangruber

    reset buffer to array

    juliangruber authored
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/service.js
View
2  lib/service.js
@@ -81,7 +81,7 @@ function Service (opts, req, res) {
buffered.forEach(function (buf) {
ps.stdin.write(buf);
});
- buffered = undefined;
+ buffered = [];
self.pipe(ps.stdin);
ps.on('exit', self.emit.bind(self, 'exit'));
Something went wrong with that request. Please try again.