-
Notifications
You must be signed in to change notification settings - Fork 4.3k
Open
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomerssize: small
Description
Description
core/framework/credentials/aden/client.py uses print() statements instead of the logging module, which is inconsistent with the rest of the codebase.
Location
- Line 23:
print(f"Token expires at: {response.expires_at}") - Line 230:
print(f"{info.integration_id}: {info.status}")
Expected Behavior
Should use logger.info() or logger.debug() for consistency with the rest of the framework.
Proposed Fix
# Before
print(f"Token expires at: {response.expires_at}")
# After
logger.debug(f"Token expires at: {response.expires_at}")Why This Matters
- Print statements don't respect log levels
- Can't be filtered/configured like proper logging
- Inconsistent with project conventions
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomerssize: small