]> git.koha-community.org Git - koha.git/commit
Bug 15006: Centralize timeout logic and allow zero client timeout
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Wed, 6 Jul 2016 14:14:45 +0000 (15:14 +0100)
committerJulian Maurice <julian.maurice@biblibre.com>
Wed, 17 Aug 2016 09:56:47 +0000 (11:56 +0200)
commitc6f1fa2aad6469e9c4221f903ea571e259a92e4f
treed1986e6b28e3e6da9f9c13a27d728342804e66db
parentef3c9078f94446783ef3ac9b7e21ab935a685636
Bug 15006: Centralize timeout logic and allow zero client timeout

Moving timeout logic to one routine (with unit test).

This further implements two suggestions from Kyle and Larry:

[1] You could use a client_timeout of 0 to specify no timeout at all.
[2] Have the client_timeout default to the timeout if not defined.

Test plan:
[1] Run t/db_dependent/SIP/SIPServer.t.
[2] Test login timeout for raw and telnet.
[3] Check ACS status message for timeout value. Should match policy
    timeout from institution.
[4] Test client timeout (zero and non-zero).
[5] Remove client timeout. Test fallback to service.
[6] Remove service timeout too. Test fallback to 30 at login.

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Amended to incorporate Srdjan's suggestion to move get_timeout to
SIPServer.pm; this requires some additional mocking in the unit test.
And even makes the test db dependent, as documented.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit eb8666357ce72e910569843f8acce674a332f4b0)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
(cherry picked from commit e8a03065affb09718d16781a8beecfa262497639)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
C4/SIP/SIPServer.pm
C4/SIP/Sip/MsgType.pm
t/SIP/Sip.t [new file with mode: 0755]
t/SIP_Sip.t [deleted file]
t/db_dependent/SIP/SIPServer.t [new file with mode: 0755]