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

installer: Add code signing certificate before installing the driver #100

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rozmansi
Copy link
Contributor

This avoids prompts on Windows 7 (with KB2921916 applied), 8, 8.1, Server 2008R2, 2012R2. Note there is no prompt on Windows 10 and Server 2016 and 2019 already as the driver for Win10 is Microsoft signed.

Rather than shipping separate installers for Windows 10 and pre Windows
10, pack all Windows driver and extend selection logic to install
correct one.

Signed-off-by: Simon Rozman <[email protected]>
This avoids prompts on Windows 7 (with KB2921916 applied), 8, 8.1,
Server 2008R2, 2012R2. Note there is no prompt on Windows 10 and Server
2016 and 2019 already as the driver for Win10 is Microsoft signed.

Signed-off-by: Simon Rozman <[email protected]>
@mattock
Copy link
Member

mattock commented Nov 10, 2019

I'll test this on the Windows 10 ARM64 laptop when I get back home. As discussed in the hackathon I'll create a new combined tap-windows6 installer based on the latest signed tap-windows6 drivers to get the benefits immediately.

@agrawalamit2005
Copy link

agrawalamit2005 commented Nov 12, 2019

Hi rozmansi, mattock

Could you confirm with any of win7 machine if such cert is installed or the reg entry is created once user opt for Always Trust .... publisher checkbox. In my case, i do not see any impact of the reg entry created/cert installation for trusted publisher (tried in both stores) prompt.

Thanks

@mattock
Copy link
Member

mattock commented Nov 12, 2019

@agrawalamit2005 are you saying that even if you have clicked "Trust this publisher" you get the same prompt when you install/upgrade tap-windows6 again?

@agrawalamit2005
Copy link

@mattock I have not tried on tap-windows6 yet. Please read my comment more as a question on approach used to avoid Trust prompt. With other driver, i notices similar prompt but i do not see any entry created in registry at Trusted publisher place. Have you seen this entry?

@rozmansi
Copy link
Contributor Author

This avoids prompts on Windows 7 (with KB2921916 applied)

Windows 7 really really really needs the KB2921916 for their driver install prompt to work correctly with SHA-256 driver signatures.

Windows 7 without KB2921916 will keep prompting - regardless of what certificate we import and regardless how many times you tick that "Don't prompt again for this publisher" checkbox.

Could you confirm with any of win7 machine if such cert is installed or the reg entry is created once user opt for Always Trust .... publisher checkbox. In my case, i do not see any impact of the reg entry created/cert installation for trusted publisher (tried in both stores) prompt.

Yes, I can confirm this works without a prompt on Windows 7 with KB2921916. I tested it personally. I have tested it again once and for all - this time recording:

  1. Installing on a Win7 without KB2921916: https://1drv.ms/u/s!AsRKV9itoeUTiz4NspnipAuziWuE?e=xcNHAm
  2. Reverting to the snapshot before TAP-Windows6 was installed.
  3. Installing KB2921916: https://1drv.ms/u/s!AsRKV9itoeUTiz-ESCzDlYsXvd_S?e=C5jsGW
  4. Rebooting
  5. Installing TAP-Windows6 again: https://1drv.ms/u/s!AsRKV9itoeUTi0Aa9cJ-wfKKMs6G?e=u8dZRE

@agrawalamit2005
Copy link

agrawalamit2005 commented Nov 13, 2019 via email

@rozmansi
Copy link
Contributor Author

I have downloaded the KB2921916 for testing purposes here:

I am not sure if OpenVPN community is legally entitled to host the download. At least not without double-checking the license that was included with the original download at Microsoft Download Server (no longer available).

Without a license, I don't believe we are legally entitled to include it in our TAP-Windows6 installer and deploy it.

@rozmansi
Copy link
Contributor Author

One thing, I probably should mention explicitly... This PR includes #99, since it reuses its logic to detect if Windows version is <10.

@mattock mattock mentioned this pull request Mar 23, 2020
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