Move the user role down to company_user relation
I was starting to add the public page for campsite types, creating more
granular row-level security policies for select, insert, update, and
delete, because now the guest users needed to SELECT them and they have
no related company to filter the rows with. Suddenly, i realized that
the role was wrong in the user relation: a user can be an admin to one
company, and employee to another, and guess to yet another company;
the role should be in the company_user relation instead.
That means that to know the role to set to, the user alone is not enough
and have to know the company as well. Had to change all the
cookie-related function to accept also the company’s host name, as this
is the information that the Go application has.
2023-08-08 00:22:16 +00:00
|
|
|
-- Test current_company_id
|
|
|
|
|
set client_min_messages to warning;
|
|
|
|
|
create extension if not exists pgtap;
|
|
|
|
|
reset client_min_messages;
|
|
|
|
|
|
|
|
|
|
begin;
|
|
|
|
|
|
|
|
|
|
set search_path to camper, auth, public;
|
|
|
|
|
|
|
|
|
|
select plan(15);
|
|
|
|
|
|
|
|
|
|
select has_function('camper', 'current_company_id', array []::name[]);
|
|
|
|
|
select function_lang_is('camper', 'current_company_id', array []::name[], 'sql');
|
|
|
|
|
select function_returns('camper', 'current_company_id', array []::name[], 'integer');
|
|
|
|
|
select isnt_definer('camper', 'current_company_id', array []::name[]);
|
|
|
|
|
select volatility_is('camper', 'current_company_id', array []::name[], 'stable');
|
|
|
|
|
select function_privs_are('camper', 'current_company_id', array []::name[], 'guest', array ['EXECUTE']);
|
|
|
|
|
select function_privs_are('camper', 'current_company_id', array []::name[], 'employee', array ['EXECUTE']);
|
|
|
|
|
select function_privs_are('camper', 'current_company_id', array []::name[], 'admin', array ['EXECUTE']);
|
|
|
|
|
select function_privs_are('camper', 'current_company_id', array []::name[], 'authenticator', array []::text[]);
|
|
|
|
|
|
|
|
|
|
set client_min_messages to warning;
|
|
|
|
|
truncate company_host cascade;
|
|
|
|
|
truncate company_user cascade;
|
|
|
|
|
truncate company cascade;
|
|
|
|
|
truncate auth."user" cascade;
|
|
|
|
|
reset client_min_messages;
|
|
|
|
|
|
|
|
|
|
insert into auth."user" (user_id, email, name, password, cookie, cookie_expires_at)
|
|
|
|
|
values (1, 'demo@tandem.blog', 'Demo', 'test', '44facbb30d8a419dfd4bfbc44a4b5539d4970148dfc84bed0e', current_timestamp + interval '1 month')
|
|
|
|
|
, (5, 'admin@tandem.blog', 'Demo', 'test', '12af4c88b528c2ad4222e3740496ecbc58e76e26f087657524', current_timestamp + interval '1 month')
|
|
|
|
|
;
|
|
|
|
|
|
2024-01-14 01:09:17 +00:00
|
|
|
insert into company (company_id, business_name, vatin, trade_name, phone, email, web, address, city, province, postal_code, rtc_number, tourist_tax, country_code, currency_code, default_lang_tag)
|
|
|
|
|
values (2, 'Company 2', 'XX123', '', '555-555-555', 'a@a', '', '', '', '', '', '', 60, 'ES', 'EUR', 'ca')
|
|
|
|
|
, (4, 'Company 4', 'XX234', '', '666-666-666', 'b@b', '', '', '', '', '', '', 60, 'FR', 'USD', 'es')
|
Move the user role down to company_user relation
I was starting to add the public page for campsite types, creating more
granular row-level security policies for select, insert, update, and
delete, because now the guest users needed to SELECT them and they have
no related company to filter the rows with. Suddenly, i realized that
the role was wrong in the user relation: a user can be an admin to one
company, and employee to another, and guess to yet another company;
the role should be in the company_user relation instead.
That means that to know the role to set to, the user alone is not enough
and have to know the company as well. Had to change all the
cookie-related function to accept also the company’s host name, as this
is the information that the Go application has.
2023-08-08 00:22:16 +00:00
|
|
|
;
|
|
|
|
|
|
|
|
|
|
insert into company_user (company_id, user_id, role)
|
|
|
|
|
values (2, 1, 'employee')
|
|
|
|
|
, (4, 5, 'admin')
|
|
|
|
|
;
|
|
|
|
|
|
|
|
|
|
insert into company_host (company_id, host)
|
|
|
|
|
values (2, 'co2')
|
|
|
|
|
, (4, 'co4')
|
|
|
|
|
;
|
|
|
|
|
|
|
|
|
|
select lives_ok(
|
|
|
|
|
$$ select set_cookie('44facbb30d8a419dfd4bfbc44a4b5539d4970148dfc84bed0e/demo@tandem.blog', 'co2') $$,
|
|
|
|
|
'Should change ok for the first user'
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
select is(
|
|
|
|
|
current_company_id(),
|
|
|
|
|
2,
|
|
|
|
|
'Should return the company ID of the first user'
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
reset role;
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
select lives_ok(
|
|
|
|
|
$$ select set_cookie('12af4c88b528c2ad4222e3740496ecbc58e76e26f087657524/admin@tandem.blog', 'co4') $$,
|
|
|
|
|
'Should change ok for the second user'
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
select is(
|
|
|
|
|
current_company_id(),
|
|
|
|
|
4,
|
|
|
|
|
'Should return the cookie of the second user'
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
reset role;
|
|
|
|
|
|
|
|
|
|
select lives_ok(
|
|
|
|
|
$$ select set_cookie('', 'co2') $$,
|
|
|
|
|
'Should change ok for a guest user'
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
select is(current_company_id(), 0, 'Should return zero');
|
|
|
|
|
|
|
|
|
|
reset role;
|
|
|
|
|
|
|
|
|
|
select *
|
|
|
|
|
from finish();
|
|
|
|
|
|
|
|
|
|
rollback;
|
