Skip to content

Conversation

@thesohamdatta
Copy link

Summary

Removes leftover debug prefixes from production error logging in api/routers/sources.py.

Problem

Lines 116 and 124 contain "DEBUG - " prefixes in error messages, which are development artifacts that pollute production logs.
Before:

logger.error(f"DEBUG - Invalid JSON in notebooks field: {notebooks}")
logger.error(f"DEBUG - Invalid JSON in transformations field: {transformations}")
After:

python
logger.error(f"Invalid JSON in notebooks field: {notebooks}")
logger.error(f"Invalid JSON in transformations field: {transformations}")
Changes
Removed "DEBUG - " prefix from notebooks JSON parsing error (line 116)
Removed "DEBUG - " prefix from transformations JSON parsing error (line 124)
Testing
No functional changes. Error handling behavior remains identical - only log message formatting is affected.

Type
 Bug fix (non-breaking change)
 New feature
 Breaking change

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@lfnovo lfnovo merged commit e69485b into lfnovo:main Jan 5, 2026
5 of 6 checks passed
@lfnovo
Copy link
Owner

lfnovo commented Jan 5, 2026

Thank you!

@thesohamdatta
Copy link
Author

Guess what you made just my Pull request

@lfnovo
Copy link
Owner

lfnovo commented Jan 9, 2026

Guess what you made just my Pull request

Sorry.. didn't understand what you meant.. can you explain?

kt-mattie-langenberg pushed a commit to kt-mattie-langenberg/open-notebook that referenced this pull request Jan 10, 2026
…ging-prefixes

Remove debug prefixes from production error logging
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