-
Notifications
You must be signed in to change notification settings - Fork 22
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
initial test at signing with sigstore #43
base: master
Are you sure you want to change the base?
Conversation
with the following commit, IT WORKS
adding one additional shortcoming to solve: we need o avoid |
tested today with a multi-module build: currently, each module (having its own classloader) has its own sigstore signature sessions, then an OIDC flow. Not the best UX, but IMHO the 10 minutes timespan is the most important shortcoming issue to solve, that will also solve the multi-module experience another key aspect to solve is adding sigstore signature verification to pgpverify-maven-plugin or equivalent. From a UX perspective, filing the keys map (which in sisgstore is not a key id but an OIDC identity) with checked data remains hard... |
remaining known limitation: OIDC auth required on each module, even when plugin put as extension |
@hboutemy can this PR be removed? Is this still valid? |
@hboutemy ping ^ |
PoC using sigstore-java that does all the heavy sigstore work: https://github.com/sigstore/sigstore-java
GpgSignAttachedMojo.java
logic to createSigstoreSignAttachedMojo.java
(I will probably refactorGpgSignAttachedMojo.java
later to make algorithm independant from signature details more visible: extract FilesCollector #45 )