Skip to content

doc: clarify limitations of path.isAbsolute #61576

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.

Sign up for GitHub

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

Jump to bottom
Open
dellson-arch wants to merge 2 commits into
base: main
Choose a base branch
from
Open

doc: clarify limitations of path.isAbsolute #61576

dellson-arch wants to merge 2 commits into from
+4 −3

Conversation

@dellson-arch
Copy link

@dellson-arch dellson-arch commented Jan 29, 2026

This PR clarifies the documentation for path.isAbsolute() by explicitly stating
that it only checks whether a path is absolute and does not validate whether the
path is safe to use.

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. path Issues and PRs related to the path subsystem. labels Jan 29, 2026
@dellson-arch
Copy link
Author

dellson-arch commented Jan 29, 2026

Thanks for reviewing! Happy to update wording or make changes if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cjihrig cjihrig cjihrig approved these changes

Assignees

No one assigned

Labels

doc Issues and PRs related to the documentations. path Issues and PRs related to the path subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dellson-arch @cjihrig @nodejs-github-bot