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.

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

Fix header in local connection External API Key docs example #2063

Merged
merged 1 commit into from
May 20, 2024

Conversation

TJaniF
Copy link
Contributor

@TJaniF TJaniF commented May 10, 2024

What's being changed:

Just a one line change in the External API keys example, changing the headers key from X-OpenAI-Api to X-OpenAI-Api-key.

With the current header key I get the following error:

WeaviateQueryError: Query call with protocol GRPC search failed with message explorer: get class: vectorize params:
 vectorize params: vectorize params: vectorize keywords: remote client vectorize: API Key: no api key found neither in
  request header: X-Openai-Api-Key nor in environment variable under OPENAI_APIKEY.

My connection works with the changed header :)

Type of change:

  • Documentation updates (non-breaking change to fix/update documentation)

How Has This Been Tested?

  • not tested

@daveatweaviate
Copy link
Contributor

Thanks for your suggestion @TJaniF !

Have you accepted the contributor agreement? Please let us know that you've accepted the agreement, and then I can merge in your update.

Thanks again

@TJaniF
Copy link
Contributor Author

TJaniF commented May 20, 2024

Thanks for your suggestion @TJaniF !

Have you accepted the contributor agreement? Please let us know that you've accepted the agreement, and then I can merge in your update.

Thanks again

Yes, I accept the agreement :)

@daveatweaviate daveatweaviate merged commit ba6b183 into weaviate:main May 20, 2024
@daveatweaviate
Copy link
Contributor

Thanks @TJaniF!

We really appreciate your contribution, and wish you success with your project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants