[GH-ISSUE #366] Route resource issues #123

Closed
opened 2026-10-18 21:06:20 +00:00 by bsakaguchi · 1 comment
Owner

Originally created by @wiiaam on GitHub (Mar 12, 2026).
Original GitHub issue: https://github.com/open-webui/helm-charts/issues/366

There are a couple of issues with the current Route template, specifically the implementation of https redirects.

github.com/open-webui/helm-charts@71222d2a9c/charts/open-webui/templates/route.yaml (L54-L64)

  1. If sectionName is not specified in parentRefs, the template will force sectionName: http to be applied. parentRefs should instead be left untouched unless httpsRedirect is enabled, as it's common to not specify a sectionName and allow the route to bind to all listeners.

  2. With httpsRedirect: true, the sectionName in the main https route is not configurable, and is hardcoded to https. Configuration of sectionName in what you would assume to be the main route configuration is actually reflected in the new redirect route.

I can see the benefit in having an httpsRedirect option, but right now the current implementation is restrictive. A configuration option should be available to change either sectionName or parentRefs in the redirect route. parentRefs in the main route should be left as is, unless sectionName is omitted, where a default of https can be used.

The better approach for this, which a lot of charts are doing, is to allow individual configuration of separate routes in the route: section, similar to this:

route:
  main:
    # main route, including parentRefs pointing to https listener
  redirect:
    # redirect route, including parentRefs pointing to http listener
    httpsRedirect: true

Unfortunately this will likely be a breaking change but I would consider making it now before GatewayAPI starts to overtake Ingress.

Originally created by @wiiaam on GitHub (Mar 12, 2026). Original GitHub issue: https://github.com/open-webui/helm-charts/issues/366 There are a couple of issues with the current Route template, specifically the implementation of https redirects. https://github.com/open-webui/helm-charts/blob/71222d2a9c69975c625d212732f5fa009d5c2ca7/charts/open-webui/templates/route.yaml#L54-L64 1. If `sectionName` is not specified in `parentRefs`, the template will force `sectionName: http` to be applied. parentRefs should instead be left untouched unless httpsRedirect is enabled, as it's common to not specify a sectionName and allow the route to bind to all listeners. 2. With `httpsRedirect: true`, the `sectionName` in the main https route is not configurable, and is hardcoded to `https`. Configuration of `sectionName` in what you would assume to be the main route configuration is actually reflected in the new redirect route. I can see the benefit in having an httpsRedirect option, but right now the current implementation is restrictive. A configuration option should be available to change either `sectionName` or `parentRefs` in the redirect route. `parentRefs` in the main route should be left as is, unless `sectionName` is omitted, where a default of https can be used. The better approach for this, which a lot of charts are doing, is to allow individual configuration of separate routes in the `route:` section, similar to this: ``` route: main: # main route, including parentRefs pointing to https listener redirect: # redirect route, including parentRefs pointing to http listener httpsRedirect: true ``` Unfortunately this will likely be a breaking change but I would consider making it now before GatewayAPI starts to overtake Ingress.
Author
Owner

@james-westbrook commented on GitHub (Apr 8, 2026):

Thanks for the report @wiiaam, I've opened #377 which looks to have addressed all the concerns you raised here in my local testing. I'm going to merge it and close this out. Please let me know if you see any further issues and I'm happy to circle back!

<!-- gh-comment-id:4209832469 --> @james-westbrook commented on GitHub (Apr 8, 2026): Thanks for the report @wiiaam, I've opened #377 which looks to have addressed all the concerns you raised here in my local testing. I'm going to merge it and close this out. Please let me know if you see any further issues and I'm happy to circle back!
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
open-webui/helm-charts#123
No description provided.