https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 --- Comment #1 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 120843 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120843&action=edit Bug 28320: Add DB connection check to the SIP SC status message This patch adds a lookup of the sip user during an SC status, confirming that our DB ocnnection is working, and that our user is still valid Additionally, it adds support for SC status to the sip_cli_emulator and adds basic test coverage for the SC status message To test: 1 - Apply patch 2 - Restart SP server 3 - perl misc/sip_cli_emulator.pl -a localhost -p 6001 -l CPL -m sc_status_request -su term1 -sp term1 4 - prove -v t/db_dependent/SIP/Message.t -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |joonas.kylmala@helsinki.fi -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 --- Comment #4 from David Cook <dcook@prosentient.com.au> --- Also, DBIx::Class should reconnect to the database, so I'm a bit confused by what's going on here... -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 --- Comment #6 from Nick Clemens <nick@bywatersolutions.com> --- (In reply to Martin Renvoize from comment #5)

I should have commented on the bug instead of asking Nick/Kyle directly.. I agree, the die feels weird and I'm surprised the db connection isn't retried.

(In reply to David Cook from comment #3)

What happens with that die()? Does that crash the server or just severs the connection?

The die simply kills the current connection, the server keeps running and can be reconnected to, to test: 1 - telnet localhost 6001 2 - input: 9300CNterm1|COterm1|CPCPL| 3 - Should get a 94 response 4 - Stop the DB in another terminal - docket stop koha_db_1 5 - input to telnet: 9900302.00 6 - "Connection closed by foreign host." 7 - Start the DB 8 - repeat 1-3 9 - input to telnet: 9900302.00 10 - You get a 98 response The DB connection should be retried, and generally is: 1 - telnet localhost 6001 2 - input: 9300CNterm1|COterm1|CPCPL| 3 - Should get a 94 response 4 - In another terminal: sudo koha-mysql kohadev 5 - SHOW FULL PROCESSLIST; 6 - You should see a connection that represents your SIP session above 7 - You can send a 99 SC status to reset the sleep/time so you can identify which one is your sip connection input to telnet: 9900302.00 8 - Once you have identified, in MySQL, kill the correct prcoess number: KILL 3 9 - Send the 99 again 10 - The DB reconnects and continues without interrupting the SIP session What we have seen is that for long standing SIP sessions, the DB won't reconnect gracefully. In these cases the SC statuses keep reporting connection, but the next checkin fails. After that failure the DB may reconnect and continue, but numbers of items can pass through the AMH/Self check without checking in. Pinging the DB connection eliminates the issue, as does a restart of the SIP server by forcing new connections. In the case where the SC status would fail/die, the machines reconnect to the SIP server, and we fail a status message rather than a checkin -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #120860|0 |1 is obsolete| | --- Comment #8 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 120955 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120955&action=edit Bug 28320: Add DB connection check to the SIP SC status message This patch adds a lookup of the sip user during an SC status, confirming that our DB ocnnection is working, and that our user is still valid Additionally, it adds support for SC status to the sip_cli_emulator and adds basic test coverage for the SC status message To test: 1 - Apply patch 2 - Restart SP server 3 - perl misc/sip_cli_emulator.pl -a localhost -p 6001 -l CPL -m sc_status_request -su term1 -sp term1 4 - prove -v t/db_dependent/SIP/Message.t Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #9 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Tested and works as expected. I read into the die and agree it's how it should be done in this case.. nice to learn. Passing QA -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 --- Comment #10 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28320 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|21.05.00,20.11.06 |21.05.00,20.11.06,20.05.12 released in| | Status|Pushed to stable |Pushed to oldstable CC| |andrew@bywatersolutions.com --- Comment #12 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Pushed to 20.05.x for 20.05.12 -- You are receiving this mail because: You are watching all bug changes.