Skip to content

Conversation

@mcikatricis
Copy link
Contributor

Removed log level to prevent log spamming. The flag is propagated to other libs as well causing many more libs to spam logs.

Removed log level to prevent log spamming. The flag is propagated to other libs as well causing many more libs to spam logs.
@mcikatricis mcikatricis changed the title Update signer_client.py Removed log level DEBUG Sep 27, 2025
@alexvelea
Copy link
Collaborator

Thanks for bringing this up @mcikatricis
According to ChatGPT

Note: basicConfig() does nothing if logging is already configured. In Python 3.8+ you can use basicConfig(force=True) to reconfigure.

Using logging.basicConfig(level=logging.DEBUG, force=True) does not print any Debug logs.
This should solve your issues.

@alexvelea alexvelea closed this Oct 13, 2025
@mcikatricis
Copy link
Contributor Author

Screenshot 2025-10-13 at 12 18 17

@alexvelea alexvelea reopened this Oct 13, 2025
@alexvelea
Copy link
Collaborator

I forgot this has side-effects outside just the library -- thanks for the pointing this out.
Somehow this part didn't register > The flag is propagated to other libs as well causing many more libs to spam logs.

@alexvelea alexvelea merged commit a722b03 into elliottech:main Oct 13, 2025
4 checks passed
@alexvelea
Copy link
Collaborator

Merged. Ty again @mcikatricis for the fix and most of all for the persistance!

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