Skip to content

Comments

Add plugin sign capabaility#176

Merged
SteveLasker merged 16 commits intonotaryproject:mainfrom
qmuntal:feat-plugin
May 19, 2022
Merged

Add plugin sign capabaility#176
SteveLasker merged 16 commits intonotaryproject:mainfrom
qmuntal:feat-plugin

Conversation

@qmuntal
Copy link
Member

@qmuntal qmuntal commented May 13, 2022

This PR adds support for using external keys to sign images with the plugins.

qmuntal added 12 commits May 18, 2022 13:44
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
qmuntal added 3 commits May 18, 2022 13:57
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Co-authored-by: Milind Gokarn <milind81@gmail.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
@qmuntal qmuntal requested a review from gokarnm May 19, 2022 12:27
Copy link
Contributor

@gokarnm gokarnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@SteveLasker SteveLasker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SteveLasker SteveLasker merged commit c61102c into notaryproject:main May 19, 2022
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
* support plugin keys

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* update plugins branch

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* implement plugin signing

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* pass PluginConfig

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* bump notation-go

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* upgrade notation-go

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* upgrade notation-go

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* bump notation-go

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* revert name change

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* remove unnecessary casts

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* remove unnecessary casts

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* simplify isDefaultKey

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* revert unnecessary change

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* support config maps

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* switch to notaryproject/notation-go@main

Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

* Apply suggestions from code review

Co-authored-by: Milind Gokarn <milind81@gmail.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>

Co-authored-by: Milind Gokarn <milind81@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants