feat: Add TLS support for Go Feature Server#6229
Conversation
Sorry, something went wrong.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
|
@shuchu optional nitpick - I see duplication of code in http vs https in start and serve methods |
Sorry, something went wrong.
yes, you are right. let me put those two functions into one. |
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
franciscojavierarceo
merged commit 28a58d0
into
feast-dev:master
Apr 14, 2026
Reviewers
3 participants
What this PR does / why we need it:
Add TLS support for users who have enterprise security requirement
Which issue(s) this PR fixes:
Fix #6095
Checks
git commit -s)Testing Strategy
Misc