-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DPE-2447] Secret labels #200
Conversation
d880bfb
to
197e10c
Compare
ea1e3e3
to
56ef157
Compare
56ef157
to
061ca77
Compare
061ca77
to
cbe98fc
Compare
cbe98fc
to
af79f51
Compare
af79f51
to
49ce711
Compare
49ce711
to
ddfd961
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks AMAZING Judit, thank you so much for your dedication and help here.
Mostly of my comments are questions; but there are a few nits and a requested change
b8ca0ec
b8ca0ec
to
f5229e7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really awesome work Judit, thank you very much!
Using generated secret labels instead of storing the ID locally.
Also: major refactor to clean up secrets code to a better shape, standard across DP-libs (and more-or-less Opensearch and who knows, perhaps in the future a few more :) ).