Add additional docs to clarify key generation and format for config file - #1719
Add additional docs to clarify key generation and format for config file#1719wgoulet wants to merge 1 commit into
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
/gcbrun |
|
Agreed with you that this is how to convert the keys to a hex format that can be specified in the config. It should be possible to pass the keys directly with the pem file though, without having to go through the whole conversion: https://github.com/wgoulet/certificate-transparency-go/blob/551ff912277a60b5fef9f4c6379084ac7aa27b5b/trillian/docs/ManualDeployment.md#ctfe-configuration Is the hex hex key required anywhere else? |
|
@phbnf At least for the Trillian deployment I was able to deploy using these instructions, I only needed to put the hex hex keys in the proto config file for ctfe to get the service running. I didn't have to deviate from instructions to get the log signer or server running. |
|
Yes indeed, I tried to use the file directly.. and it did not work. I could not get the public key in, and eventually ran into https://github.com/google/trillian/blob/master/crypto/keys/pem/pem.go#L35 for the private key. Many thanks for flagging this. For the record, here is the matching command for the public key, it's ever so slightly different:
Two things of note:
|
Hola |
This is a documentation enhancement to help folks understand how to convert the log public/private keys into the correct format in the ctfe config file. It took a lot of trial and error for me to figure out how to do this and I finally dug up this comment in an older open issue: #780.
Checklist