feat: model name sanitisation #172

Closed
opened 2025-11-11 14:09:33 -06:00 by GiteaMirror · 1 comment
Owner

Originally created by @robertvazan on GitHub (Jan 12, 2024).

Bug Report

Description

Bug Summary:
Model download does not strip whitespace from model name. This results in confusing error messages. Whitespace gets there easily when copy-n-pasting from ollama library.

Steps to Reproduce:
Just enter " tinyllama" or "tinyllama " (with the space) in the form and click download.

Expected Behavior:
Whitespace is stripped and download starts.

Actual Behavior:
Error "pull model manifest: file does not exist"

Reproduction Details

Confirmation:

  • I have read and followed all the instructions provided in the README.md.
  • I have reviewed the troubleshooting.md document.
  • I have included the browser console logs.
  • I have included the Docker container logs.

Additional Information

It would be great to also accept the full command from ollama library, e.g. "ollama run tinyllama", or an URL, which are both easy to copy-n-paste.

Originally created by @robertvazan on GitHub (Jan 12, 2024). # Bug Report ## Description **Bug Summary:** Model download does not strip whitespace from model name. This results in confusing error messages. Whitespace gets there easily when copy-n-pasting from ollama library. **Steps to Reproduce:** Just enter " tinyllama" or "tinyllama " (with the space) in the form and click download. **Expected Behavior:** Whitespace is stripped and download starts. **Actual Behavior:** Error "pull model manifest: file does not exist" ## Reproduction Details **Confirmation:** - [x] I have read and followed all the instructions provided in the README.md. - [x] I have reviewed the troubleshooting.md document. - [ ] I have included the browser console logs. - [ ] I have included the Docker container logs. ## Additional Information It would be great to also accept the full command from ollama library, e.g. "ollama run tinyllama", or an URL, which are both easy to copy-n-paste.
Author
Owner

@anuraagdjain commented on GitHub (Jan 13, 2024):

Good catch, missed this case when refactored the logic for multiple downloads (PR).

@anuraagdjain commented on GitHub (Jan 13, 2024): Good catch, missed this case when refactored the logic for multiple downloads ([PR](https://github.com/ollama-webui/ollama-webui/pull/465)).
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/open-webui#172