Skip to content

Install directory permissions broken on non-English Windows systems #39224

Description

@DvdGiessen
  • Version: 16.4.1
  • Platform: Windows

What steps will reproduce the bug?

  • Install Node v16.4.1 on a non-English Windows system.

How often does it reproduce? Is there a required condition?

It always reproduces as far as I can tell.

What do you see?

afbeelding

Additional information

I presume this bug was introduced in e817ba7.

Core issue seems to be that Windows localizes names of built-in users and groups. On my non-English Windows system, there does not exist a group called "Authenticated Users". It exists using a localized name. Hence, the solution implemented in that commit will never work on non-English Windows systems.

Instead, we should probably use a SID to reference the correct group. See https://support.microsoft.com/kb/243330.

EDIT: Presumably also affects v14.17.2 and v12.22.2, as these seem to include this same change. Haven't checked.

EDIT2: To clarify; this completely breaks Node.js installs, make it unable to complete installation, and when upgrading will leave Node.js in a broken state.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    confirmed-bugIssues and PRs for confirmed bugs.installIssues and PRs related to installing Node.js and its installers.windowsIssues and PRs related to the Windows platform.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions