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

feat(bundler/nsis): sign uninstaller, closes #7348 #7398

Merged
merged 4 commits into from
Jul 11, 2023

Conversation

amrbashir
Copy link
Member

@amrbashir amrbashir commented Jul 11, 2023

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Docs
  • New Binding issue #___
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes, and the changes were approved in issue #___
  • No

Checklist

  • When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
  • A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
  • I have added a convincing reason for adding this feature, if necessary

Other information

@amrbashir amrbashir requested a review from a team as a code owner July 11, 2023 00:24
@amrbashir
Copy link
Member Author

This is not tested yet, so I will convert to a draft

@amrbashir amrbashir marked this pull request as draft July 11, 2023 00:26
@amrbashir
Copy link
Member Author

I am having a bit of a hard time trying to generate a signing certificate to test. @FabianLars if you have one, could you test this PR?

@FabianLars
Copy link
Member

So, according to the logs all went well:

Finished dev [unoptimized + debuginfo] target(s) in 1m 21s
        Info Target: x64
     Signing C:\Users\Fabian-Lars\dev\_target\debug\tauri-app2.exe
     Signing C:\Users\Fabian-Lars\dev\_target\debug\tauri-app2.exe with identity "88FB696BE8176B9668F55E027D5AD83027495749"
        Info "Done Adding Additional Store\r\nSuccessfully signed: C:\\Users\\Fabian-Lars\\dev\\_target\\debug\\tauri-app2.exe\r\r\n"
 Downloading https://go.microsoft.com/fwlink/p/?LinkId=2124703
     Running candle for "main.wxs"
     Running light to produce C:\Users\Fabian-Lars\dev\_target\debug\bundle\msi\tauri-app2_0.1.0_x64_en-US.msi
     Signing C:\Users\Fabian-Lars\dev\_target\debug\bundle\msi\tauri-app2_0.1.0_x64_en-US.msi
     Signing C:\Users\Fabian-Lars\dev\_target\debug\bundle/msi/tauri-app2_0.1.0_x64_en-US.msi with identity "88FB696BE8176B9668F55E027D5AD83027495749"
        Info "Done Adding Additional Store\r\nSuccessfully signed: C:\\Users\\Fabian-Lars\\dev\\_target\\debug\\bundle/msi/tauri-app2_0.1.0_x64_en-US.msi\r\r\n"
        Info Target: x64
     Signing C:\Users\Fabian-Lars\dev\_target\debug\tauri-app2.exe
     Signing C:\Users\Fabian-Lars\dev\_target\debug\tauri-app2.exe with identity "88FB696BE8176B9668F55E027D5AD83027495749"
        Info "Done Adding Additional Store\r\nSuccessfully signed: C:\\Users\\Fabian-Lars\\dev\\_target\\debug\\tauri-app2.exe\r\r\n"
 Downloading https://go.microsoft.com/fwlink/p/?LinkId=2124703
     Running makensis.exe to produce C:\Users\Fabian-Lars\dev\_target\debug\bundle\nsis\tauri-app2_0.1.0_x64-setup.exe
Done Adding Additional Store
Successfully signed: C:\Users\FABIAN~1\AppData\Local\Temp\nst3D2C.tmp
     Signing C:\Users\Fabian-Lars\dev\_target\debug\bundle\nsis\tauri-app2_0.1.0_x64-setup.exe
     Signing C:\Users\Fabian-Lars\dev\_target\debug\bundle/nsis/tauri-app2_0.1.0_x64-setup.exe with identity "88FB696BE8176B9668F55E027D5AD83027495749"
        Info "Done Adding Additional Store\r\nSuccessfully signed: C:\\Users\\Fabian-Lars\\dev\\_target\\debug\\bundle/nsis/tauri-app2_0.1.0_x64-setup.exe\r\r\n"
    Finished 2 bundles at:
        C:\Users\Fabian-Lars\dev\_target\debug\bundle\msi\tauri-app2_0.1.0_x64_en-US.msi
        C:\Users\Fabian-Lars\dev\_target\debug\bundle\nsis\tauri-app2_0.1.0_x64-setup.exe

Gonna try and see if i can confirm that somehow...

@FabianLars
Copy link
Member

Well, at least according to the "digital signatures" (translated from german) tab in the file properties of the uninstall.exe it correctly lists the cert like it does on the installer & app. 🥳

@jetli
Copy link
Contributor

jetli commented Jul 11, 2023

I tested this pr too, confirm that the uninstall.exe is signed correctly, thanks! and with the few latest fixes, NSIS works great now!

@amrbashir amrbashir marked this pull request as ready for review July 11, 2023 12:14
@amrbashir
Copy link
Member Author

amrbashir commented Jul 11, 2023

thanks for testing, I marked this as ready for review now

@amrbashir amrbashir merged commit 764968a into 1.x Jul 11, 2023
@amrbashir amrbashir deleted the feat/nsis/sign-uninstaller branch July 11, 2023 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔎 In audit
Development

Successfully merging this pull request may close these issues.

3 participants