mirror of
https://github.com/FiloSottile/mkcert.git
synced 2026-04-25 05:26:03 +03:00
[PR #46] [MERGED] Add Windows Support #369
Labels
No labels
TLS stack issue
Windows
bug
duplicate
duplicate
enhancement
help wanted
help wanted
pull-request
question
question
root store
waiting for info
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/mkcert#369
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/FiloSottile/mkcert/pull/46
Author: @cretz
Created: 7/11/2018
Status: ✅ Merged
Merged: 8/13/2018
Merged by: @FiloSottile
Base:
master← Head:windows-support📝 Commits (2)
a218f37Add Windows support to mkcert5dd27e6Use serial number to identify cert for deletion on Windows📊 Changes
1 file changed (+130 additions, -0 deletions)
View changed files
➕
truststore_windows.go(+130 -0)📄 Description
This is to address issue #42. This uses the win32 crypt API which is what Go uses elsewhere when interfacing with the OS cert store. Also, though there is a "certutil.exe" in Windows, it's not the NSS one and therefore while I reference the FF paths, I left
hasCertutilas false for this OS incert.go.I took care to make the smallest, self-contained changes I could here for minimal Windows support. However as I mention in #45, this code could benefit from a lot of refactoring and "library-ification". I don't like panicking on error, returning bools, hardcoding CA org name, etc.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.