X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/6b3cfe60beb46ba087de5093926363d8b03ab889..0110ce93702def9a641d92d90d5544d4d0adf22b:/services/api/app/models/api_client_authorization.rb diff --git a/services/api/app/models/api_client_authorization.rb b/services/api/app/models/api_client_authorization.rb index b158faa272..651eacf626 100644 --- a/services/api/app/models/api_client_authorization.rb +++ b/services/api/app/models/api_client_authorization.rb @@ -87,122 +87,227 @@ class ApiClientAuthorization < ArvadosModel end def self.remote_host(uuid_prefix:) - Rails.configuration.remote_hosts[uuid_prefix] || - (Rails.configuration.remote_hosts_via_dns && + (Rails.configuration.RemoteClusters[uuid_prefix].andand["Host"]) || + (Rails.configuration.RemoteClusters["*"]["Proxy"] && uuid_prefix+".arvadosapi.com") end - def self.validate(token:, remote:) - return nil if !token - remote ||= Rails.configuration.uuid_prefix + def self.make_http_client(uuid_prefix:) + clnt = HTTPClient.new + + if uuid_prefix && (Rails.configuration.RemoteClusters[uuid_prefix].andand.Insecure || + Rails.configuration.RemoteClusters['*'].andand.Insecure) + clnt.ssl_config.verify_mode = OpenSSL::SSL::VERIFY_NONE + else + # Use system CA certificates + ["/etc/ssl/certs/ca-certificates.crt", + "/etc/pki/tls/certs/ca-bundle.crt"] + .select { |ca_path| File.readable?(ca_path) } + .each { |ca_path| clnt.ssl_config.add_trust_ca(ca_path) } + end + clnt + end + + def self.check_system_root_token token + if token == Rails.configuration.SystemRootToken + return ApiClientAuthorization.new(user: User.find_by_uuid(system_user_uuid), + api_token: token, + api_client: ApiClient.new(is_trusted: true, url_prefix: "")) + else + return nil + end + end + + def self.validate(token:, remote: nil) + return nil if token.nil? or token.empty? + remote ||= Rails.configuration.ClusterID + + auth = self.check_system_root_token(token) + if !auth.nil? + return auth + end case token[0..2] when 'v2/' - _, uuid, secret = token.split('/') - unless uuid.andand.length == 27 && secret.andand.length.andand > 0 + _, token_uuid, secret, optional = token.split('/') + unless token_uuid.andand.length == 27 && secret.andand.length.andand > 0 return nil end + if !optional.nil? + # if "optional" is a container uuid, check that it + # matches expections. + c = Container.where(uuid: optional).first + if !c.nil? + if !c.auth_uuid.nil? and c.auth_uuid != token_uuid + # token doesn't match the container's token + return nil + end + if !c.runtime_token.nil? and "v2/#{token_uuid}/#{secret}" != c.runtime_token + # token doesn't match the container's token + return nil + end + if ![Container::Locked, Container::Running].include?(c.state) + # container isn't locked or running, token shouldn't be used + return nil + end + end + end + + # fast path: look up the token in the local database auth = ApiClientAuthorization. includes(:user, :api_client). - where('uuid=? and (expires_at is null or expires_at > CURRENT_TIMESTAMP)', uuid). + where('uuid=? and (expires_at is null or expires_at > CURRENT_TIMESTAMP)', token_uuid). first if auth && auth.user && (secret == auth.api_token || secret == OpenSSL::HMAC.hexdigest('sha1', auth.api_token, remote)) + # found it return auth end - uuid_prefix = uuid[0..4] - if uuid_prefix == Rails.configuration.uuid_prefix - # If the token were valid, we would have validated it above + token_uuid_prefix = token_uuid[0..4] + if token_uuid_prefix == Rails.configuration.ClusterID + # Token is supposedly issued by local cluster, but if the + # token were valid, we would have been found in the database + # in the above query. return nil - elsif uuid_prefix.length != 5 + elsif token_uuid_prefix.length != 5 # malformed return nil end - host = remote_host(uuid_prefix: uuid_prefix) + # Invariant: token_uuid_prefix != Rails.configuration.ClusterID + # + # In other words the remaing code in this method below is the + # case that determines whether to accept a token that was issued + # by a remote cluster when the token absent or expired in our + # database. To begin, we need to ask the cluster that issued + # the token to [re]validate it. + clnt = ApiClientAuthorization.make_http_client(uuid_prefix: token_uuid_prefix) + + host = remote_host(uuid_prefix: token_uuid_prefix) if !host - Rails.logger.warn "remote authentication rejected: no host for #{uuid_prefix.inspect}" + Rails.logger.warn "remote authentication rejected: no host for #{token_uuid_prefix.inspect}" return nil end - # Token was issued by a different cluster. If it's expired or - # missing in our database, ask the originating cluster to - # [re]validate it. begin - clnt = HTTPClient.new - if Rails.configuration.sso_insecure - clnt.ssl_config.verify_mode = OpenSSL::SSL::VERIFY_NONE - end remote_user = SafeJSON.load( clnt.get_content('https://' + host + '/arvados/v1/users/current', - {'remote' => Rails.configuration.uuid_prefix}, + {'remote' => Rails.configuration.ClusterID}, {'Authorization' => 'Bearer ' + token})) rescue => e Rails.logger.warn "remote authentication with token #{token.inspect} failed: #{e}" return nil end - if !remote_user.is_a?(Hash) || !remote_user['uuid'].is_a?(String) || remote_user['uuid'][0..4] != uuid[0..4] + + # Check the response is well formed. + if !remote_user.is_a?(Hash) || !remote_user['uuid'].is_a?(String) Rails.logger.warn "remote authentication rejected: remote_user=#{remote_user.inspect}" return nil end - act_as_system_user do - # Add/update user and token in our database so we can - # validate subsequent requests faster. - - user = User.find_or_create_by(uuid: remote_user['uuid']) do |user| - # (this block runs for the "create" case, not for "find") - user.is_admin = false - user.email = remote_user['email'] - if remote_user['username'].andand.length.andand > 0 - user.set_initial_username(requested: remote_user['username']) - end - end - if Rails.configuration.new_users_are_active - # Update is_active to whatever it is at the remote end - user.is_active = remote_user['is_active'] - elsif !remote_user['is_active'] - # Remote user is inactive; our mirror should be, too. - user.is_active = false + remote_user_prefix = remote_user['uuid'][0..4] + + # Clusters can only authenticate for their own users. + if remote_user_prefix != token_uuid_prefix + Rails.logger.warn "remote authentication rejected: claimed remote user #{remote_user_prefix} but token was issued by #{token_uuid_prefix}" + return nil + end + + # Invariant: remote_user_prefix == token_uuid_prefix + # therefore: remote_user_prefix != Rails.configuration.ClusterID + + # Add or update user and token in local database so we can + # validate subsequent requests faster. + + user = User.find_by_uuid(remote_user['uuid']) + + if !user + # Create a new record for this user. + user = User.new(uuid: remote_user['uuid'], + is_active: false, + is_admin: false, + email: remote_user['email'], + owner_uuid: system_user_uuid) + user.set_initial_username(requested: remote_user['username']) + end + + # Sync user record. + if remote_user_prefix == Rails.configuration.Login.LoginCluster + # Remote cluster controls our user database, set is_active if + # remote is active. If remote is not active, user will be + # unsetup (see below). + user.is_active = true if remote_user['is_active'] + user.is_admin = remote_user['is_admin'] + else + if Rails.configuration.Users.NewUsersAreActive || + Rails.configuration.RemoteClusters[remote_user_prefix].andand["ActivateUsers"] + # Default policy is to activate users + user.is_active = true if remote_user['is_active'] end + end + + %w[first_name last_name email prefs].each do |attr| + user.send(attr+'=', remote_user[attr]) + end - %w[first_name last_name email prefs].each do |attr| - user.send(attr+'=', remote_user[attr]) + act_as_system_user do + if user.is_active && !remote_user['is_active'] + user.unsetup end user.save! - auth = ApiClientAuthorization.find_or_create_by(uuid: uuid) do |auth| + # We will accept this token (and avoid reloading the user + # record) for 'RemoteTokenRefresh' (default 5 minutes). + # Possible todo: + # Request the actual api_client_auth record from the remote + # server in case it wants the token to expire sooner. + auth = ApiClientAuthorization.find_or_create_by(uuid: token_uuid) do |auth| auth.user = user - auth.api_token = secret auth.api_client_id = 0 end - - # Accept this token (and don't reload the user record) for - # 5 minutes. TODO: Request the actual api_client_auth - # record from the remote server in case it wants the token - # to expire sooner. auth.update_attributes!(user: user, api_token: secret, api_client_id: 0, - expires_at: Time.now + 5.minutes) + expires_at: Time.now + Rails.configuration.Login.RemoteTokenRefresh) end return auth else + # token is not a 'v2' token auth = ApiClientAuthorization. - includes(:user, :api_client). - where('api_token=? and (expires_at is null or expires_at > CURRENT_TIMESTAMP)', token). - first + includes(:user, :api_client). + where('api_token=? and (expires_at is null or expires_at > CURRENT_TIMESTAMP)', token). + first if auth && auth.user return auth end end + return nil end + def token + v2token + end + + def v1token + api_token + end + + def v2token + 'v2/' + uuid + '/' + api_token + end + + def salted_token(remote:) + if remote.nil? + token + end + 'v2/' + uuid + '/' + OpenSSL::HMAC.hexdigest('sha1', api_token, remote) + end + protected def permission_to_create