Skip to content

feat: add config option to allow symlinks (#1372)#1378

Merged
gilescope merged 1 commit into
release/node-1.0.0from
ozgb-allow-symlinks-option
Apr 21, 2026
Merged

feat: add config option to allow symlinks (#1372)#1378
gilescope merged 1 commit into
release/node-1.0.0from
ozgb-allow-symlinks-option

Conversation

@ozgb

@ozgb ozgb commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Overview

Same as #1372, re-based onto release/node-1.0.0

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

* feat: add UNSAFE_ALLOW_SYMLINKS option for safe-read function

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>

* docs: add change file

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>

* chore: add pr link to change file

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>

* chore: ignore clippy warn

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>

* test: add another symlink test

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>

---------

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>
@ozgb ozgb requested a review from a team as a code owner April 21, 2026 15:14
@gilescope gilescope merged commit 9d60fe3 into release/node-1.0.0 Apr 21, 2026
35 of 41 checks passed
@gilescope gilescope deleted the ozgb-allow-symlinks-option branch April 21, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants