More rigorous session-cleanup for BecomeUser

Kevin Falcone commented:
> We cheched it out with our become-user-thingie which basically
> does:
> $session{'CurrentUser'}->Load($user);
> delete $session{'my_rt_portlets'};
> $session{'CurrentUser'}->{'_principal_obj'}->{'values'}->{'id'} = $session{'CurrentUser'}->id;

That should be enough to work around the SeeQueue cache, although I'd
probably make the 3rd line just a
delete $session{'CurrentUser}{'_principal_obj'}
and let RT reload the Principal when it needs it. That avoids any
potential "what?" happening.

diff --git a/local/html/Admin/Tools/BecomeUser.html b/local/html/Admin/Tools/BecomeUser.html
index a39e3f643aa8cf8e0f9706d06d732e0ae9ce3b56..1106ae67c1ce0d1db7c950c0dd03621dd73f8c48 100644 (file)
--- a/local/html/Admin/Tools/BecomeUser.html
+++ b/local/html/Admin/Tools/BecomeUser.html
@@ -65,7 +65,7 @@
       } 
       delete $session{'my_rt_portlets'};
       unless ( defined($ARGS{'submit'}) && $ARGS{'submit'} =~ m/super/ ){
-          $session{'CurrentUser'}->{'_principal_obj'}->{'values'}->{'id'} = $session{'CurrentUser'}->id;
+          delete $session{'CurrentUser'}->{'_principal_obj'};
       }
    }