Fix TLS configuration (#3010)

The Server struct was refactored in https://github.com/stellar/go/pull/2958/files so that the TLS configuration was stored in ServerConfig. However, the ServerConfig instance was never assigned in the NewServer() constructor.
stellar · Sep 15, 2020 · 2b2e015 · 2b2e015
1 parent db9763c
commit 2b2e015
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/services/horizon/CHANGELOG.md b/services/horizon/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this
 file. This project adheres to [Semantic Versioning](http://semver.org/).x
 
+## v1.8.2
+
+* Fixed a bug which prevented Horizon from accepting TLS connections.
+
 ## v1.8.1
 
 * Fixed a bug in a code ingesting fee bump transactions.

diff --git a/services/horizon/internal/httpx/server.go b/services/horizon/internal/httpx/server.go
@@ -74,6 +74,7 @@ func NewServer(serverConfig ServerConfig, routerConfig RouterConfig) (*Server, e
 	result := &Server{
 		Router:  router,
 		Metrics: sm,
+		config:  serverConfig,
 		server: &http.Server{
 			Addr:        addr,
 			Handler:     router,