Skip to content

[v13.x] fs: fix writeFile[Sync] for non-seekable files - #32172

Closed
mildsunrise wants to merge 1 commit into
nodejs:v13.x-stagingfrom
mildsunrise:v13.x-writefile-non-seekable
Closed

[v13.x] fs: fix writeFile[Sync] for non-seekable files#32172
mildsunrise wants to merge 1 commit into
nodejs:v13.x-stagingfrom
mildsunrise:v13.x-writefile-non-seekable

Conversation

@mildsunrise

Copy link
Copy Markdown
Contributor

Backport of #32006

Completely disables the use of positioned writes at
writeFile and writeFileSync, which allows it to work
with non-seekable files.
Fixes: nodejs#31926
PR-URL: nodejs#32006
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@nodejs-github-botnodejs-github-bot added fs Issues and PRs related to the fs subsystem / file system. v13.x labels Mar 10, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

MylesBorins pushed a commit that referenced this pull request Mar 10, 2020
Completely disables the use of positioned writes at
writeFile and writeFileSync, which allows it to work
with non-seekable files.
Fixes: #31926
Backport-PR-URL: #32172
PR-URL: #32006
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@MylesBorins

Copy link
Copy Markdown
Contributor

landed in 1c4f4cc

@mildsunrise
mildsunrise deleted the v13.x-writefile-non-seekable branch March 10, 2020 21:17
targos pushed a commit that referenced this pull request Apr 20, 2020
Completely disables the use of positioned writes at
writeFile and writeFileSync, which allows it to work
with non-seekable files.
Fixes: #31926
Backport-PR-URL: #32172
PR-URL: #32006
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fsIssues and PRs related to the fs subsystem / file system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@mildsunrise@nodejs-github-bot@MylesBorins