summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
54423d0)
There was a rare race where if a client disconnected during the
initial Telnet option negotiation pause, an attempt to log an entry
about it would raise an exception because the "initial" state
placeholder in the default menu data was incorrectly-named. Fix
that, and rename the state to "telopt_negotiation" so it's less
confusing to debug if similar issues are discovered with this state
in the future.
self.output_queue = []
self.partial_input = b""
self.password_tries = 0
self.output_queue = []
self.partial_input = b""
self.password_tries = 0
+ self.state = "telopt_negotiation"
self.telopts = {}
def quit(self):
self.telopts = {}
def quit(self):
self.check_idle()
# if output is paused, decrement the counter
self.check_idle()
# if output is paused, decrement the counter
- if self.state == "initial":
+ if self.state == "telopt_negotiation":
if self.negotiation_pause:
self.negotiation_pause -= 1
else:
if self.negotiation_pause:
self.negotiation_pause -= 1
else:
menu.entering_new_password.error_differs: The two passwords did not match. Try
again...
menu.entering_new_password.error_differs: The two passwords did not match. Try
again...
+menu.telopt_negotiation.prompt:
menu.main_utility.action_c: user.new_avatar()
menu.main_utility.branch_a: activate_avatar
menu.main_utility.action_c: user.new_avatar()
menu.main_utility.branch_a: activate_avatar