Update patch set 6

Browse Source

Patch Set 6: Code-Review+1

Still looks good to me.

On the topic of removing the ConfKeyManager, I disagree -- I think there is a place for it here, because it is the local-dev option that will allow for testing without access to a real backend (like Barbican). I am planning on adding something similar with my CertManager CR (LocalCertManager, which stores certs/pk/etc in files on the local FS). It is horribly insecure and obviously not intended for production use, but is very useful for development and debugging purposes. I will definitely push for having those types of implementations available.

Patch-set: 6

...

This commit is contained in:

Gerrit User 10273 2015-01-30 22:49:33 +00:00 committed by Gerrit Code Review

parent cd653a6383

commit 3466b892d8

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Show Stats Download Patch File Download Diff File Expand all files Collapse all files

Diff Content Not Available