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

If TouchOSC /ping enabled, Synergize may crash #15

Closed
chinenual opened this issue Nov 7, 2020 · 2 comments
Closed

If TouchOSC /ping enabled, Synergize may crash #15

chinenual opened this issue Nov 7, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@chinenual
Copy link
Owner

chinenual commented Nov 7, 2020

Synergize does not properly filter messages TouchOSC can be configured to send (/accxyx, /z and /ping). This results in messages such as the following in the log:

2020/11/06 20:20:03 ERROR:  SendToUI failed to decode json response : accxyz 0: unexpected end of JSON input

In my testing I was able to crash Synergize by flooding it with /accxyz accelerometer messages; others have reported network related errors (e.g. CONNRESET) which may be related. See #14

Until fixed, workaround is to ensure that these are disabled in TouchOSC (Options->OSC-> all toggles should be off)

@chinenual chinenual self-assigned this Nov 7, 2020
@chinenual chinenual added the bug Something isn't working label Nov 7, 2020
@chinenual
Copy link
Owner Author

Fixed for version 2.2.1. Added a blacklist filter to ignore /ping, /accxyz and /z messages from TouchOSC

@chinenual
Copy link
Owner Author

FWIW, it appears that the crash was caused by /ping messages from TouchOSC -- the accelerometer and touch messages are "harmless" (?), but flood the logs with errors and cause unnecessary traffic between the app and the UI

@chinenual chinenual changed the title If TouchOSC Accelerometer enabled, Synergize may crash If TouchOSC /ping enabled, Synergize may crash Nov 7, 2020
@chinenual chinenual removed their assignment Nov 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant