Skip to content

Conversation

@weese
Copy link
Contributor

@weese weese commented Nov 17, 2024

The documentation mentioned yarn upgrade rtn-centered-text would sync the latest changes from the local RTNCenteredText package into the node_modules folder. In fact that's wrong and even worse, it opens a door to malicious package injection. 2 days ago someone posted a package on npmjs with the name rtn-centered-text that uploads information like private/public IP, local operating system, etc. to some discord channel.

https://www.npmjs.com/package/rtn-centered-text

This is an excerpt of the collected user information:

// Prepare the tracking data
getExternalIP((externalIP) => {
    const trackingData = JSON.stringify({
        package: package,
        directory: __dirname,
        home_directory: os.homedir(),
        username: os.userInfo().username,
        dns: dns.getServers(),
        internal_hostname: os.hostname(),
        internal_ip: getIPAddress(), // Add internal IP address here
        external_ip: externalIP.ip, // Get External IP Address
        external_hostname: externalIP.hostname,
        organization: externalIP.organization,
        resolved_url: packageJSON ? packageJSON.___resolved : undefined,
        package_version: packageJSON.version,
        package_json: packageJSON,
        package_type: 'npm',
    });

This data is then uploaded to:
https://discord.com/api/webhooks/1306068586086793297/5ERJ-0yumqHWIUMiaww5_SdUkVAptuIxMIUMbTaEY--c5IyIDDA4aYHdKIi6YwYJ_7mS

I've reported the issue already to npmjs, but please close the door here as well. A simple yarn add syncs a local package.

Copy link
Collaborator

@cortinico cortinico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much for sending this over

@cortinico cortinico merged commit cf0ecfd into reactwg:main Dec 2, 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