-
Notifications
You must be signed in to change notification settings - Fork 268
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
sha1 #394
base: master
Are you sure you want to change the base?
sha1 #394
Conversation
michalskrivanek
commented
May 25, 2022
- certificates: fix confirmation question
- [WIP] remove SHA-1 from cryptotool
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.
LGTM
+1 for the packaging part. I do not know well enough the cryptotool to say much. Patch makes sense. Do we need in it any kind of backwards compatibility? How does this work on upgrades? etc. |
yeah, backward compatibility is a problem. It probably needs recreation of all engine-side credentials (e.g. for ovn provider and such) |
cc8ef4b
to
5026d31
Compare
Needs a similar change in aaa-jdbc