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

Make signing and verification use stricter checks #6

Closed
wants to merge 1 commit into from

Conversation

bkero
Copy link

@bkero bkero commented Jul 3, 2018

This commit adds some new flags to the codesigning and checking portion of Sparkle for the purposes of stronger security.

Closes #5

@w0ts0n w0ts0n requested a review from RyanJarv July 11, 2018 16:45
@RyanJarv
Copy link

Just an fyi this isn't a blocker for the unofficial release.

https://github.com/brave/internal/issues/378

@RyanJarv
Copy link

We still need to investigate what issues this could potentially cause and why it's not included upstream by default.

@mbacchi mbacchi requested review from riastradh-brave and mbacchi and removed request for RyanJarv June 5, 2019 20:11
@mihaiplesa mihaiplesa self-requested a review February 20, 2020 18:02
@mihaiplesa mihaiplesa closed this Nov 25, 2020
@mihaiplesa mihaiplesa deleted the branch brave:master November 25, 2020 07:58
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.

Consider doing deeper Apple code signing verification
3 participants