Skip to content
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

🐛 Bug Report: TRACELOOP_TRACE_CONTENT #2777

Open
1 task done
FCramer opened this issue Mar 13, 2025 · 1 comment
Open
1 task done

🐛 Bug Report: TRACELOOP_TRACE_CONTENT #2777

FCramer opened this issue Mar 13, 2025 · 1 comment

Comments

@FCramer
Copy link

FCramer commented Mar 13, 2025

Which component is this bug for?

AlephAlpha Instrumentation

📜 Description

In my understanding if the TRACELOOP_TRACE_CONTENT is set to false, the TRACELOOP_ENTITY_INPUT and TRACELOOP_ENTITY_OUTPUT attributes should not be set. Do I understand that correctly?
However it seems to me that TRACELOOP_TRACE_CONTENT is ignored since the should_send_prompts() function is never called in _handle_call() and _handle_stream_call() ?

👟 Reproduction steps

  1. Set TRACELOOP_TRACE_CONTENT to false

👍 Expected behavior

It should not add the input and output (body of the http requests)

👎 Actual Behavior with Screenshots

It actually sets the input and output attributes

🤖 Python Version

3.12

📃 Provide any additional context for the Bug.

No response

👀 Have you spent some time to check if this bug has been raised before?

  • I checked and didn't find similar issue

Are you willing to submit PR?

None

@nirga
Copy link
Member

nirga commented Mar 13, 2025

Hey @FCramer, it is checked in _handle_span_input and _handle_span_output

Do you see outputs and inputs being set? Can you provide an example?

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

No branches or pull requests

2 participants