Skip to content
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.

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

fix(ai-proxy): internal server error on path parse failure #12998

Merged
merged 1 commit into from
May 8, 2024

Conversation

tysoekong
Copy link
Contributor

Summary

Fixes "Unexpected error" when using self-hosted Llama2 or Mistral models. Path parsing was incomplete, resulting in Kong trying to gsub a nil table reference.

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

#12869

@tysoekong
Copy link
Contributor Author

tysoekong commented May 8, 2024

Can make 3.7.1 instead if necessary, came from community reported error #12869

@samugi samugi merged commit 6374c55 into master May 8, 2024
42 checks passed
@samugi samugi deleted the fix/ai_proxy_path_parsing branch May 8, 2024 13:38
@team-gateway-bot
Copy link
Collaborator

Successfully created backport PR for release/3.7.x:

@team-gateway-bot
Copy link
Collaborator

Successfully created cherry-pick PR for master:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants