Skip to content

Logging System Fixes and Improvements#33

Merged
darkdks merged 3 commits intomainfrom
fix-and-refact-logs
Sep 17, 2025
Merged

Logging System Fixes and Improvements#33
darkdks merged 3 commits intomainfrom
fix-and-refact-logs

Conversation

@darkdks
Copy link
Contributor

@darkdks darkdks commented Sep 17, 2025

Logging System Fixes and Improvements

  • Fix and unify logging system: Standardized logging implementation across the API client
  • Configurable log levels: Added support for different logging levels for response and request (ALL, ON ERROR)
  • Enhanced request/response logging: More detailed logging with structured data including method, URL, status codes, request/response bodies, and error information
  • Remove unnecessary code: Eliminated redundant implementations and unused functions, clean up to avoid unnecessary complexity. #18
  • Reduce complexity: Simplified overly complex logic and removed duplicate functionality

@CLAassistant
Copy link

CLAassistant commented Sep 17, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ darkdks
❌ samuel.coelho


samuel.coelho seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@darkdks darkdks force-pushed the fix-and-refact-logs branch 2 times, most recently from f8e35e9 to 7562ef3 Compare September 17, 2025 19:40
@darkdks darkdks merged commit 59da6f1 into main Sep 17, 2025
6 of 7 checks passed
@darkdks darkdks deleted the fix-and-refact-logs branch September 17, 2025 19:45
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