Problem/Motivation

$client->models() in LiteLlmAiConfigForm::validateForm will throw exception if hostname has trailing slash in it.

Steps to reproduce

Add trailing slash to working host name and it will show error message 'The API Key is invalid.'

Proposed resolution

Either update description something like 'The base URL that API requests should be made against without trailing slash.' or add validation.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 issue3532838.jpg53.77 KBmatthews

Issue fork ai_provider_litellm-3532838

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

narendrar created an issue. See original summary.

dan2k3k4 made their first commit to this issue’s fork.

dan2k3k4 opened merge request !9

dan2k3k4’s picture

dan2k3k4 commented
Status: Active » Needs review

Created a Merge Request:
https://git.drupalcode.org/project/ai_provider_litellm/-/merge_requests/9

matthews’s picture

matthews commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new issue3532838.jpg53.77 KB

I tested this locally and the fix appears to work. I now get an error message indicating I should not have a trailing slash. Setting to RTBC.

issue 3532838

  • dan2k3k4 committed a5cb5fdd on 1.2.x 
    [#3532838] feat: Validate/Remove trailing slash in host name

By:...
dan2k3k4’s picture

dan2k3k4 commented
Status: Reviewed & tested by the community » Fixed

Thanks, merged MR and credited everyone using the new Contribution Record system
https://new.drupal.org/contribution-record/11415156

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

dan2k3k4’s picture

dan2k3k4 commented
Status: Fixed » Closed (fixed)