github
/
fail2ban
mirror of https://github.com/fail2ban/fail2ban
1
0
Fork 0
Code Issues Releases Wiki Activity

Fix for missing value in transmitter delaction

pull/124/head
Steven Hiscocks 2013-02-20 23:24:46 +00:00
parent 088e40c481
commit b6a68f5138
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
server/transmitter.py
View File

@ -183,6 +183,7 @@ class Transmitter:
self.__server.addAction(name, value) self.__server.addAction(name, value)
return self.__server.getLastAction(name).getName() return self.__server.getLastAction(name).getName()
elif command[1] == "delaction": elif command[1] == "delaction":
value = command[2]
self.__server.delAction(name, value) self.__server.delAction(name, value)
return None return None
elif command[1] == "setcinfo": elif command[1] == "setcinfo":

7
testcases/servertestcase.py
View File

@ -433,9 +433,10 @@ class Transmitter(unittest.TestCase):
self.__transm.proceed( self.__transm.proceed(
["set", self.jailName, "delcinfo", action, "KEY"]), ["set", self.jailName, "delcinfo", action, "KEY"]),
(0, None)) (0, None))
#FIXME: This is broken in server.transmitter (value not defined)
"""
self.assertEqual( self.assertEqual(
self.__transm.proceed(["set", self.jailName, "delaction", action]), self.__transm.proceed(["set", self.jailName, "delaction", action]),
(0, None)) (0, None))
""" self.assertEqual(
self.__transm.proceed(
["set", self.jailName, "delaction", "Doesn't exist"]),
(0, None)) #NOTE: Should this return 1?