Skip to content

Add the ability to create keys from certificate private keys#5

Open
pschou wants to merge 12 commits into
kayrus:masterfrom
pschou:master
Open

Add the ability to create keys from certificate private keys#5
pschou wants to merge 12 commits into
kayrus:masterfrom
pschou:master

Conversation

@pschou

@pschou pschou commented Sep 18, 2022

Copy link
Copy Markdown

This PR allows your tool to create ppk files from a given private key. I also rounded out the ability to read, write, and verify all the putty version formats (SSH1 1.1, and the more current 1,2, and 3).

@kayrus

kayrus commented Sep 18, 2022

Copy link
Copy Markdown
Owner

Thanks for the PR. It's quite a big change and I need a bigger free time slot to review it.

@kayrus kayrus left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the first code review. The rest will submit later.

Comment thread dsa.go Outdated
Comment thread marshal.go Outdated
Comment thread marshal.go Outdated
Comment thread marshal.go
Comment thread marshal.go Outdated
Comment thread putty.go Outdated
Comment thread putty.go Outdated
Comment thread putty.go Outdated
Comment thread putty.go Outdated
Comment thread putty.go Outdated
@pschou

pschou commented Sep 18, 2022

Copy link
Copy Markdown
Author

@kayrus I am sorry for the go test not passing on the PR on the first submission. I thought I had pushed my latest change on disk and blindly did the PR before doing the commit and push. I appreciate your patience. The test should be good now!

@pschou

pschou commented Oct 1, 2022

Copy link
Copy Markdown
Author

@kayrus: Just a heads up here, I finished implementing the SSH1 read and writing routines, so this library can thoroughly read and write every possible putty key format!

All the best!

@kayrus

kayrus commented Jul 20, 2023

Copy link
Copy Markdown
Owner

@pschou I apologies for the delay. Could you please rebase your branch to resolve the conflicts? I'll try to review this PR ASAP.

@kayrus kayrus force-pushed the master branch 4 times, most recently from 00f4caf to 4ee737f Compare March 13, 2025 14:54
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.

2 participants