SIPServer tosses away current account prematurely
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
SIPServer |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
When using the Prefork Net::Server personality, SIPServer backends use raw_transport(
Commit cd455136 for bug 1227273 ensures that the current account is cleared when a new connection is made; this prevents a client from getting access to a backend that is already authenticated.
However, that commit also clears the current account between raw_transport(
Changed in sipserver: | |
status: | New → Fix Committed |
Changed in sipserver: | |
status: | Fix Committed → Fix Released |
The user/gmcharlt/ lp1464748_ do_not_ toss_login_ session_ prematurely branch in the working/SIPServer repository contains two patches: a signed-off patch for bug 1463459 and a prospective patch for this bug:
http:// git.evergreen- ils.org/ ?p=working/ SIPServer. git;a=shortlog; h=refs/ heads/user/ gmcharlt/ lp1464748_ do_not_ toss_login_ session_ prematurely
Here is the test plan for both patches:
[1] Set up a SIP configuration with at least two institutions
with different capabilities and at least one account
for each institution. Assume that account A1 is for
institution I1 and A2 for I2.
[2] Prior to applying the patches for 1463459 and 1464748,
start the SIP server and try the following sequence:
a. 93 to log in as A2
b. 99 to get SC status
c. Note that the 98 response has values for I1, not I2
[3] Apply the patches and restart SIP, then repeat the steps
in [2]. This time, the 98 response should have the
values for I2.