Closed JulianGriggs closed 5 years ago
Argh, "update branch" used a merge commit. @JulianGriggs mind if I revert, rebase, and force push the branch?
Also, are the code changes ready for review? I'm happy to take a look if so :)
@MatthewDolan @iredelmeier This is ready for review. I tested this via the following:
user:~$ certstrap init --common-name "Satellite Testing Cert Auth"
user:~$ certstrap request-cert --domain "localhost"
user:~$ certstrap sign localhost --CA Satellite_Testing_Cert_Auth
user:~$ cp ./out/localhost.key ./out/localhost.key.pem
user:~$ cp ./out/localhost.crt ./out/localhost.bundle.pem
user:~$ sendspan -access_token [token] -collector_host localhost -collector_port 9997 -operation_name testGRPCSecure -secure=true -transport grpc -custom_ca_cert_file ./out/Satellite_Testing_Cert_Auth.crt
user:~$ sendspan -access_token [token] -collector_host localhost -collector_port 9997 -operation_name testHTTPSecure -secure=true -transport http -custom_ca_cert_file ./out/Satellite_Testing_Cert_Auth.crt
The objective is to be able to support TLS communication with a Satellite signed by a non-system default Root CA.