-
Notifications
You must be signed in to change notification settings - Fork 241
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Trillian TLS support for ct_server #1551
Conversation
Signed-off-by: Firas Ghanmi <fghanmi@redhat.com>
Signed-off-by: Firas Ghanmi <fghanmi@redhat.com>
@fghanmi Do you have any use case to support TLS in migrillian? If so, we can consider keeping the TLS support there. |
Gotcha. This PR looks good to me to remove it from migrillian if there is no such a use case at the moment. |
Signed-off-by: Firas Ghanmi <fghanmi@redhat.com>
* Add Trillian TLS support for ct_server Signed-off-by: Firas Ghanmi <fghanmi@redhat.com> * update CHANGELOG.md Signed-off-by: Firas Ghanmi <fghanmi@redhat.com> * update trillianTLSCACertFile variable name Signed-off-by: Firas Ghanmi <fghanmi@redhat.com> --------- Signed-off-by: Firas Ghanmi <fghanmi@redhat.com>
Add Trillian TLS support for ct_server (google#1551)
Add Trillian TLS support for ct_server:
In a previous PR: #1525, this change was intended but was done on the wrong file
trillian/migrillian/main.go
instead oftrillian/ctfe/ct_server/main.go
This issue was identified while testing a real scenario. It should be fixed by this PR.
related isssue: #1524
Checklist