Skip to content

[fix] fix model config logging to remove api key#1619

Merged
ahau-square merged 2 commits intomainfrom
ahau/fix-api-tokens-in-logs
Mar 11, 2025
Merged

[fix] fix model config logging to remove api key#1619
ahau-square merged 2 commits intomainfrom
ahau/fix-api-tokens-in-logs

Conversation

@ahau-square
Copy link
Copy Markdown
Contributor

@ahau-square ahau-square commented Mar 11, 2025

Incorrect trace payload was being written to logs which included api keys on the provider.

@ahau-square ahau-square requested a review from kalvinnchau March 11, 2025 17:52
@ahau-square ahau-square merged commit 746c915 into main Mar 11, 2025
6 checks passed
@ahau-square ahau-square deleted the ahau/fix-api-tokens-in-logs branch March 11, 2025 17:59
michaelneale added a commit that referenced this pull request Mar 11, 2025
* origin:
  docs: Persistent Command History (#1627)
  change to make build work on windows, macos, linux (#1618)
  chore(release): release version 1.0.13 (#1623)
  fix: handle mac screenshots with the image tool (#1622)
  feat: write eval results to eval dir (#1620)
  [fix] fix model config logging to remove api key (#1619)
  fix: ensure repeating benches return to initial run-dir (#1617)
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
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.

2 participants