-
Notifications
You must be signed in to change notification settings - Fork 44
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
Add key creation, through the plugin spec #163
Comments
@SteveLasker , @dtzar - Shouldn't we keep the key creation outside of Notation. I agree the Notation should somehow refer to those keys that are used for signing , but the creation scan itself be outside of Notation. cc: @gokarnm |
What Steve is referring to here though is the core plugin capability being available for |
Yes to both. It's really a matter of enabling the user to ask for a new key, through |
The details of key creation, and its prerequisites are specific to each provider. Key creation may be done by a different persona, outside of the signing workflow, using provider specific tools/APIs, and may involve setting up polices for key access and rotation. It would be preferable to create keys outside notation and only use it for signing within the tool. Do we have specific scenarios this would unblock, simplify? |
Simply looking to keep the getting started process easy, and let the complex be complex. |
@gokarnm - I think it's fair to keep out of scope read/update/delete here since the command we're targeting is It would make sense if we could use the same plugin system already there (i.e. one AWS plugin or one Azure plugin), but just extend it to be able to generate a certificate via this sub-command. |
It certainly would be a better user experience to not have to download a plugin to get a cert from the provider, but that said then you introduce a ton of dependencies into notation cli itself for every provider who wants to integrate and you lose the agility/speed to update releases of the integration independently. An idea might be to make the installation of the plugin super simple and embed this in the notation cli. For instance: I feel a one-liner addition like this is a good compromise. Another alternative would be for people to independently package the notation bits along with the plugin together, but that's a lot of maintenance work on every plugin provider to maintain in notaryproject org repo(s). |
As discussed in the community meeting "09/15" to be moved to Post RC1. |
@SteveLasker Is this issue still valid since no activities for almost one year? |
I defer to the providers to decide the importance. We can close this, pending their request. |
Various keyvault providers have key vault specific APIs to create their keys.
Rather than have notary/notation understand each key vault provider, Notary follows a plugin specification, enabling plugins to define their behavior for sign, without having to engage with the notary maintainers, or ask for approval.
This issue captures the ability to call
notation cert generate --plugin <name>
with a set of parameters. This would enable common means for users, while the notation plugin can adapt to any notation specific formatting.The text was updated successfully, but these errors were encountered: