From: Dan Wells <dbw2@calvin.edu>
Date: Tue, 13 Sep 2016 17:40:43 +0000 (-0400)
Subject: LP#1620803 Add missing workstation passthru to AuthProxy
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2db335069b260e6f884aebdcb4c9ee0375476b0a;p=evergreen%2Ftadl.git

LP#1620803 Add missing workstation passthru to AuthProxy

The AuthProxy.pm login function was not passing through a possible
'workstation' parameter to auth_internal's 'session.create' method.
Without this, all login's end up with a workstation set to the user's
home_ou, which can affect things including opt-in and transit
behaviors.

While not needed for the initial 'user.validate', it seems cleaner
and simpler to just bring it over when constructing the initial
$args hash.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
---

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/AuthProxy.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/AuthProxy.pm
index 66a1b712c3..1f7832c653 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/AuthProxy.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/AuthProxy.pm
@@ -258,6 +258,7 @@ sub login {
                 my $trimmed_args = {
                     user_id => $args->{user_id},
                     login_type => $args->{type},
+                    workstation => $args->{workstation},
                     org_unit => $args->{org}
                 };
                 $event = &_auth_internal('user.validate', $trimmed_args);