Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Unknown reference to "status" in a log message. #45

Closed
xx12345798 opened this issue Jan 7, 2025 · 0 comments · Fixed by #44
Closed

Unknown reference to "status" in a log message. #45

xx12345798 opened this issue Jan 7, 2025 · 0 comments · Fixed by #44
Labels
bug Something isn't working

Comments

@xx12345798
Copy link
Contributor

xx12345798 commented Jan 7, 2025

Bug

In eb4b780, a log statement was added to log the reason behind a vnc.create_connection() failure when the reason is not empty (a failure occurs), which would cause an exception. Later in 5df9698, an attempt was made to correct the name as status, but an conversion pattern (i.e., %s) was still missing in the log message. As a result, an exception would still occur when the reason is not empty.

iCtrl version

5df9698

Environment

Windows

Reproduction steps

Observed in file https://github.com/junhaoliao/iCtrl/blob/main/application/routes/vnc.py#L107

@xx12345798 xx12345798 added the bug Something isn't working label Jan 7, 2025
@xx12345798 xx12345798 changed the title Error log message in application/routes/vnc.py line 107 not properly created Unknown reference in log message Jan 8, 2025
@junhaoliao junhaoliao changed the title Unknown reference in log message Unknown reference to "status" in a log message. Jan 8, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
1 participant