Skip to content

Conversation

@g-linville
Copy link
Member

@g-linville g-linville commented Nov 21, 2025

for #5045

@g-linville g-linville force-pushed the registry-spec-rebased branch 2 times, most recently from 70f1ba1 to 13a87ec Compare November 24, 2025 19:57
@g-linville g-linville marked this pull request as draft November 24, 2025 20:23
@g-linville g-linville force-pushed the registry-spec-rebased branch 3 times, most recently from ecf7ff4 to e9f3661 Compare November 25, 2025 20:55
Signed-off-by: Grant Linville <grant@acorn.io>
Signed-off-by: Grant Linville <grant@acorn.io>
Signed-off-by: Grant Linville <grant@acorn.io>
Signed-off-by: Grant Linville <grant@acorn.io>
Signed-off-by: Grant Linville <grant@acorn.io>
Signed-off-by: Grant Linville <grant@acorn.io>
@g-linville g-linville force-pushed the registry-spec-rebased branch from 73d2f3c to 87caf7c Compare November 26, 2025 14:46
@g-linville g-linville marked this pull request as ready for review November 26, 2025 16:30
Signed-off-by: Grant Linville <grant@acorn.io>
Signed-off-by: Grant Linville <grant@acorn.io>
# config.OBOT_SERVER_ENABLE_AUTHENTICATION -- Enables authentication for Obot
OBOT_SERVER_ENABLE_AUTHENTICATION: false
# config.OBOT_SERVER_ENABLE_REGISTRY_AUTH -- Enables authentication for the MCP registry API. When false (default), registry is accessible without authentication and returns only default catalog items with wildcard access control rules.
OBOT_SERVER_ENABLE_REGISTRY_AUTH: false
Copy link
Contributor

Choose a reason for hiding this comment

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

im curious about this choice to turn the auth off by default, even if OBOT_SERVER_ENABLE_AUTHENTICATION is true - won't most people expect OBOT_SERVER_ENABLE_REGISTRY_AUTH to be true by default if they enabled auth?

Copy link
Member Author

Choose a reason for hiding this comment

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

This was by request from @ibuildthecloud

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