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.

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

Sentry getting repeated reports of "the len is 1 but the index is 1" #656

Closed
qdot opened this issue Sep 1, 2024 · 2 comments
Closed

Sentry getting repeated reports of "the len is 1 but the index is 1" #656

qdot opened this issue Sep 1, 2024 · 2 comments
Assignees

Comments

@qdot
Copy link
Member

qdot commented Sep 1, 2024

We're getting absolutely spammed on sentry with the error message "the len is 1 but the index is 1"

From a user on discord, the top of the crash stack is:

"thread 'intiface-thread-5' panicked at 'index out of bounds: the len is 1 but the index is 1': C/Users/qdot/.cargo/registry/src/index.crates.io-6f17d22bba15001f/buttplug-8.0.0/src/server/device/server_device.rs:467"

This relates to intiface/intiface-central#139

@qdot qdot added the bug label Sep 1, 2024
@qdot qdot self-assigned this Sep 1, 2024
@qdot
Copy link
Member Author

qdot commented Sep 1, 2024

Line in question. This has to do with converting message attributes to features, a system that was rewritten in the upcoming Buttplug 8.0.1 where we never deal with message attributes within the server code anymore, so hopefully it's fixed there. We'll see, I suppose.

https://github.com/buttplugio/buttplug/blob/master/buttplug/src/server/device/server_device.rs#L467

@qdot
Copy link
Member Author

qdot commented Nov 16, 2024

This is probably #668 but that has more details and a fix, so dup'ing this one.

@qdot qdot closed this as completed Nov 16, 2024
@qdot qdot added the duplicate label Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant