Skip to content

fix: Fixing auth headers#101

Merged
tazarov merged 7 commits into
mainfrom
trayan-02-28-fix_fixing_auth_headers
May 23, 2025
Merged

fix: Fixing auth headers#101
tazarov merged 7 commits into
mainfrom
trayan-02-28-fix_fixing_auth_headers

Conversation

@tazarov

@tazarov tazarov commented Feb 28, 2025

Copy link
Copy Markdown
Contributor

Closes #99

tazarov commented Feb 28, 2025

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@tazarov tazarov mentioned this pull request Feb 28, 2025
{{- else if or (eq $headerType "x_chroma_token") (eq $headerType "x-chroma-token") }}X-Chroma-Token{{- else }}
{{- fail (printf "Invalid ChromaDB auth token header type: %s. Allowed values: Authorization, X-Chroma-Token" .Values.chromadb.auth.token.headerType) }}
{{- end }}
{{- end }} No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The file is missing a newline at the end, which is a common best practice and can prevent issues with certain tools and version control systems. Consider adding a blank line at the end of the _helpers.tpl file.

Suggested change
{{- end }}
{{- end }}

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.

tazarov commented May 23, 2025

Copy link
Copy Markdown
Contributor Author

Merge activity

  • May 23, 8:29 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 23, 8:29 AM UTC: @tazarov merged this pull request with Graphite.

@tazarov tazarov merged commit 16e827b into main May 23, 2025
10 of 14 checks passed
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.

[Bug]: can't authenticate

1 participant