⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@leandrowilliam
Copy link
Contributor

@leandrowilliam leandrowilliam commented Jan 7, 2026

Closes

Closes #1464

Overview

Allows component console output to be configured independently of the registry's storage location (local filesystem vs remote).

Solution

Add a new configuration option to control component logging, allowing:

  • Console output in development and debugging scenarios
  • Custom logging implementations for monitoring integration
  • Disabled by default in production for security and performance

cc: @kmcrawford @GinGatt

@leandrowilliam leandrowilliam force-pushed the issues/1464-logger-param branch from e965fe4 to 13ed32f Compare January 7, 2026 21:09
@ricardo-devis-agullo
Copy link
Collaborator

ricardo-devis-agullo commented Jan 14, 2026

Hi, the distinction here is not local filesystem or remote storage, but local development or production. I don't think it's a great idea to let all components log on the registry without control. Even if we were I don't think enableComponentConsoleOutput is a great name, and even in that case, if we are going to give a pass to let components log, it might be better to let the the registry provide a console interface ({ componentConsole: console }), so perhaps you could pass a console interface that sends logs to a monitoring provider instead

@leandrowilliam
Copy link
Contributor Author

Hi, the distinction here is not local filesystem or remote storage, but local development or production. I don't think it's a great idea to let all components log on the registry without control. Even if we were I don't think enableComponentConsoleOutput is a great name, and even in that case, if we are going to give a pass to let components log, it might be better to let the the registry provide a console interface ({ componentConsole: console }), so perhaps you could pass a console interface that sends logs to a monitoring provider instead

Got it, thanks for the clarification. I agree with that direction. I’ll update the approach and push the changes shortly

@leandrowilliam leandrowilliam marked this pull request as draft January 14, 2026 13:55
@leandrowilliam leandrowilliam marked this pull request as ready for review January 14, 2026 14:50
@leandrowilliam
Copy link
Contributor Author

leandrowilliam commented Jan 14, 2026

@ricardo-devis-agullo Let me know if that works for you or if you think we should adjust anything else. Thanks

Update: I also left a note in the issue

@leandrowilliam leandrowilliam changed the title Add enableComponentConsoleOutput config to decouple logging from storage location Add componentConsole config to decouple logging from storage location Jan 14, 2026
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.

Component console output is not visible when registry uses remote storage

2 participants