(fix) Pass subject with correct value to OpenMeter in OpenMeter integration

[togmund]

Title

OpenMeter rejects requests from this logger here:

litellm/litellm/integrations/openmeter.py

Lines 128 to 135 in 1890fde

	try:
	await self.async_http_handler.post(
	url=_url,
	data=json.dumps(_data),
	headers=_headers,
	)
	except httpx.HTTPStatusError as e:
	raise Exception(f"OpenMeter logging error: {e.response.text}")

Because the value of subject is not a string. See schema here.

This is because instead of assigning the subject property with the value of the user param in kwargs, it is instead wrapping that value in a tuple:

litellm/litellm/integrations/openmeter.py

Line 73 in 1890fde

subject = (kwargs.get("user", None),) # end-user passed in via 'user' param

Relevant issues

N/A

Type

🐛 Bug Fix

Changes

• Assign subject with the value of the user param, rather than as a tuple containing the value of the user param.

[REQUIRED] Testing - Attach a screenshot of any new tests passing locally

If UI changes, send a screenshot/GIF of working UI fixes

• Set up OpenMeter according to the docs
• Observe the OpenMeter rejection responses without the changes
• Observe the OpenMeter success responses with the changes

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
litellm	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 19, 2024 9:38pm