Skip to content

Node -> 18#405

Open
AprilArcus wants to merge 1 commit intographile:mainfrom
AprilArcus:node-18
Open

Node -> 18#405
AprilArcus wants to merge 1 commit intographile:mainfrom
AprilArcus:node-18

Conversation

@AprilArcus
Copy link
Copy Markdown

@AprilArcus AprilArcus commented Apr 4, 2026

Update Node to version 18.

  • Node 18 changes the resolution of localhost to ::1 (ipv6) instead of 127.0.0.1 (ipv4). Tell Docker to map port 5432 (postgres) on ipv6 instead of ipv4.

  • Node 18 no longer spawns yarn.cmd with { shell: true } by default, so in scripts/lib/run.js, we now set it explicitly if we detect that the OS is win32.

@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 4, 2026

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn Critical
Critical CVE: Authorization Bypass in Next.js Middleware

CVE: GHSA-f82v-jwr5-mffw Authorization Bypass in Next.js Middleware (CRITICAL)

Affected versions: >= 13.0.0 < 13.5.9; >= 14.0.0 < 14.2.25; >= 15.0.0 < 15.2.3; >= 12.0.0 < 12.3.5

Patched version: 13.5.9

From: @app/client/package.jsonnpm/next@13.2.3

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/next@13.2.3. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn High
Obfuscated code: npm zxcvbn is 98.0% likely obfuscated

Confidence: 0.98

Location: Package overview

From: @app/lib/package.jsonnpm/zxcvbn@4.4.2

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/zxcvbn@4.4.2. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@AprilArcus AprilArcus force-pushed the node-18 branch 2 times, most recently from cf86eb4 to 2cdc3e8 Compare April 6, 2026 23:56
@AprilArcus AprilArcus changed the title CI: Node -> 18 Node -> 18 Apr 7, 2026
@AprilArcus AprilArcus mentioned this pull request Apr 7, 2026
- Node 18 changes the resolution of `localhost` to ::1 (ipv6) instead of
  127.0.0.1 (ipv4). In CI, we must tell Docker to map port 5432 (postgres)
  on ipv6 instead of ipv4.

- Node 18 no longer spawns yarn.cmd with { shell: true } by default, so
  in `scripts/lib/run.js`, we now set it explicitly if we detect that
  the OS is win32.
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.

1 participant