phoenix icon indicating copy to clipboard operation
phoenix copied to clipboard

PHOENIX-6579 ACL check doesn't honor the namespace mapping for mapped…

Open ss77892 opened this issue 3 years ago • 3 comments

… views.

ss77892 avatar Oct 27 '21 21:10 ss77892

Thanks for the submission. I think the changes look okay but can you please add a test to ensure this doesn't regress in the future?

dbwong avatar Oct 27 '21 22:10 dbwong

retest

ss77892 avatar Oct 28 '21 19:10 ss77892

This got me thinking that we should in fact use "^"+tableName.getNameWithNamespaceInclAsString()+"$"

to avoid a similar stituation, where we have permission for ns:prefixTable, and we do not add the required new permission for ns:prefixTableFoo because we THINK we already have it.

I think I'm going to open am HBase ticket about the horrific API design (if one doesn't exist yet).

stoty avatar Jan 07 '22 06:01 stoty

Closing as this has been merged.

stoty avatar Apr 03 '23 14:04 stoty