Skip to content

docs(ollama): update ollama.mdx with note about basic auth support #32327

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

btelles
Copy link

@btelles btelles commented Jul 30, 2025

This is related to #25055

Although langchain now allows users to specify headers, the client itself does not gracefully handle a standard URL of the form https://user:[email protected]:11343

Copy link

vercel bot commented Jul 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 30, 2025 4:02pm

Copy link

codspeed-hq bot commented Jul 30, 2025

CodSpeed WallTime Performance Report

Merging #32327 will not alter performance

Comparing btelles:patch-1 (3027752) with master (a9e52ca)

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 13 untouched benchmarks

Copy link

codspeed-hq bot commented Jul 30, 2025

CodSpeed Instrumentation Performance Report

Merging #32327 will not alter performance

Comparing btelles:patch-1 (3027752) with master (a9e52ca)

Summary

✅ 14 untouched benchmarks

@mdrxy mdrxy changed the title docs docs: update ollama.mdx to account for lack of basic auth support. docs(ollama): update ollama.mdx to account for lack of basic auth support. Jul 30, 2025
@mdrxy mdrxy changed the title docs(ollama): update ollama.mdx to account for lack of basic auth support. docs(ollama): update ollama.mdx with auth support Jul 30, 2025
@mdrxy mdrxy changed the title docs(ollama): update ollama.mdx with auth support docs(ollama): update ollama.mdx with note about basic auth support Jul 30, 2025
@mdrxy
Copy link
Collaborator

mdrxy commented Jul 30, 2025

@btelles have a look at #32328 and let me know if that looks suitable

@mdrxy mdrxy added the integration Related to a provider partner package integration label Jul 30, 2025
@btelles
Copy link
Author

btelles commented Aug 1, 2025

@btelles have a look at #32328 and let me know if that looks suitable

Yes! Amazing! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration Related to a provider partner package integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants