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
Next Next commit
fix: Fix unit test error. 
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 d87d8957aa7cadd0ea867b79735403ad2263bee0
2 changes: 1 addition & 1 deletion go/main_test.go
View file
Open in desktop

@devin-ai-integration devin-ai-integration Bot Apr 14, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 MockServerStarter missing StartHttpsServer method required by ServerStarter interface

The ServerStarter interface at go/main.go:51 now includes StartHttpsServer, but MockServerStarter in go/main_test.go:29 does not implement this method. The comment on line 28 explicitly states it is a mock of ServerStarter interface for testing, yet it no longer satisfies that interface. Any test that attempts to use MockServerStarter as a ServerStarter will fail to compile.

(Refers to lines 29-41)

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ func TestStartHttpsServerHealthEndpoint(t *testing.T) {
func TestStartHttpsServerTLSFilesRequired(t *testing.T) {
err := StartHttpsServer(&feast.FeatureStore{}, "127.0.0.1", 0, 0, "", "", nil, nil)
require.Error(t, err)
assert.Contains(t, err.Error(), "TLS_CERT_FILE and TLS_KEY_FILE must be set")
assert.Contains(t, err.Error(), "--tls-cert-file and --tls-key-file must be provided")
}

func getFreePort(t *testing.T) int {
Expand Down