Skip to content

Conversation

@philips
Copy link
Contributor

@philips philips commented Nov 20, 2024

Summary

gitsign differs from GPG in that it requires an internet access on every commit. This breaks the git model of working offline. So, probably worth calling out.

Release Note

NONE

Documentation

@cpanato
Copy link
Member

cpanato commented Nov 21, 2024

thanks @philips can you please sign the DCO

gitsign differs from GPG in that it requires an internet access on every
commit. This breaks the git model of working offline. So, probably worth
calling out.

Signed-off-by: Brandon Philips <brandon@ifup.org>
@philips
Copy link
Contributor Author

philips commented Nov 21, 2024

@cpanato signed DCO



# Warning: Setting this will make git commit/tag reliant on internet.
# Alternatively, don't use these settings and add the -S flag instead.
Copy link
Member

Choose a reason for hiding this comment

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

The -S flag for commits will still rely on the internet, no? I wonder if this message belongs elsewhere more prominently for gitsign use as a whole?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, explaining that every signed commit now reaches out to the internet might be worth mentioning somewhere. I had setup this setting and then couldn't figure out why commits were happening so slowly.

Copy link
Member

Choose a reason for hiding this comment

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

perhaps adding some text in the signing commits and signing tags sections makes sense? https://github.com/sigstore/gitsign/blob/main/README.md?plain=1#L115

Copy link
Member

Choose a reason for hiding this comment

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

will be good to add some clarification in this section as well

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

seems ok to me

thanks

@cpanato cpanato merged commit 6914495 into sigstore:main Nov 30, 2024
7 checks passed
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