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

Feature: Sign all dlls to pass Microsoft Windows Smart App control #3131

Merged
merged 10 commits into from
Oct 16, 2023

Conversation

infeo
Copy link
Member

@infeo infeo commented Sep 28, 2023

Closes #3130.

This PR consists of two parts:

  1. Similarly to macOS, we extract and sign all shared libraries which are shipped in a jar dependency
  2. We explicitly sign the wixhelper.dll provided by the JDK and used by jpackage with our certificate and use it during installer creation

@infeo infeo self-assigned this Sep 28, 2023
@infeo infeo added this to the 1.11.0 milestone Sep 28, 2023
@infeo
Copy link
Member Author

infeo commented Oct 11, 2023

@overheadhunter In 9b55f6f I refactored the extraction and signing into seperate steps. This way, we only need to execute the signing action once for all files.

@infeo infeo merged commit 560a979 into develop Oct 16, 2023
10 checks passed
@infeo infeo deleted the feature/sign-all-dlls branch October 16, 2023 14:21
@overheadhunter overheadhunter added the type:enhancement Improvements on an existing feature label Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement Improvements on an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows: MSI Installer blocked by Smart App Control
2 participants