Skip to content

Commit

Permalink
Do not clobber existing temporary files
Browse files Browse the repository at this point in the history
closes #103
  • Loading branch information
dougwilson committed Jan 19, 2015
1 parent da25538 commit 1af9213
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion index.js
Original file line number Diff line number Diff line change
Expand Up @@ -631,7 +631,7 @@ function handleFile(self, fileStream) {
fileStream.on('error', function(err) {
self.handleError(err);
});
fs.open(publicFile.path, 'w', function(err, fd) {
fs.open(publicFile.path, 'wx', function(err, fd) {
if (err) return self.handleError(err);
var slicer = fdSlicer.createFromFd(fd, {autoClose: true});

Expand Down

4 comments on commit 1af9213

@andrewrk
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@dougwilson
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to release whenever, or I can :) I'm thinking at least a feature release rather than a patch release, but I really don't know :)

@andrewrk
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd consider this a bug fix. Release is all yours.

@dougwilson
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done as 4.1.1 :D

Please sign in to comment.