Visitar URL original
feat: Add TLS support for Go Feature Server by shuchu · Pull Request #6229 · feast-dev/feast · GitHub
Skip to content

feat: Add TLS support for Go Feature Server #6229

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.

Sign up for GitHub

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

Merged
franciscojavierarceo merged 10 commits into from
Apr 14, 2026
Merged

feat: Add TLS support for Go Feature Server #6229

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
7dce12f
feat: Add HTTPs server support.
shuchu Mar 15, 2026
064f612
feat: Add serverTSL() method.
shuchu Apr 5, 2026
5d64fcb
fix: Lint the Go code.
shuchu Apr 6, 2026
ed19573
feat: Add TLS configuration.
shuchu Apr 6, 2026
2ca0bba
fix: Add unit tests for HTTPS server.
shuchu Apr 10, 2026
531042a
fix: Format code. Fixed bad comments.
shuchu Apr 10, 2026
d87d895
fix: Fix unit test error.
shuchu Apr 10, 2026
c6f5d69
fix: use single StartHttpServer to cover HTTP and HTTPs cases.
shuchu Apr 12, 2026
f3d93c5
fix: Change back the pyproject.toml file.
shuchu Apr 12, 2026
03d7156
fix: Fix name convertion of https_enable.
shuchu Apr 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix: Fix name convertion of https_enable. 
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
  • Loading branch information
@shuchu SHUCHU HAN
shuchu authored and SHUCHU HAN committed Apr 14, 2026
commit 03d71569dde94cd018b52d5bf46c1b7288b03cd0
6 changes: 3 additions & 3 deletions go/main.go
View file
Open in desktop
Comment thread
shuchu marked this conversation as resolved.
Original file line number Diff line number Diff line change
Expand Up @@ -247,8 +247,8 @@ func StartGrpcServer(fs *feast.FeatureStore, host string, port int, metricsPort
// StartHttpServerWithLogging starts HTTP server with enabled feature logging
// Go does not allow direct assignment to package-level functions as a way to
// mock them for tests
func StartHttpServer(fs *feast.FeatureStore, host string, port int, metricsPort int, writeLoggedFeaturesCallback logging.OfflineStoreWriteCallback, loggingOpts *logging.LoggingOptions, https_enable bool, certFile string, keyFile string) error {
if https_enable && (certFile == "" || keyFile == "") {
func StartHttpServer(fs *feast.FeatureStore, host string, port int, metricsPort int, writeLoggedFeaturesCallback logging.OfflineStoreWriteCallback, loggingOpts *logging.LoggingOptions, httpsEnable bool, certFile string, keyFile string) error {
if httpsEnable && (certFile == "" || keyFile == "") {
return fmt.Errorf("--tls-cert-file and --tls-key-file must be provided for HTTPS server.")
}

Expand Down Expand Up @@ -308,7 +308,7 @@ func StartHttpServer(fs *feast.FeatureStore, host string, port int, metricsPort
}
}()

if https_enable {
if httpsEnable {
err = ser.ServeTLS(host, port, certFile, keyFile)
} else {
err = ser.Serve(host, port)
Expand Down
Loading