Skip to content

chore: Move user traffic data to trace level #498

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 7 commits into from

Conversation

newtork
Copy link
Contributor

@newtork newtork commented Jul 15, 2025

Context

https://github.com/SAP/ai-sdk-java-backlog/issues/290

Please provide a short description of what your change does and why it is needed.

Feature scope:

  • Investigate all log\, occurrences
    • Fix cases
  • (Optional) Consider automatic static-code quality check

Definition of Done

  • Functionality scope stated & covered
  • Tests cover the scope above
  • Error handling created / updated & covered by the tests above
  • Aligned changes with the JavaScript SDK
  • Documentation updated
  • Release notes updated

@newtork newtork changed the title Move user traffic data from info/error to debug level chore: Move user traffic data from info/error to debug level Jul 15, 2025
@newtork newtork changed the title chore: Move user traffic data from info/error to debug level chore: Move user traffic data to trace level Jul 16, 2025
@newtork newtork added the please-review Request to review a pull-request label Jul 16, 2025
@jjtang1985 jjtang1985 requested a review from MatKuhr July 16, 2025 10:38
@newtork newtork added dont-merge and removed please-review Request to review a pull-request labels Jul 19, 2025
@newtork
Copy link
Contributor Author

newtork commented Jul 19, 2025

We decided to remove the logging of HTTP responses (from our namespace) altogether.
Instead we will offer a reference to the (failing) HTTP request/response to the respective exception objects.

@newtork
Copy link
Contributor Author

newtork commented Aug 6, 2025

Closed in favor of #529

@newtork newtork closed this Aug 6, 2025
@newtork newtork deleted the move-traffic-to-debug branch August 6, 2025 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants