Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions langfuse/_client/span_processor.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,8 @@ def __init__(
# Prepare default headers
default_headers = {
"Authorization": basic_auth_header,
"x_langfuse_sdk_name": "python",
"x_langfuse_sdk_version": langfuse_version,
"x-langfuse-sdk-name": "python",
"x-langfuse-sdk-version": langfuse_version,
"x_langfuse_public_key": public_key,
Comment thread
hassiebp marked this conversation as resolved.
Outdated
}

Expand Down
4 changes: 2 additions & 2 deletions langfuse/_utils/request.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,8 @@ def generate_headers(self) -> dict:
f"{self._public_key}:{self._secret_key}".encode("utf-8")
).decode("ascii"),
"Content-Type": "application/json",
"x_langfuse_sdk_name": "python",
"x_langfuse_sdk_version": self._version,
"x-langfuse-sdk-name": "python",
"x-langfuse-sdk-version": self._version,
"x_langfuse_public_key": self._public_key,
Comment thread
hassiebp marked this conversation as resolved.
Outdated
}

Expand Down
4 changes: 2 additions & 2 deletions tests/test_additional_headers_simple.py
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ def test_span_processor_has_additional_headers_in_otel_exporter(self):

# Verify default headers are still present
assert "Authorization" in exporter._headers
assert "x_langfuse_sdk_name" in exporter._headers
assert "x-langfuse-sdk-name" in exporter._headers
assert "x_langfuse_public_key" in exporter._headers
Comment thread
hassiebp marked this conversation as resolved.
Outdated

# Check that our override worked
Expand All @@ -179,5 +179,5 @@ def test_span_processor_none_additional_headers_works(self):

# Verify default headers are present
assert "Authorization" in exporter._headers
assert "x_langfuse_sdk_name" in exporter._headers
assert "x-langfuse-sdk-name" in exporter._headers
assert "x_langfuse_public_key" in exporter._headers
Comment thread
hassiebp marked this conversation as resolved.
Outdated