2017-07-14 11:41:49 -07:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'rails_helper'
|
|
|
|
|
|
|
|
describe ApplicationController, type: :controller do
|
|
|
|
controller do
|
|
|
|
include SignatureVerification
|
|
|
|
|
|
|
|
def success
|
|
|
|
head 200
|
|
|
|
end
|
|
|
|
|
|
|
|
def alternative_success
|
|
|
|
head 200
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
2017-08-09 14:54:14 -07:00
|
|
|
routes.draw { match via: [:get, :post], 'success' => 'anonymous#success' }
|
2017-07-14 11:41:49 -07:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'without signature header' do
|
|
|
|
before do
|
|
|
|
get :success
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#signed_request?' do
|
|
|
|
it 'returns false' do
|
|
|
|
expect(controller.signed_request?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#signed_request_account' do
|
|
|
|
it 'returns nil' do
|
|
|
|
expect(controller.signed_request_account).to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with signature header' do
|
|
|
|
let!(:author) { Fabricate(:account) }
|
|
|
|
|
2017-08-09 14:54:14 -07:00
|
|
|
context 'without body' do
|
|
|
|
before do
|
|
|
|
get :success
|
2017-07-14 11:41:49 -07:00
|
|
|
|
2017-08-09 14:54:14 -07:00
|
|
|
fake_request = Request.new(:get, request.url)
|
|
|
|
fake_request.on_behalf_of(author)
|
2017-07-14 11:41:49 -07:00
|
|
|
|
2017-08-09 14:54:14 -07:00
|
|
|
request.headers.merge!(fake_request.headers)
|
|
|
|
end
|
2017-07-14 11:41:49 -07:00
|
|
|
|
2017-08-09 14:54:14 -07:00
|
|
|
describe '#signed_request?' do
|
|
|
|
it 'returns true' do
|
|
|
|
expect(controller.signed_request?).to be true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#signed_request_account' do
|
|
|
|
it 'returns an account' do
|
|
|
|
expect(controller.signed_request_account).to eq author
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns nil when path does not match' do
|
|
|
|
request.path = '/alternative-path'
|
|
|
|
expect(controller.signed_request_account).to be_nil
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns nil when method does not match' do
|
|
|
|
post :success
|
|
|
|
expect(controller.signed_request_account).to be_nil
|
|
|
|
end
|
2017-07-14 11:41:49 -07:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-10-11 15:15:55 -07:00
|
|
|
context 'with request older than a day' do
|
|
|
|
before do
|
|
|
|
get :success
|
|
|
|
|
|
|
|
fake_request = Request.new(:get, request.url)
|
|
|
|
fake_request.add_headers({ 'Date' => 2.days.ago.utc.httpdate })
|
|
|
|
fake_request.on_behalf_of(author)
|
|
|
|
|
|
|
|
request.headers.merge!(fake_request.headers)
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#signed_request?' do
|
|
|
|
it 'returns true' do
|
|
|
|
expect(controller.signed_request?).to be true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#signed_request_account' do
|
|
|
|
it 'returns nil' do
|
|
|
|
expect(controller.signed_request_account).to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-08-09 14:54:14 -07:00
|
|
|
context 'with body' do
|
|
|
|
before do
|
|
|
|
post :success, body: 'Hello world'
|
|
|
|
|
|
|
|
fake_request = Request.new(:post, request.url, body: 'Hello world')
|
|
|
|
fake_request.on_behalf_of(author)
|
|
|
|
|
|
|
|
request.headers.merge!(fake_request.headers)
|
2017-07-14 11:41:49 -07:00
|
|
|
end
|
|
|
|
|
2017-08-09 14:54:14 -07:00
|
|
|
describe '#signed_request?' do
|
|
|
|
it 'returns true' do
|
|
|
|
expect(controller.signed_request?).to be true
|
|
|
|
end
|
2017-07-14 11:41:49 -07:00
|
|
|
end
|
|
|
|
|
2017-08-09 14:54:14 -07:00
|
|
|
describe '#signed_request_account' do
|
|
|
|
it 'returns an account' do
|
|
|
|
expect(controller.signed_request_account).to eq author
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns nil when path does not match' do
|
|
|
|
request.path = '/alternative-path'
|
|
|
|
expect(controller.signed_request_account).to be_nil
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns nil when method does not match' do
|
|
|
|
get :success
|
|
|
|
expect(controller.signed_request_account).to be_nil
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns nil when body has been tampered' do
|
2018-08-12 03:25:23 -07:00
|
|
|
post :success, body: 'doo doo doo'
|
2017-08-09 14:54:14 -07:00
|
|
|
expect(controller.signed_request_account).to be_nil
|
|
|
|
end
|
2017-07-14 11:41:49 -07:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|