Skip to content

Conversation

sd2k
Copy link
Collaborator

@sd2k sd2k commented Apr 1, 2025

Closes #70.

@sd2k sd2k requested a review from a team as a code owner April 1, 2025 20:29
@sd2k
Copy link
Collaborator Author

sd2k commented Apr 1, 2025

Good call on the redacted URLs, updated.

Copy link
Contributor

@csmarchbanks csmarchbanks left a comment

Choose a reason for hiding this comment

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

A refactor on how we parse URLs might be nice in the future, but LGTM for now.

@sd2k sd2k merged commit a3bf6a6 into main Apr 2, 2025
5 checks passed
@sd2k sd2k deleted the add-logging branch April 2, 2025 07:05
ioanarm pushed a commit that referenced this pull request Apr 22, 2025
* Add some basic logging of config on startup when using stdio

Closes #70.

* Still log address when using SSE transport

* Log redacted URLs
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.

[RFE] debug logging capability
2 participants