Server freezes when incorrect stream key used
Bruno-DaSilva opened this issue · comments
It seems when someone includes a bad stream key the whole app freezes and needs to be restarted.
I think this might be related to the lack of unlock before return on https://github.com/zorchenhimer/MovieNight/blob/master/handlers.go#L376.
Is it safe to add a l.Unlock()
before this return? Not sure if there's some side effects of simply returning to deny a stream.
See #147
Confirmed that the patch errors with incorrect stream keys while no longer freezing the app. I'm assuming it works for the other error cases there as well.
Should be all set now.