services/api/test/integration/remote_user_test.rb

   1 # Copyright (C) The Arvados Authors. All rights reserved.
   2 #
   3 # SPDX-License-Identifier: AGPL-3.0
   4
   5 require 'webrick'
   6 require 'webrick/https'
   7 require 'test_helper'
   8 require 'helpers/users_test_helper'
   9
  10 class RemoteUsersTest < ActionDispatch::IntegrationTest
  11   include DbCurrentTime
  12
  13   def salted_active_token(remote:)
  14     salt_token(fixture: :active, remote: remote).sub('/zzzzz-', '/'+remote+'-')
  15   end
  16
  17   def auth(remote:)
  18     token = salted_active_token(remote: remote)
  19     {"HTTP_AUTHORIZATION" => "Bearer #{token}"}
  20   end
  21
  22   # For remote authentication tests, we bring up a simple stub server
  23   # (on a port chosen by webrick) and configure the SUT so the stub is
  24   # responsible for clusters "zbbbb" (a well-behaved cluster) and
  25   # "zbork" (a misbehaving cluster).
  26   #
  27   # Test cases can override the stub's default response to
  28   # .../users/current by changing @stub_status and @stub_content.
  29   setup do
  30     clnt = HTTPClient.new
  31     clnt.ssl_config.verify_mode = OpenSSL::SSL::VERIFY_NONE
  32     HTTPClient.stubs(:new).returns clnt
  33
  34     @controller = Arvados::V1::UsersController.new
  35     ready = Thread::Queue.new
  36     srv = WEBrick::HTTPServer.new(
  37       Port: 0,
  38       Logger: WEBrick::Log.new(
  39         Rails.root.join("log", "webrick.log").to_s,
  40         WEBrick::Log::INFO),
  41       AccessLog: [[File.open(Rails.root.join(
  42                               "log", "webrick_access.log").to_s, 'a+'),
  43                    WEBrick::AccessLog::COMBINED_LOG_FORMAT]],
  44       SSLEnable: true,
  45       SSLVerifyClient: OpenSSL::SSL::VERIFY_NONE,
  46       SSLPrivateKey: OpenSSL::PKey::RSA.new(
  47         File.open(Rails.root.join("tmp", "self-signed.key")).read),
  48       SSLCertificate: OpenSSL::X509::Certificate.new(
  49         File.open(Rails.root.join("tmp", "self-signed.pem")).read),
  50       SSLCertName: [["CN", WEBrick::Utils::getservername]],
  51       StartCallback: lambda { ready.push(true) })
  52     srv.mount_proc '/discovery/v1/apis/arvados/v1/rest' do |req, res|
  53       Rails.cache.delete 'arvados_v1_rest_discovery'
  54       res.body = Arvados::V1::SchemaController.new.send(:discovery_doc).to_json
  55     end
  56     srv.mount_proc '/arvados/v1/users/current' do |req, res|
  57       res.status = @stub_status
  58       res.body = @stub_content.is_a?(String) ? @stub_content : @stub_content.to_json
  59     end
  60     Thread.new do
  61       srv.start
  62     end
  63     ready.pop
  64     @remote_server = srv
  65     @remote_host = "127.0.0.1:#{srv.config[:Port]}"
  66     Rails.configuration.remote_hosts['zbbbb'] = @remote_host
  67     Rails.configuration.remote_hosts['zbork'] = @remote_host
  68     Arvados::V1::SchemaController.any_instance.stubs(:root_url).returns "https://#{@remote_host}"
  69     @stub_status = 200
  70     @stub_content = {
  71       uuid: 'zbbbb-tpzed-000000000000000',
  72       email: 'foo@example.com',
  73       username: 'barney',
  74       is_admin: true,
  75       is_active: true,
  76     }
  77   end
  78
  79   teardown do
  80     @remote_server.andand.stop
  81   end
  82
  83   test 'authenticate with remote token' do
  84     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
  85     assert_response :success
  86     assert_equal 'zbbbb-tpzed-000000000000000', json_response['uuid']
  87     assert_equal false, json_response['is_admin']
  88     assert_equal 'foo@example.com', json_response['email']
  89     assert_equal 'barney', json_response['username']
  90
  91     # revoke original token
  92     @stub_status = 401
  93
  94     # re-authorize before cache expires
  95     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
  96     assert_response :success
  97
  98     # simulate cache expiry
  99     ApiClientAuthorization.where(
 100       uuid: salted_active_token(remote: 'zbbbb').split('/')[1]).
 101       update_all(expires_at: db_current_time - 1.minute)
 102
 103     # re-authorize after cache expires
 104     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
 105     assert_response 401
 106
 107     # simulate cached token indicating wrong user (e.g., local user
 108     # entry was migrated out of the way taking the cached token with
 109     # it, or authorizing cluster reassigned auth to a different user)
 110     ApiClientAuthorization.where(
 111       uuid: salted_active_token(remote: 'zbbbb').split('/')[1]).
 112       update_all(user_id: users(:active).id)
 113
 114     # revive original token and re-authorize
 115     @stub_status = 200
 116     @stub_content[:username] = 'blarney'
 117     @stub_content[:email] = 'blarney@example.com'
 118     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
 119     assert_response :success
 120     assert_equal 'barney', json_response['username'], 'local username should not change once assigned'
 121     assert_equal 'blarney@example.com', json_response['email']
 122   end
 123
 124   test 'authenticate with remote token, remote username conflicts with local' do
 125     @stub_content[:username] = 'active'
 126     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
 127     assert_response :success
 128     assert_equal 'active2', json_response['username']
 129   end
 130
 131   test 'authenticate with remote token, remote username is nil' do
 132     @stub_content.delete :username
 133     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
 134     assert_response :success
 135     assert_equal 'foo', json_response['username']
 136   end
 137
 138   test 'authenticate with remote token from misbhehaving remote cluster' do
 139     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbork')
 140     assert_response 401
 141   end
 142
 143   test 'authenticate with remote token that fails validate' do
 144     @stub_status = 401
 145     @stub_content = {
 146       error: 'not authorized',
 147     }
 148     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
 149     assert_response 401
 150   end
 151
 152   ['v2',
 153    'v2/',
 154    'v2//',
 155    'v2///',
 156    "v2/'; delete from users where 1=1; commit; select '/lol",
 157    'v2/foo/bar',
 158    'v2/zzzzz-gj3su-077z32aux8dg2s1',
 159    'v2/zzzzz-gj3su-077z32aux8dg2s1/',
 160    'v2/3kg6k6lzmp9kj5cpkcoxie963cmvjahbt2fod9zru30k1jqdmi',
 161    'v2/3kg6k6lzmp9kj5cpkcoxie963cmvjahbt2fod9zru30k1jqdmi/zzzzz-gj3su-077z32aux8dg2s1',
 162    'v2//3kg6k6lzmp9kj5cpkcoxie963cmvjahbt2fod9zru30k1jqdmi',
 163    'v8/zzzzz-gj3su-077z32aux8dg2s1/3kg6k6lzmp9kj5cpkcoxie963cmvjahbt2fod9zru30k1jqdmi',
 164    '/zzzzz-gj3su-077z32aux8dg2s1/3kg6k6lzmp9kj5cpkcoxie963cmvjahbt2fod9zru30k1jqdmi',
 165    '"v2/zzzzz-gj3su-077z32aux8dg2s1/3kg6k6lzmp9kj5cpkcoxie963cmvjahbt2fod9zru30k1jqdmi"',
 166    '/',
 167    '//',
 168    '///',
 169   ].each do |token|
 170     test "authenticate with malformed remote token #{token}" do
 171       get '/arvados/v1/users/current', {format: 'json'}, {"HTTP_AUTHORIZATION" => "Bearer #{token}"}
 172       assert_response 401
 173     end
 174   end
 175
 176   test "ignore extra fields in remote token" do
 177     token = salted_active_token(remote: 'zbbbb') + '/foo/bar/baz/*'
 178     get '/arvados/v1/users/current', {format: 'json'}, {"HTTP_AUTHORIZATION" => "Bearer #{token}"}
 179     assert_response :success
 180   end
 181
 182   test 'remote api server is not an api server' do
 183     @stub_status = 200
 184     @stub_content = '<html>bad</html>'
 185     get '/arvados/v1/users/current', {format: 'json'}, auth(remote: 'zbbbb')
 186     assert_response 401
 187   end
 188
 189   ['zbbbb', 'z0000'].each do |token_valid_for|
 190     test "validate #{token_valid_for}-salted token for remote cluster zbbbb" do
 191       salted_token = salt_token(fixture: :active, remote: token_valid_for)
 192       get '/arvados/v1/users/current', {format: 'json', remote: 'zbbbb'}, {
 193             "HTTP_AUTHORIZATION" => "Bearer #{salted_token}"
 194           }
 195       if token_valid_for == 'zbbbb'
 196         assert_response 200
 197         assert_equal(users(:active).uuid, json_response['uuid'])
 198       else
 199         assert_response 401
 200       end
 201     end
 202   end
 203
 204   test "list readable groups with salted token" do
 205     salted_token = salt_token(fixture: :active, remote: 'zbbbb')
 206     get '/arvados/v1/groups', {
 207           format: 'json',
 208           remote: 'zbbbb',
 209           limit: 10000,
 210         }, {
 211           "HTTP_AUTHORIZATION" => "Bearer #{salted_token}"
 212         }
 213     assert_response 200
 214     group_uuids = json_response['items'].collect { |i| i['uuid'] }
 215     assert_includes(group_uuids, 'zzzzz-j7d0g-fffffffffffffff')
 216     refute_includes(group_uuids, 'zzzzz-j7d0g-000000000000000')
 217     assert_includes(group_uuids, groups(:aproject).uuid)
 218     refute_includes(group_uuids, groups(:trashed_project).uuid)
 219     refute_includes(group_uuids, groups(:testusergroup_admins).uuid)
 220   end
 221 end