1
76%
BABEL_5_X_DEV: 77%

Ran 30 Sep 2024 01:29PM UTC

Files 115

Run time 4s

Badge

Embed ▾

Committed 30 Sep 2024 12:56PM UTC coverage: 74.505% (+0.001%) from 74.504%

Job # 11106562150.1

Build Type

push

github

Committed by

web-flow

Commit Message

Modify APIs to return palloc'ed values and catalog lookup to validate the user in get_physical_user_name (#2960)

Earlier in Babelfish, The get_*_role_name() and get_*_schema_name() APIs returned constant values or string literal for
few cases and palloc'ed copy for other cases. The get_physical_user_name() used to return the user_name after
appending the db_name to the user_name without any checks to verify its existence/validity.

With this change, We return Palloc'ed values for the API every time. In get_physical_user_name() if the user or role is
not found in the sys.babelfish_authid_user_ext catalog, then an error is thrown. Additionally, In multidb.c, made
changes to the APIs like get_role_name() and get_schema_name() to return palloc'ed values instead of constant
values. 

Implemented a catalog lookup in get_physical_user_name() to verify if the user or role is found in the
sys.babelfish_authid_user_ext catalog. The 'suppress_role_error' flag indicates if it is ok for the user or role to be
absent from the catalog (required for cases when a new user/role is created and it can't be found in the catalog).

Checked all the functions where the APIs are called and pfree the variables containing values from the above APIs after they are no longer used.

Task: BABEL-4933
Signed-off-by: P Aswini Kumar <aswinii@amazon.com>

Run Details

45062 of 60482 relevant lines covered (74.5%)

162407.76 hits per line

babelfish-for-postgresql / babelfish_extensions / 11106562150 / 1
76%
BABEL_5_X_DEV: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11106562150.1

babelfish-for-postgresql / babelfish_extensions / 11106562150 / 1 76% BABEL_5_X_DEV: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11106562150.1

babelfish-for-postgresql / babelfish_extensions / 11106562150 / 1
76%
BABEL_5_X_DEV: 77%

README BADGES
x