search

entechlog / dbt-snow-mask

This repository contains source code for dbt package dbt_snow_mask.
https://hub.getdbt.com/entechlog/dbt_snow_mask/latest/
GNU General Public License v3.0
60 stars 25 forks source link

Fix masking policy schema only, not db #31

Closed mbritton-neuroflow closed 2 years ago

mbritton-neuroflow commented 2 years ago

Hello, thanks for this great package!

I would like to use the use_common_masking_policy_db functionality to fix only the schema, but not the database. So for example my masking policies will always live in the public schema, but I want to maintain separation between my prod and dev environments. Otherwise as I understand it testing out a new masking policy definition in dev could update my prod environment!

If my understanding is correct, and this change fits in with your product vision, I'd be happy to contribute a fork. The line in question is here, should be a pretty simple change.

entechlog commented 2 years ago

@mbritton-neuroflow I believe the functionality you are looking for has been added here, can you please check and confirm when you have a few .

Release candidate https://github.com/entechlog/dbt-snow-mask/releases/tag/0.2.0rc1. 

mbritton-neuroflow commented 2 years ago

Hi! Yes, that looks like it accomplishes what I was looking for. I am OOO until June 13th but I will test when I return. Thank you!

entechlog commented 2 years ago

@mbritton-neuroflow , Thank You and will wait for your test feedback before I close this issue

mbritton-neuroflow commented 2 years ago

Tested and works great. Thank you again!