osu-atri / osu-al1s

A simple osu! bot framework written in Java
Apache License 2.0
2 stars 0 forks source link

`OsuChannelImpl#waitForNewMessages` causes dead lock. #6

Open astro-angelfish opened 2 months ago

astro-angelfish commented 2 months ago

https://github.com/osu-atri/osu-al1s/blob/adaeec09c4c08036fe804d27ac759ce4ed77efbd/osu-al1s-core/src/main/java/moe/orangemc/osu/al1s/chat/OsuChannelImpl.java#L98-L110

To avoid it we need to make a new thread instead of original thread. It seems that KICL appears to invoke our event handler on their message processing thread directly. Creating a new thread to do further processes might fix this.