[GH-ISSUE #15572] feat: ExternalReranker must honor ENABLE_FORWARD_USER_INFO_HEADERS parameter #88346

New Issue

GiteaMirror · 2026-05-15T12:54:12-05:00

GiteaMirror commented

2026-05-15 12:54:12 -05:00

Originally created by @athoik on GitHub (Jul 7, 2025).
Original GitHub issue: https://github.com/open-webui/open-webui/issues/15572

Check Existing Issues

I have searched the existing issues and discussions.

Problem Description

External reranker currently doesn't honor the variable ENABLE_FORWARD_USER_INFO_HEADERS

5eca495d3e/backend/open_webui/retrieval/models/external.py (L39-L46)

Desired Solution you'd like

It would be great if we could add the handling of ENABLE_FORWARD_USER_INFO_HEADERS

We need more or less the following:

 r = requests.post( 
     f"{self.url}", 
     headers={ 
         "Content-Type": "application/json", 
         "Authorization": f"Bearer {self.api_key}",
          **(
              {
                  "X-OpenWebUI-User-Name": user.name,
                  "X-OpenWebUI-User-Id": user.id,
                  "X-OpenWebUI-User-Email": user.email,
                  "X-OpenWebUI-User-Role": user.role,
              }
              if ENABLE_FORWARD_USER_INFO_HEADERS and user
              else {}
     }, 
     json=payload, 
 )

Getting the ENABLE_FORWARD_USER_INFO_HEADERS is straight forward.

from open_webui.env import (
    ENABLE_FORWARD_USER_INFO_HEADERS,
)

But getting the user: UserModel = None is not very clear.

We are calling ExternaReranker from here: 5eca495d3e/backend/open_webui/routers/retrieval.py (L819)

and get_rf is called from here: 5eca495d3e/backend/open_webui/routers/retrieval.py (L617-L618)

Is it enough to use the user variable?

Can you please assist to add that missing feature? @tjbck

Alternatives Considered

None, there is no alternative.

Additional Context

Having that feature it's possible now to track spending on LiteLLM: https://docs.litellm.ai/docs/tutorials/openweb_ui#add-custom-headers-to-spend-tracking

Currently requests using external reranker do not contain information about user, because we do not honor the ENABLE_FORWARD_USER_INFO_HEADERS.

Originally created by @athoik on GitHub (Jul 7, 2025). Original GitHub issue: https://github.com/open-webui/open-webui/issues/15572 ### Check Existing Issues - [x] I have searched the existing issues and discussions. ### Problem Description External reranker currently doesn't honor the variable `ENABLE_FORWARD_USER_INFO_HEADERS` https://github.com/open-webui/open-webui/blob/5eca495d3e3b3066e7831141ed2adffbd6d179b4/backend/open_webui/retrieval/models/external.py#L39-L46 ### Desired Solution you'd like It would be great if we could add the handling of `ENABLE_FORWARD_USER_INFO_HEADERS` We need more or less the following: ``` r = requests.post( f"{self.url}", headers={ "Content-Type": "application/json", "Authorization": f"Bearer {self.api_key}", **( { "X-OpenWebUI-User-Name": user.name, "X-OpenWebUI-User-Id": user.id, "X-OpenWebUI-User-Email": user.email, "X-OpenWebUI-User-Role": user.role, } if ENABLE_FORWARD_USER_INFO_HEADERS and user else {} }, json=payload, ) ``` Getting the `ENABLE_FORWARD_USER_INFO_HEADERS` is straight forward. ``` from open_webui.env import ( ENABLE_FORWARD_USER_INFO_HEADERS, ) ``` But getting the `user: UserModel = None` is not very clear. We are calling ExternaReranker from here: https://github.com/open-webui/open-webui/blob/5eca495d3e3b3066e7831141ed2adffbd6d179b4/backend/open_webui/routers/retrieval.py#L819 and get_rf is called from here: https://github.com/open-webui/open-webui/blob/5eca495d3e3b3066e7831141ed2adffbd6d179b4/backend/open_webui/routers/retrieval.py#L617-L618 Is it enough to use the user variable? Can you please assist to add that missing feature? @tjbck ### Alternatives Considered None, there is no alternative. ### Additional Context Having that feature it's possible now to track spending on LiteLLM: https://docs.litellm.ai/docs/tutorials/openweb_ui#add-custom-headers-to-spend-tracking Currently requests using external reranker do not contain information about user, because we do not honor the `ENABLE_FORWARD_USER_INFO_HEADERS`.

GiteaMirror closed this issue

2026-05-15 12:54:13 -05:00

GiteaMirror commented

2026-05-15 12:54:14 -05:00

@tjbck commented on GitHub (Jul 8, 2025):

PR Welcome!

@tjbck commented on GitHub (Jul 8, 2025): PR Welcome!

GiteaMirror commented

2026-05-15 12:54:15 -05:00

@rgaricano commented on GitHub (Jul 8, 2025):

PR: https://github.com/open-webui/open-webui/pull/15586

(sorry for the two wrong closed pulls)

@rgaricano commented on GitHub (Jul 8, 2025): PR: https://github.com/open-webui/open-webui/pull/15586 (sorry for the two wrong closed pulls)

GiteaMirror commented

2026-05-15 12:54:15 -05:00

@athoik commented on GitHub (Jul 8, 2025):

I think now i've got it how we are getting user variable :)

user=Depends(get_verified_user),

I was missing that part in my original qustion. @rgaricano thanks for taking over the PR.

@athoik commented on GitHub (Jul 8, 2025): I think now i've got it how we are getting user variable :) ``` user=Depends(get_verified_user), ``` I was missing that part in my original qustion. @rgaricano thanks for taking over the PR.

GiteaMirror commented

2026-05-15 12:54:16 -05:00

@rgaricano commented on GitHub (Jul 8, 2025):

I just assignate it directly in function def, but as it's now probably is more clear.

But I having problems to test it, when I switch on hybrid mode & save it allways switch off again!

@rgaricano commented on GitHub (Jul 8, 2025): I just assignate it directly in function def, but as it's now probably is more clear. But I having problems to test it, when I switch on hybrid mode & save it allways switch off again!

GiteaMirror commented

2026-05-15 12:54:17 -05:00

@athoik commented on GitHub (Jul 9, 2025):

@rgaricano it was tricky to fix it, but figure out how to do it... thanks for your efford. I added you as co-author on the commit.

A new PR created https://github.com/open-webui/open-webui/pull/15615

@athoik commented on GitHub (Jul 9, 2025): @rgaricano it was tricky to fix it, but figure out how to do it... thanks for your efford. I added you as co-author on the commit. A new PR created https://github.com/open-webui/open-webui/pull/15615

GiteaMirror commented

2026-05-15 12:54:18 -05:00

@rgaricano commented on GitHub (Jul 9, 2025):

Yes, similar solution as I did, but I used Users instead UserModel. By the way I found some other functions that might also need to add user.

@rgaricano commented on GitHub (Jul 9, 2025): Yes, similar solution as I did, but I used Users instead UserModel. By the way I found some other functions that might also need to add user.

GiteaMirror commented

2026-05-15 12:54:19 -05:00

@athoik commented on GitHub (Jul 11, 2025):

PR Welcome!

@tjbck PR created in collaboration with @rgaricano

https://github.com/open-webui/open-webui/pull/15615

@athoik commented on GitHub (Jul 11, 2025): > PR Welcome! @tjbck PR created in collaboration with @rgaricano https://github.com/open-webui/open-webui/pull/15615

GiteaMirror commented

2026-05-15 12:54:19 -05:00

@tjbck commented on GitHub (Jul 14, 2025):

Thanks everyone!

@tjbck commented on GitHub (Jul 14, 2025): Thanks everyone!

GiteaMirror commented

2026-05-15 12:54:20 -05:00

@athoik commented on GitHub (Jul 14, 2025):

You did it much beter @tjbck ! Thank you! You Rock! 🥇

@athoik commented on GitHub (Jul 14, 2025): You did it much beter @tjbck ! Thank you! You Rock! 🥇

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/open-webui#88346