X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/ce4285dd9a6310a799b861237918273329390316..0ab9897ec19e24b7964b2467984065327010d400:/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 12ef8eb3eb..38538cb4ff 100644 --- a/services/api/app/models/api_client_authorization.rb +++ b/services/api/app/models/api_client_authorization.rb @@ -98,11 +98,31 @@ class ApiClientAuthorization < ArvadosModel case token[0..2] when 'v2/' - _, uuid, secret = token.split('/') + _, uuid, secret, optional = token.split('/') unless 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 != uuid + # token doesn't match the container's token + return nil + end + if !c.runtime_token.nil? and "v2/#{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 + auth = ApiClientAuthorization. includes(:user, :api_client). where('uuid=? and (expires_at is null or expires_at > CURRENT_TIMESTAMP)', uuid). @@ -135,6 +155,12 @@ class ApiClientAuthorization < ArvadosModel clnt = HTTPClient.new if Rails.configuration.sso_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 remote_user = SafeJSON.load( clnt.get_content('https://' + host + '/arvados/v1/users/current', @@ -216,6 +242,13 @@ class ApiClientAuthorization < ArvadosModel '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