[ARVADOS] updated: 1.3.0-1535-g4653d82e5

[Git user]

Summary of changes:
 services/api/test/functional/user_sessions_controller_test.rb | 9 +++++++++
 1 file changed, 9 insertions(+)

       via  4653d82e57660b2f2d5cc9ce0960f44684b160ec (commit)
      from  383122c1bc4a08772df618a415420ac3ea527051 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 4653d82e57660b2f2d5cc9ce0960f44684b160ec
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Thu Aug 22 09:44:09 2019 -0400

    15529: Add test
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/services/api/test/functional/user_sessions_controller_test.rb b/services/api/test/functional/user_sessions_controller_test.rb
index f846eaca0..d96ccb090 100644
--- a/services/api/test/functional/user_sessions_controller_test.rb
+++ b/services/api/test/functional/user_sessions_controller_test.rb
@@ -55,4 +55,13 @@ class UserSessionsControllerTest < ActionController::TestCase
     assert_nil assigns(:api_client)
   end
 
+  test "don't go into redirect loop if LoginCluster is self" do
+    Rails.configuration.Login.LoginCluster = 'zzzzz'
+    api_client_page = 'http://client.example.com/home'
+    get :login, params: {return_to: api_client_page}
+    assert_response :redirect
+    assert_equal("http://test.host/auth/joshid?return_to=%2Chttp%3A%2F%2Fclient.example.com%2Fhome", @response.redirect_url)
+    assert_nil assigns(:api_client)
+  end
+
 end

-----------------------------------------------------------------------


hooks/post-receive
--