Conversation
Generate changelog in
|
| hostname = HOSTNAME_VAR.get() | ||
| if hostname is None: | ||
| raise RuntimeError("Foundry API gateway base URL is not available in the current context.") | ||
| if hostname.startswith("https://"): |
There was a problem hiding this comment.
do all callers of this method handle the non-prefixed hostname as well?
There was a problem hiding this comment.
the only callers today are in this file, and they already are assuming no scheme is present and prepend https://
|
|
|
Thanks for checking me, I actually put them in the wrong place. These are a special case where we add them to the |
Before this PR
The LLM utils:
https://from the beginning of the context var which is present in some environments, leading to connection failures when trying to call a URL starting withhttps://https://After this PR
==COMMIT_MSG==
Updates the LLM utility methods
==COMMIT_MSG==
Possible downsides?
None