Skip to content

feat: Warn when the registry server starts with authentication disabled#6512

Open
AgentGymLeader wants to merge 1 commit into
feast-dev:masterfrom
AgentGymLeader:warn-registry-server-no-auth
Open

feat: Warn when the registry server starts with authentication disabled#6512
AgentGymLeader wants to merge 1 commit into
feast-dev:masterfrom
AgentGymLeader:warn-registry-server-no-auth

Conversation

@AgentGymLeader

Copy link
Copy Markdown

Add a startup warning when the gRPC registry server is launched with authentication disabled (auth type no_auth).

Why

The registry server accepts control-plane writes such as applying feature views. Materializing those definitions loads user-provided transformation code on the registry host. When authentication is disabled the server accepts these requests unauthenticated, so it is only safe to run inside a trusted network boundary. Today nothing in the startup path signals this to the operator — the only startup logs are info-level TLS/non-TLS messages.

What

  • Add _warn_if_auth_disabled(auth_manager_type), which emits a single logger.warning when the auth manager type is NONE.
  • Call it from start_server right after the auth type is resolved.
  • Add a unit test covering that the warning fires for NONE and stays silent for an authenticated type.

Behavior is unchanged; this only adds an operator-facing warning. This is a defense-in-depth / operator-visibility improvement, not a behavior change.

The registry server accepts control-plane writes such as applying feature
views, which materialize user-provided transformation code on the registry
host. When authentication is disabled (auth type no_auth) it accepts those
requests unauthenticated, so it is only safe to run inside a trusted network
boundary. Nothing in the startup path signals this today.

Add a startup warning emitted when the auth manager type is NONE, plus a
unit test covering that the warning fires for NONE and stays silent for an
authenticated type. Behavior is otherwise unchanged.

Signed-off-by: FugoP <264910004+AgentGymLeader@users.noreply.github.com>
@AgentGymLeader AgentGymLeader requested a review from a team as a code owner June 13, 2026 10:01
@suzunn

suzunn commented Jun 13, 2026

Copy link
Copy Markdown

I like that this keeps the behavior unchanged and only raises operator visibility. One regression I would add is a thin start_server-level test for the no_auth config path, with the security/server setup calls monkeypatched out. The helper tests prove the warning logic, but a startup-path test would also catch a future refactor that resolves auth_manager_type correctly and then accidentally stops calling _warn_if_auth_disabled before the server is exposed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants