LP#1620803 Add missing workstation passthru to AuthProxy
authorDan Wells <dbw2@calvin.edu>
Tue, 13 Sep 2016 17:40:43 +0000 (13:40 -0400)
committerDan Scott <dscott@laurentian.ca>
Tue, 13 Sep 2016 17:53:54 +0000 (13:53 -0400)
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>
Open-ILS/src/perlmods/lib/OpenILS/Application/AuthProxy.pm

index 66a1b71..1f7832c 100644 (file)
@@ -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);