Skip to content

Clean up credential source fallbacks#444

Open
shanselman wants to merge 1 commit into
masterfrom
shanselman/credential-source-cleanup
Open

Clean up credential source fallbacks#444
shanselman wants to merge 1 commit into
masterfrom
shanselman/credential-source-cleanup

Conversation

@shanselman
Copy link
Copy Markdown
Contributor

Summary

  • Remove the legacy settings Token/BootstrapToken fallback from interactive chat/dashboard credential resolution; these flows now resolve from the active GatewayRegistry record plus per-gateway identity only.
  • Keep one-time legacy migration intact and retain the legacy root identity startup backstop with a warning for failed/old migrations.
  • Tighten local gateway setup credential persistence so setup credentials merge through GatewayRegistry, preserve existing record metadata, and explicit clears remove registry tokens instead of preserving stale values.
  • Add regression coverage for active-registry credential resolution and local setup registry-only credential persistence/clear behavior.

Validation

  • ./build.ps1
  • dotnet test ./tests/OpenClaw.Shared.Tests/OpenClaw.Shared.Tests.csproj --no-restore
  • dotnet test ./tests/OpenClaw.Tray.Tests/OpenClaw.Tray.Tests.csproj --no-restore
  • dotnet test ./tests/OpenClaw.Connection.Tests/OpenClaw.Connection.Tests.csproj --no-restore

Remove legacy settings credential fallback from interactive gateway credential resolution so chat and dashboard flows require the active GatewayRegistry record and per-gateway identity data. Tighten local setup persistence so setup credentials merge through GatewayRegistry without writing legacy settings fields, and explicitly-cleared setup credentials clear registry tokens instead of preserving stale values.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant