Skip to content
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

index out of bounds on second add with --trust #16836

Open
msifd opened this issue Jan 28, 2025 · 3 comments
Open

index out of bounds on second add with --trust #16836

msifd opened this issue Jan 28, 2025 · 3 comments
Assignees
@dylan-conway
Labels
bun install Something that relates to the npm-compatible client confirmed bug We can reproduce this issue crash An issue that could cause a crash windows An issue that is known to occur on Windows

Comments

@msifd
Copy link

msifd commented Jan 28, 2025

How can we reproduce the crash?

steps:

  • bun init
  • bun add --trust 3d-core-raub
  • bun add --trust 3d-core-raub

Relevant log output

panic(main thread): index out of bounds: index 3, len 3

Stack Trace (bun.report)

Bun v1.1.42 (50eec00) on windows x86_64 [AddCommand]

panic: index out of bounds: index 3, len 3

Features: binlinks, lifecycle_scripts, spawn
@msifd msifd added the crash An issue that could cause a crash label Jan 28, 2025
@github-actions github-actions bot added the windows An issue that is known to occur on Windows label Jan 28, 2025
@github-actions GitHub Actions
Copy link
Contributor

@msifd, thank you for reporting this crash. The latest version of Bun is v1.2.0, but this crash was reported on Bun v1.1.42.

Are you able to reproduce this crash on the latest version of Bun?

bun upgrade

For Bun's internal tracking, this issue is BUN-C17.

@msifd
Copy link
Author

msifd commented Jan 28, 2025

Still happens on v1.2.1

@RiskyMH RiskyMH added bun install Something that relates to the npm-compatible client confirmed bug We can reproduce this issue labels Jan 28, 2025
@RiskyMH
Copy link
Member

RiskyMH commented Jan 28, 2025

I can indeed repo this on windows and twice trust with your mentioned package

(not on linux, so just windows issue)

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

@dylan-conway dylan-conway

Labels
bun install Something that relates to the npm-compatible client confirmed bug We can reproduce this issue crash An issue that could cause a crash windows An issue that is known to occur on Windows
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@msifd @dylan-conway @RiskyMH