2018-12-23 12:14:25 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2016-09-29 09:46:39 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2018-12-23 12:14:25 +05:30
|
|
|
describe Gitlab::LfsToken, :clean_gitlab_redis_shared_state do
|
2016-10-01 15:18:49 +05:30
|
|
|
describe '#token' do
|
2016-09-29 09:46:39 +05:30
|
|
|
shared_examples 'an LFS token generator' do
|
2018-12-23 12:14:25 +05:30
|
|
|
it 'returns a computed token' do
|
|
|
|
expect(Settings).to receive(:attr_encrypted_db_key_base).and_return('fbnbv6hdjweo53qka7kza8v8swxc413c05pb51qgtfte0bygh1p2e508468hfsn5ntmjcyiz7h1d92ashpet5pkdyejg7g8or3yryhuso4h8o5c73h429d9c3r6bjnet').twice
|
|
|
|
|
|
|
|
token = lfs_token.token
|
2016-09-29 09:46:39 +05:30
|
|
|
|
|
|
|
expect(token).not_to be_nil
|
|
|
|
expect(token).to be_a String
|
2018-12-23 12:14:25 +05:30
|
|
|
expect(described_class.new(actor).token_valid?(token)).to be_truthy
|
2016-09-29 09:46:39 +05:30
|
|
|
end
|
2018-12-23 12:14:25 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the actor is a user' do
|
|
|
|
let(:actor) { create(:user, username: 'test_user_lfs_1') }
|
|
|
|
let(:lfs_token) { described_class.new(actor) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(actor).to receive(:encrypted_password).and_return('$2a$04$ETfzVS5spE9Hexn9wh6NUenCHG1LyZ2YdciOYxieV1WLSa8DHqOFO')
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'an LFS token generator'
|
2016-09-29 09:46:39 +05:30
|
|
|
|
2018-12-23 12:14:25 +05:30
|
|
|
it 'returns the correct username' do
|
|
|
|
expect(lfs_token.actor_name).to eq(actor.username)
|
|
|
|
end
|
2016-09-29 09:46:39 +05:30
|
|
|
|
2018-12-23 12:14:25 +05:30
|
|
|
it 'returns the correct token type' do
|
|
|
|
expect(lfs_token.type).to eq(:lfs_token)
|
2016-09-29 09:46:39 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-12-23 12:14:25 +05:30
|
|
|
context 'when the actor is a key' do
|
|
|
|
let(:user) { create(:user, username: 'test_user_lfs_2') }
|
|
|
|
let(:actor) { create(:key, user: user) }
|
|
|
|
let(:lfs_token) { described_class.new(actor) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(user).to receive(:encrypted_password).and_return('$2a$04$C1GAMKsOKouEbhKy2JQoe./3LwOfQAZc.hC8zW2u/wt8xgukvnlV.')
|
|
|
|
end
|
2016-09-29 09:46:39 +05:30
|
|
|
|
|
|
|
it_behaves_like 'an LFS token generator'
|
|
|
|
|
|
|
|
it 'returns the correct username' do
|
2018-12-23 12:14:25 +05:30
|
|
|
expect(lfs_token.actor_name).to eq(user.username)
|
2016-09-29 09:46:39 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns the correct token type' do
|
2018-12-23 12:14:25 +05:30
|
|
|
expect(lfs_token.type).to eq(:lfs_token)
|
2016-09-29 09:46:39 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the actor is a deploy key' do
|
2018-12-23 12:14:25 +05:30
|
|
|
let(:actor_id) { 1 }
|
2016-09-29 09:46:39 +05:30
|
|
|
let(:actor) { create(:deploy_key) }
|
2018-12-23 12:14:25 +05:30
|
|
|
let(:project) { create(:project) }
|
|
|
|
let(:lfs_token) { described_class.new(actor) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(actor).to receive(:id).and_return(actor_id)
|
|
|
|
end
|
2016-09-29 09:46:39 +05:30
|
|
|
|
|
|
|
it_behaves_like 'an LFS token generator'
|
|
|
|
|
|
|
|
it 'returns the correct username' do
|
2018-12-23 12:14:25 +05:30
|
|
|
expect(lfs_token.actor_name).to eq("lfs+deploy-key-#{actor_id}")
|
2016-09-29 09:46:39 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns the correct token type' do
|
2018-12-23 12:14:25 +05:30
|
|
|
expect(lfs_token.type).to eq(:lfs_deploy_token)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the actor is invalid' do
|
|
|
|
it 'raises an exception' do
|
|
|
|
expect { described_class.new('invalid') }.to raise_error('Bad Actor')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#token_valid?' do
|
|
|
|
let(:actor) { create(:user, username: 'test_user_lfs_1') }
|
|
|
|
let(:lfs_token) { described_class.new(actor) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(actor).to receive(:encrypted_password).and_return('$2a$04$ETfzVS5spE9Hexn9wh6NUenCHG1LyZ2YdciOYxieV1WLSa8DHqOFO')
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for an HMAC token' do
|
|
|
|
before do
|
|
|
|
# We're not interested in testing LegacyRedisDeviseToken here
|
|
|
|
allow(Gitlab::LfsToken::LegacyRedisDeviseToken).to receive_message_chain(:new, :token_valid?).and_return(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'where the token is invalid' do
|
|
|
|
context "because it's junk" do
|
|
|
|
it 'returns false' do
|
|
|
|
expect(lfs_token.token_valid?('junk')).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "because it's been fiddled with" do
|
|
|
|
it 'returns false' do
|
|
|
|
fiddled_token = lfs_token.token.tap { |token| token[0] = 'E' }
|
|
|
|
expect(lfs_token.token_valid?(fiddled_token)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "because it was generated with a different secret" do
|
|
|
|
it 'returns false' do
|
|
|
|
different_actor = create(:user, username: 'test_user_lfs_2')
|
|
|
|
different_secret_token = described_class.new(different_actor).token
|
|
|
|
expect(lfs_token.token_valid?(different_secret_token)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "because it's expired" do
|
|
|
|
it 'returns false' do
|
|
|
|
expired_token = lfs_token.token
|
|
|
|
# Needs to be at least 1860 seconds, because the default expiry is
|
|
|
|
# 1800 seconds with an additional 60 second leeway.
|
|
|
|
Timecop.freeze(Time.now + 1865) do
|
|
|
|
expect(lfs_token.token_valid?(expired_token)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'where the token is valid' do
|
|
|
|
it 'returns true' do
|
|
|
|
expect(lfs_token.token_valid?(lfs_token.token)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for a LegacyRedisDevise token' do
|
|
|
|
before do
|
|
|
|
# We're not interested in testing HMACToken here
|
|
|
|
allow_any_instance_of(Gitlab::LfsToken::HMACToken).to receive(:token_valid?).and_return(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'where the token is invalid' do
|
|
|
|
context "because it's junk" do
|
|
|
|
it 'returns false' do
|
|
|
|
expect(lfs_token.token_valid?('junk')).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "because it's been fiddled with" do
|
|
|
|
it 'returns false' do
|
|
|
|
generated_token = Gitlab::LfsToken::LegacyRedisDeviseToken.new(actor).store_new_token
|
|
|
|
fiddled_token = generated_token.tap { |token| token[0] = 'E' }
|
|
|
|
expect(lfs_token.token_valid?(fiddled_token)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "because it was generated with a different secret" do
|
|
|
|
it 'returns false' do
|
|
|
|
different_actor = create(:user, username: 'test_user_lfs_2')
|
|
|
|
different_secret_token = described_class.new(different_actor).token
|
|
|
|
expect(lfs_token.token_valid?(different_secret_token)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "because it's expired" do
|
|
|
|
it 'returns false' do
|
|
|
|
generated_token = Gitlab::LfsToken::LegacyRedisDeviseToken.new(actor).store_new_token(1)
|
|
|
|
# We need a real sleep here because we need to wait for redis to expire the key.
|
|
|
|
sleep(0.01)
|
|
|
|
expect(lfs_token.token_valid?(generated_token)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'where the token is valid' do
|
|
|
|
it 'returns true' do
|
|
|
|
generated_token = Gitlab::LfsToken::LegacyRedisDeviseToken.new(actor).store_new_token
|
|
|
|
expect(lfs_token.token_valid?(generated_token)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#deploy_key_pushable?' do
|
|
|
|
let(:lfs_token) { described_class.new(actor) }
|
|
|
|
|
|
|
|
context 'when actor is not a DeployKey' do
|
|
|
|
let(:actor) { create(:user) }
|
|
|
|
let(:project) { create(:project) }
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(lfs_token.deploy_key_pushable?(project)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when actor is a DeployKey' do
|
|
|
|
let(:deploy_keys_project) { create(:deploy_keys_project, can_push: can_push) }
|
|
|
|
let(:project) { deploy_keys_project.project }
|
|
|
|
let(:actor) { deploy_keys_project.deploy_key }
|
|
|
|
|
|
|
|
context 'but the DeployKey cannot push to the project' do
|
|
|
|
let(:can_push) { false }
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(lfs_token.deploy_key_pushable?(project)).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'and the DeployKey can push to the project' do
|
|
|
|
let(:can_push) { true }
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(lfs_token.deploy_key_pushable?(project)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#type' do
|
|
|
|
let(:lfs_token) { described_class.new(actor) }
|
|
|
|
|
|
|
|
context 'when actor is not a User' do
|
|
|
|
let(:actor) { create(:deploy_key) }
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(lfs_token.type).to eq(:lfs_deploy_token)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when actor is a User' do
|
|
|
|
let(:actor) { create(:user) }
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(lfs_token.type).to eq(:lfs_token)
|
2016-09-29 09:46:39 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|