-
Notifications
You must be signed in to change notification settings - Fork 67
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
Get userId from body of request to OpenAI rather then header #8
Comments
Good suggestion! Didn't even know OpenAI provided this field. I'll update the code to account for that too :) |
Awesome! |
Exactly had the same question. I already pass the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the
userId
for a specific request is set through theX-User-Id
header, like this:This means that, if different requests need to be associated to different users, the request header needs to be dynamic and change on each request (and therefore we need to create a new
Configuration
object on each request).However, OpenAI API already accepts a
user
field in the body of the request, which has a similar purpose.It would be handy to optionally get the userId for LLM Report from this field of the request body, rather than the
X-User-Id
header. This would avoid having to set new headers for each request.The text was updated successfully, but these errors were encountered: