Skip to content

Conversation

@HarrisonC118
Copy link
Contributor

Removed the '/v1' suffix from the spring.ai.openai.base-url examples in the OpenAI Chat documentation.

The OpenAiApi already includes /v1 in the default completionsPath (/v1/chat/completions). Including it in the base-url causes a double /v1/v1 path in the final request URL. This change aligns the documentation with the actual client behavior.

Thank you for taking time to contribute this pull request!
You might have already read the contributor guide, but as a reminder, please make sure to:

  • Add a Signed-off-by line to each commit (git commit -s) per the DCO
  • Rebase your changes on the latest main branch and squash your commits
  • Add/Update unit tests as needed
  • Run a build and make sure all tests pass prior to submission

For more details, please check the contributor guide.
Thank you upfront!

Removed the '/v1' suffix from the `spring.ai.openai.base-url` examples in the OpenAI Chat documentation.

The `OpenAiApi` already includes `/v1` in the default `completionsPath` (`/v1/chat/completions`). Including it in the base-url causes a double `/v1/v1` path in the final request URL. This change aligns the documentation with the actual client behavior.

Signed-off-by: Harrison <59154555+HarrisonC118@users.noreply.github.com>
@ilayaperumalg ilayaperumalg added documentation Improvements or additions to documentation openai for: backport-to-1.1.x labels Dec 15, 2025
@ilayaperumalg ilayaperumalg self-assigned this Dec 16, 2025
@ilayaperumalg ilayaperumalg added this to the 2.0.0.M2 milestone Dec 16, 2025
@ilayaperumalg
Copy link
Member

@HarrisonC118 Thanks for the PR fixing the doc. Rebased and merged into main d281da4 as well as 1.1.x via f1a36dc8a7f1a7ea4360d740563f88d95ba7405b

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

Labels

documentation Improvements or additions to documentation for: backport-to-1.1.x openai

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants