8000 Feature: file read/write streams by miguel-cagide · Pull Request #7 · awo00/smb2 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Feature: file read/write streams #7

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.

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

Merged
merged 10 commits into from
Mar 20, 2024

Conversation

miguel-cagide
Copy link
  • Added functionality to read a file as a stream.
  • Added functionality to write a file as a stream.
  • Stream read example:
  const fileWriteStream = fs.createWriteStream(localPath);
  const readStream = tree.createFileReadStream(path);
  
  readStream.pipe(fileWriteStream);
  • Stream write example:
const fileReadStream = fs.createReadStream(localPath);
const writeStream = tree.createFileWriteStream(path);

fileReadStream.pipe(writeStream);

miguel-cagide and others added 8 commits January 4, 2024 13:44
* feature: basic readStream

* fix: remove unneccesary return

* fix: remove unused buffer variable

* feature: createFileReadableStream on tree

* feature: working read/write streams

---------

Co-authored-by: Miguel Angel Cagide Fagin <miguelacfa@ext.inditex.com>
* feature: subchunks on writefilestream when chunk is bigger than "maxWriteChunkLength"
fix: "close" event on readfilestream

* chore: version bump, doc update
* stop emit error on close

* version

* fix: fixed subarray positions when chunk bigger than maxWriteChunkLength

---------

Co-authored-by: Andy Wooldridge <99414101+awo00@users.noreply.github.com>
@miguel-cagide miguel-cagide changed the title Feature: file read write streams Feature: file read/write streams Jan 8, 2024
fix: remove miguel-cagide changes
fix: added missing new line
@awo00 awo00 merged commit ab51182 into awo00:master Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0