langchain-ai/docs
1
0
Fork 0
mirror of https://github.com/langchain-ai/docs.git synced 2026-07-21 20:05:30 -04:00
Code Issues 490 Packages Projects Releases Wiki Activity

[PR #4288] [MERGED] Add warning: do not rotate the API key salt in self-hosted docs #4275

New Issue
Closed
opened 2026-06-05 19:13:20 -04:00 by yindo · 0 comments
yindo commented 2026-06-05 19:13:20 -04:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/4288
Author: @langsmith-fleet[bot]
Created: 6/3/2026
Status: ✅ Merged
Merged: 6/3/2026
Merged by: @katmayb

Base: main ← Head: docs/warn-do-not-rotate-api-key-salt

📝 Commits (2)

  • be2a8ac Add warning: do not rotate the API key salt in self-hosted docs
  • 3220dce Merge branch 'main' into docs/warn-do-not-rotate-api-key-salt

📊 Changes

2 files changed (+8 additions, -0 deletions)

View changed files

📝 src/langsmith/kubernetes.mdx (+4 -0)
📝 src/langsmith/self-host-using-an-existing-secret.mdx (+4 -0)

📄 Description

Fixes DOC-1201

Summary

  • Added a <Warning> callout to src/langsmith/kubernetes.mdx before the minimum required Helm config block, noting that apiKeySalt must be set once and never rotated, as doing so will invalidate all existing API keys for all users in the organization.
  • Added the same <Warning> callout to src/langsmith/self-host-using-an-existing-secret.mdx before the example secret YAML, noting the same for api_key_salt.

Links

Verification

Not run; docs-only copy change (Mintlify Warning component, no code changes).

Reviewers

Requested review from: @katmayb, @fjmorris

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/docs/pull/4288 **Author:** [@langsmith-fleet[bot]](https://github.com/apps/langsmith-fleet) **Created:** 6/3/2026 **Status:** ✅ Merged **Merged:** 6/3/2026 **Merged by:** [@katmayb](https://github.com/katmayb) **Base:** `main` ← **Head:** `docs/warn-do-not-rotate-api-key-salt` --- ### 📝 Commits (2) - [`be2a8ac`](https://github.com/langchain-ai/docs/commit/be2a8ac85d55df84a24d14a9ee65a40ac5cf3873) Add warning: do not rotate the API key salt in self-hosted docs - [`3220dce`](https://github.com/langchain-ai/docs/commit/3220dce7e3c551520ce2ba5b834e9e69819e6aa1) Merge branch 'main' into docs/warn-do-not-rotate-api-key-salt ### 📊 Changes **2 files changed** (+8 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/langsmith/kubernetes.mdx` (+4 -0) 📝 `src/langsmith/self-host-using-an-existing-secret.mdx` (+4 -0) </details> ### 📄 Description Fixes DOC-1201 ## Summary - Added a `<Warning>` callout to `src/langsmith/kubernetes.mdx` before the minimum required Helm config block, noting that `apiKeySalt` must be set once and never rotated, as doing so will invalidate all existing API keys for all users in the organization. - Added the same `<Warning>` callout to `src/langsmith/self-host-using-an-existing-secret.mdx` before the example secret YAML, noting the same for `api_key_salt`. ## Links - Linear: https://linear.app/langchain/issue/DOC-1201/add-warning-to-self-hosted-docs-do-not-rotate-the-api-key-salt - Slack: https://langchain.slack.com/archives/C09G1T60QV9/p1780509195286539 ## Verification Not run; docs-only copy change (Mintlify Warning component, no code changes). ## Reviewers Requested review from: @katmayb, @fjmorris --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-06-05 19:13:20 -04:00
yindo closed this issue 2026-06-05 19:13:21 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
deepagents
docs-infra
external
external
help wanted
integration
internal
internal
javascript
js/ts
langchain
langgraph
langsmith
needs-title
pull-request
Mirrored from GitHub Pull Request
 python
reference
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/docs#4275
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?