Skip to content

free PackageStats.archive_path at call sites#34

Open
versecafe wants to merge 1 commit into
vercel-labs:mainfrom
versecafe:fix-package-archive-path-leak
Open

free PackageStats.archive_path at call sites#34
versecafe wants to merge 1 commit into
vercel-labs:mainfrom
versecafe:fix-package-archive-path-leak

Conversation

@versecafe
Copy link
Copy Markdown

  • technically only one of these leaks right now but the others will leak without this if they ever set .archive = true

@vercel
Copy link
Copy Markdown

vercel Bot commented May 13, 2026

@versecafe is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@versecafe
Copy link
Copy Markdown
Author

versecafe commented May 13, 2026

same style of issue as #33 fixes (#32) but this path is actually in use through printDiagnostic

- technically only one of these leaks right now but the others will leak
  without this if they ever set `.archive = true`
@versecafe versecafe force-pushed the fix-package-archive-path-leak branch from 3923a3e to d597e5f Compare May 14, 2026 04:23
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