Skip to content
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

Automate validation of Nuget signatures #1881

Closed
danmoseley opened this issue Jan 24, 2019 · 8 comments
Closed

Automate validation of Nuget signatures #1881

danmoseley opened this issue Jan 24, 2019 · 8 comments

Comments

@danmoseley
Copy link
Member

danmoseley commented Jan 24, 2019

We need to automate validation that our NuGet packages are signed.

Specifically this means running nuget verify -all on every nupkg after signing completes. If that fails, upload will fail later.

We should not rely on remembering to do this manually. Perhaps this is a dupe, but I can't find it.

@mmitche @shawnro

@jcagme
Copy link
Contributor

jcagme commented Jan 24, 2019

@danmosemsft it is nuget verify, right?

FYI @JohnTortugo

@JohnTortugo
Copy link
Contributor

We have a tool to perform that validation (also including the package content). The plan is to run the tool just before publishing the packages: #1444

@danmoseley
Copy link
Member Author

@danmosemsft it is nuget verify, right?

fixed, thanks

@markwilkie
Copy link
Member

related to #1933

@JohnTortugo
Copy link
Contributor

I'm wondering if we still need nuget verify if we run SignCheck. AFAIU SignCheck already perform .nupkg signing validation and much more. What do you think @joeloff @tmat

@JohnTortugo JohnTortugo self-assigned this Mar 28, 2019
@joeloff
Copy link
Member

joeloff commented Mar 28, 2019

Probably not. SignCheck calls into the NuGet APIs to verify the package signature and it will also check the contents inside the nupkg (assuming you're running with -r). Right now for nupkgs it specifically checks that it was signed by PRSS, which it should be if it's something we're producing.

@JohnTortugo
Copy link
Contributor

I'm closing this issue as SignCheck already perform such validation and we're going to use it in our builds.

@danmoseley
Copy link
Member Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants