Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Programmatically generate ssl certs #868

Closed
cpacia opened this issue Feb 19, 2018 · 4 comments · Fixed by #944
Closed

Programmatically generate ssl certs #868

cpacia opened this issue Feb 19, 2018 · 4 comments · Fixed by #944
Labels

Comments

@cpacia
Copy link
Member

cpacia commented Feb 19, 2018

Create a gencerts command (cmd/gencerts.go) that will automatically generate self signed SSL certificates that can be used by the OpenBazaar electron client running on a remote machine. This is to make it easier for users to set up an instance on a remote machine and avoid manually typing openssl commands.

I gave it a go here but could not get the client to accept the generated cert.

@chaintip $50

@cpacia cpacia added the bounty label Feb 19, 2018
@tea-spoon
Copy link

tea-spoon commented Feb 20, 2018

Does someone have to build this into the existing codebase? Are you able to provide a small test folder with function stubs that contain comments with the input/output you expect? Once operational, this could then be moved into a functioning location within the repository later.

@cpacia
Copy link
Member Author

cpacia commented Feb 20, 2018

Well the goal is to build it into the codebase but if you can make the functions work it shouldn't really be that hard to do it.

@ab10460ef3
Copy link

@cpacia, I refactored your code to fit with the changes in cmds.

https://github.com/ab10460ef3/openbazaar-go/tree/gencertsRSA

I had the same issue, client rejects self signed cert. I think generating a CA and signing request might be necessary, or at least better received by clients.

The only reason I switched to RSA is to make it as similar as possible to the current recommended commands.

If anyone wants to pickup the torch, everything that can be done with certificates is done here:

https://github.com/cloudflare/cfssl

@chaintip
Copy link

0.03317627 BCH| ~ 38.60 USD has now been claimed by @trigun117 when pull request #944 was merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
4 participants