Bump log levels - Githubissues

mrtmm commented 3 years ago

Hi Mohab,

We'd like to suggest increasing the logging level on a few occasions:

Log the establishment and closure of a connection with info rather than debug, as those events might be helpful to know about for timing purposes, rather than for actual debugging.
Log "failed to receive instruction" as a warning, since it may point to networking issues that prematurely drop the connection.
Log "invalid protocol and no connectionid provided" as an error, since it raises an exception immediately afterwards.

mohabusama commented 3 years ago

@mrtmm Thanks! lgtm

Hopefully it won't be too verbose.

mohabusama commented 3 years ago

I will try to publish a new version some time today.

mohabusama commented 3 years ago

Changes available in 0.10.

mrtmm commented 3 years ago

Thats great! Thanks for merging! :)

mohabusama / pyguacamole