Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Tavily tools demand an api key although there's a baseUrl override. #644

Closed
1 of 12 tasks
Nana-Kwame-bot opened this issue Jan 26, 2025 · 3 comments
Closed
1 of 12 tasks

Tavily tools demand an api key although there's a baseUrl override. #644

Nana-Kwame-bot opened this issue Jan 26, 2025 · 3 comments
Assignees
@Nana-Kwame-bot
Labels
c:tools Tools. p:langchain_community langchain_community package. t:enhancement New feature or request
Milestone
Next release

Comments

@Nana-Kwame-bot
Copy link
Contributor

Nana-Kwame-bot commented Jan 26, 2025 •
edited
Loading

System Info

LangChain.dart: ^0.7.7+1
Dart: 3.6.1
Platform: Android

Related Components

  • doc-loaders
  • doc-transformers
  • prompts
  • llms
  • chat-models
  • output-parsers
  • chains
  • memory
  • stores
  • embeddings
  • retrievers
  • agents

Reproduction

Steps to reproduce the behaviour:

  1. Add the TavilyAnswerTool and provide a fake api key
  2. Add a functional baseUrl/proxy

Expected behaviour

I expect the apiKey parameter to be optional
@Nana-Kwame-bot Nana-Kwame-bot added the t:bug Something isn't working label Jan 26, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in LangChain.dart Jan 26, 2025
@Nana-Kwame-bot Nana-Kwame-bot changed the title Tavily tools demand an api key although there's a baseUrl override and the search fails if not provided. Tavily tools demand an api key although there's a baseUrl override. Jan 26, 2025
@davidmigloz davidmigloz added c:tools Tools. p:langchain_community langchain_community package. t:enhancement New feature or request and removed t:bug Something isn't working labels Jan 27, 2025
@davidmigloz
Copy link
Owner

hey @Nana-Kwame-bot ,

Thanks for the suggestion. Could you send a PR with the change?

@Nana-Kwame-bot
Copy link
Contributor Author

Thanks for the suggestion. Could you send a PR with the change?

Sure, I can get that done.

@Nana-Kwame-bot Nana-Kwame-bot mentioned this issue Feb 2, 2025
Merged
@Nana-Kwame-bot
Copy link
Contributor Author

Looks like I didn't link the issue properly. Closing this myself then.

@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in LangChain.dart Feb 3, 2025
@davidmigloz davidmigloz added this to the Next release milestone Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Nana-Kwame-bot Nana-Kwame-bot

Labels
c:tools Tools. p:langchain_community langchain_community package. t:enhancement New feature or request
Projects
LangChain.dart
Status: ✅ Done
Milestone
Next release
Development

No branches or pull requests
2 participants
@davidmigloz @Nana-Kwame-bot