Skip to content

fix: urlIdx type conversion in pipeline filters #13907

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

fix: urlIdx type conversion in pipeline filters #13907

wants to merge 2 commits into from

Conversation

anthonysetiadi
Copy link

Fix urlIdx type conversion in pipeline filters

This PR fixes the type error issue reported in open-webui/pipelines#511 by improving the urlIdx type handling in both inlet and outlet filters.

Changes:

  • Simplified urlIdx type conversion in process_pipeline_inlet_filter and process_pipeline_outlet_filter

Testing:

  • Built and tested locally with Docker (OpenWebUI v0.6.9)
  • Verified pipeline filters work correctly with integer urlIdx values
  • Confirmed the fix resolves the TypeError reported in the issue

Fixes open-webui/pipelines#511

@tjbck
Copy link
Contributor

tjbck commented May 15, 2025

Closing in favour of 72b2555

@tjbck tjbck closed this May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

TypeError: list indices must be integers or slices, not str when using Pipelines in v0.6.5
2 participants
@anthonysetiadi @tjbck