Skip to content

fix(agentic-ai): Fix socket and connection timeout config on LLM clients being too short on default settings (#7240)#7242

Merged
nikonovd merged 2 commits into
stable/8.8from
backport-7240-to-stable/8.8
May 18, 2026
Merged

fix(agentic-ai): Fix socket and connection timeout config on LLM clients being too short on default settings (#7240)#7242
nikonovd merged 2 commits into
stable/8.8from
backport-7240-to-stable/8.8

Conversation

@nikonovd
Copy link
Copy Markdown
Contributor

@nikonovd nikonovd commented May 15, 2026 •
edited
Loading

Description

Backport of #7240 to stable/8.8

Deviations:

  • Only applying socket and connection timeouts to Amazon Bedrock chat models
  • Other providers in the original PR respect the timeout settings that are already configured, as L4J implementation applies it, when no custom client is being configured.

@nikonovd nikonovd self-assigned this May 15, 2026
Copilot AI review requested due to automatic review settings May 15, 2026 14:17
@nikonovd nikonovd requested a review from a team as a code owner May 15, 2026 14:17
@nikonovd nikonovd requested review from a team as code owners May 15, 2026 14:17
@nikonovd nikonovd requested a review from chillleader May 15, 2026 14:17
Copilot AI reviewed May 15, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@nikonovd nikonovd changed the base branch from main to stable/8.8 May 15, 2026 14:17
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 15, 2026 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@nikonovd nikonovd requested review from Copilot and removed request for a team and chillleader May 15, 2026 14:17
@nikonovd
Copy link
Copy Markdown
Contributor Author

nikonovd commented May 15, 2026

ℹ️ In 8.8, there's no proxy configuration, thus, the timeout setting applies on all providers besides Amazon Bedrock.

Therefor, we are creating the apache http client here with the respective timeout config.

Copilot AI reviewed May 15, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

Comment thread .../java/io/camunda/connector/agenticai/aiagent/framework/langchain4j/ChatModelFactoryImpl.java Outdated
Comment thread .../java/io/camunda/connector/agenticai/aiagent/framework/langchain4j/ChatModelFactoryImpl.java Outdated
Comment thread .../java/io/camunda/connector/agenticai/aiagent/framework/langchain4j/ChatModelFactoryImpl.java Outdated
@nikonovd nikonovd added this pull request to the merge queue May 18, 2026
Merged via the queue into stable/8.8 with commit 8f06792 May 18, 2026
26 checks passed
@nikonovd nikonovd deleted the backport-7240-to-stable/8.8 branch May 18, 2026 09:00
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 20, 2026

🎉 This pull request has been included in release 8.8.12!

Thank you for your contribution! 🚀

@github-actions github-actions Bot added the version:8.8.12 Released in version 8.8.12 label May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@maff maff maff approved these changes

Assignees

@nikonovd nikonovd

Labels

agentic-ai e2e-tests version:8.8.12 Released in version 8.8.12

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nikonovd @CLAassistant @maff