asl97

Just realized `onConnectFail()` is also called when trying to connect to a non existing room. If `onConnectFail()` is being used to blacklist bad room names then it definitely shouldn't be...

~~one of the socket is probably just close.~~ ~~[the disconnect in the room doesn't `None` the _sock variable](https://github.com/Nullspeaker/ch.py/blob/master/ch.py#L772)~~ ~~[the getConnections doesn't filter closed sockets](https://github.com/Nullspeaker/ch.py/blob/master/ch.py#L2123)~~ oh wait... the room object is...

Fixed in https://github.com/nhammond129/ch.py/commit/995306bd14c77863e1aec3889fd9642414ccc674 with improvement in tasks handling

`JoeStaines: Any reason why unicode cant be support through this change?` windows & python2 iirc https://github.com/Nullspeaker/ch.py/issues/2

it break python2 and windows stuff

As far as I am aware of, we are now decoding without specifying the codec and it default to UTF-8 on python 3 https://docs.python.org/3/library/stdtypes.html#bytes.decode Assuming not a issue anymore since...

i blame chatango and maybe the stdlib in any case, this isn't a issue cause by ch.py

@LordYusei the original ch.py support python 2 and python 3 the last time i check, this "updated" copy also support python 2 and python3 although someone could have added python...

@LordYusei you should check the website before replying if possible cause when someone edit their post, github doesn't send any message about the edit. in any case, this is where...

wrapping the whole bot in a `try except` just to capture the error and display it is a flaw method, it's one of the reason why the old botteh code...