Summary: | SIP2 terminal authentication via telnet transport broken | ||
---|---|---|---|
Product: | Koha | Reporter: | Galen Charlton <gmcharlt> |
Component: | SIP2 | Assignee: | Galen Charlton <gmcharlt> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | critical | ||
Priority: | P5 - low | CC: | arm, chris, colin.campbell, jonathan.druart, kyle, tomascohen |
Version: | Main | Keywords: | regression |
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 13506 | ||
Bug Blocks: | |||
Attachments: |
Bug 14644: unbreak SIP2 terminal login when using telnet transport
Signed Off patch Bug 14644: unbreak SIP2 terminal login when using telnet transport |
Description
Galen Charlton
2015-08-04 15:42:05 UTC
Created attachment 41359 [details] [review] Bug 14644: unbreak SIP2 terminal login when using telnet transport This patch fixes breakage introduced by the patch for bug 13506 that prevented SIP2 clients from authenticating themselves over the telnet transport. Attempts to do so would manifest by the SIPServer backend crashing and the connection getting closed. To test: [1] Use the stock SIPServer config, etc/SIPconfig.xml, and ensure that the database contains a staff user whose username is 'term1' and password is 'term1' that has the circulate permission. [2] Start SIPServer, telnet to port 8023, and attempt to log in. The connection will be aborted: $ telnet localhost 8023 Trying 127.0.0.1... Connected to localhost. Escape character is '^]'. login: term1 password: term1 Connection closed by foreign host. [3] Apply the patch and restart SIPSever, then repeat step 2. This time, the login should succeed: $ telnet localhost 8023 Trying 127.0.0.1... Connected to localhost. Escape character is '^]'. login: term1 password: term1 Login OK. Initiating SIP Signed-off-by: Galen Charlton <gmc@esilibrary.com> Created attachment 41373 [details] [review] Signed Off patch Signed off - works as described Created attachment 41657 [details] [review] Bug 14644: unbreak SIP2 terminal login when using telnet transport This patch fixes breakage introduced by the patch for bug 13506 that prevented SIP2 clients from authenticating themselves over the telnet transport. Attempts to do so would manifest by the SIPServer backend crashing and the connection getting closed. To test: [1] Use the stock SIPServer config, etc/SIPconfig.xml, and ensure that the database contains a staff user whose username is 'term1' and password is 'term1' that has the circulate permission. [2] Start SIPServer, telnet to port 8023, and attempt to log in. The connection will be aborted: $ telnet localhost 8023 Trying 127.0.0.1... Connected to localhost. Escape character is '^]'. login: term1 password: term1 Connection closed by foreign host. [3] Apply the patch and restart SIPSever, then repeat step 2. This time, the login should succeed: $ telnet localhost 8023 Trying 127.0.0.1... Connected to localhost. Escape character is '^]'. login: term1 password: term1 Login OK. Initiating SIP Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Should be backported to 3.20.x. Patch pushed to master. Thanks Galen! Pushed to 3.20.x will be in 3.20.3 |