-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(secrets): enable user-supplied storage and authproxy secrets #189
feat(secrets): enable user-supplied storage and authproxy secrets #189
Conversation
e585fee
to
562710e
Compare
I guess we are waiting on response in #187? Maybe we can move forward and when the all changes are merged on |
Yes, I say we go ahead. It would be easier for Grzegorz to test anyway if it was merged, I think, and if there are any other fixes required we can do them as separate PRs. |
6850b16
to
0d1dea6
Compare
fad64c0
to
0d0b633
Compare
Hmm some issues during rebasing? Some conflicts seem to be reported and the change diffs are a bit odd :D |
Ah, I think I just rebased it onto main instead of the separate-db-storage branch. |
0d1dea6
to
bf2af72
Compare
0d0b633
to
fdbc6b9
Compare
bf2af72
to
b962ec5
Compare
b962ec5
to
22009bb
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 good to me :D
Fixes #188