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

Notarization fails #61

Open
sphen13 opened this issue Jul 11, 2024 · 6 comments
Open

Notarization fails #61

sphen13 opened this issue Jul 11, 2024 · 6 comments
Assignees

Comments

@sphen13
Copy link
Contributor

sphen13 commented Jul 11, 2024

Seems #60 caused a regression in behavior. When i try to use munki-rebrand, signing as usual and then submitting for notarization it fails with a laundry list of issues. reverting to the previous commit fixes the behavior. This is with the latest 6.5.1 munki.

@sphen13
Copy link
Contributor Author

sphen13 commented Jul 11, 2024

here are the apple logs for the failed run (current code) and the successful run (one commit back)

fail:
https://gist.github.com/sphen13/40ad3d95ebb029b467cf46627646c041

success:
https://gist.github.com/sphen13/b073a9b91caef0cf39899dee510a0094

@fuzzylogiq
Copy link
Collaborator

Please can you provide the exact command line you used (you can redact anything sensitive)?

@sphen13
Copy link
Contributor Author

sphen13 commented Jul 11, 2024

sure -

/Users/sphen/src/munki-rebrand/munki_rebrand.py -a "Third Vantage Software Center" -i /path/to/Icon-1024.png -o "/Users/sphen/src/thirdvantage/Munki/ThirdVantageSC_${clientForAgent}" -s "Developer ID Installer: XXX" -S "Developer ID Application: XXX" -p "/Users/sphen/src/thirdvantage/Munki/postinstall_$clientForAgent"

@fuzzylogiq fuzzylogiq self-assigned this Jul 12, 2024
@fuzzylogiq
Copy link
Collaborator

Thanks. Can recreate locally. I need to see if I can fix both notarization & @SteveKueng 's issue with mixed signing in Python by experimenting with the signing options. For now, please use the previous release.

@sphen13
Copy link
Contributor Author

sphen13 commented Jul 12, 2024

Tried to help a bit here - thought i would add back deep=True to the signing commands. This resolved a large amount of errors but some remain still specific to entitlements etc. I am unable to duplicate @SteveKueng 's initial issue as when i run my signed version of munki-python, importing modules seems to work fine? so i dont really want to troubleshoot this if i cant duplicate his initial issue.

https://gist.github.com/sphen13/be340516c22451058d29308068001bb6

@sphen13
Copy link
Contributor Author

sphen13 commented Aug 22, 2024

this has been resolved in 71223e5

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

No branches or pull requests

2 participants