Skip to content

Conversation

OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues Fix #127
License MIT

Allows configuring custom HTTP client services for all AI platforms (except VertexAI which requires special authentication).

This enables customization of timeouts, proxy settings, SSL configuration, and other HTTP-specific options per platform.

@OskarStark OskarStark self-assigned this Sep 16, 2025
@carsonbot carsonbot added AI Bundle Issues & PRs about the AI integration bundle Feature New feature Status: Needs Review labels Sep 16, 2025
Allows configuring custom HTTP client services for all AI platforms
(except VertexAI which requires special authentication).

This enables customization of timeouts, proxy settings, SSL configuration,
and other HTTP-specific options per platform.

- Add http_client configuration option to all platforms
- Update platform factory logic to use configured HTTP clients
- Add comprehensive documentation with examples
- Improve code formatting for better readability
chr-hertel added a commit that referenced this pull request Sep 16, 2025
…form (OskarStark)

This PR was merged into the main branch.

Discussion
----------

[AI Bundle][Ollama] Fix duplicate `setFactory` call for platform

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        | --
| License       | MIT

This looks wrong to me.

Follows #604 (comment)

Commits
-------

2c19d13 [AI Bundle] Fix duplicate setFactory call for Ollama platform
@chr-hertel
Copy link
Member

this:
image

leads to:
image

issue solved! 💪

Thanks @OskarStark 🙏

@chr-hertel
Copy link
Member

Thank you @OskarStark.

@chr-hertel chr-hertel merged commit c8d0b75 into symfony:main Sep 16, 2025
21 of 24 checks passed
@OskarStark
Copy link
Contributor Author

Great, let's add an example to the demo?

devmatt000 pushed a commit to devmatt000/dev_ai that referenced this pull request Oct 2, 2025
…form (OskarStark)

This PR was merged into the main branch.

Discussion
----------

[AI Bundle][Ollama] Fix duplicate `setFactory` call for platform

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        | --
| License       | MIT

This looks wrong to me.

Follows symfony/ai#604 (comment)

Commits
-------

2c19d135 [AI Bundle] Fix duplicate setFactory call for Ollama platform
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Bundle Issues & PRs about the AI integration bundle Feature New feature Status: Reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Manage HttpClient timeout
3 participants