mirror of
https://github.com/simstudioai/sim.git
synced 2026-09-24 15:45:35 +08:00
fix(sanitization): secret exposure in function and agent trace spans (#6000)
* fix trace span secret sanitization * sanitize workflow output logs * preserve streaming usage estimates * Fix logging session test after staging merge * secrets sanitization correctness * fix(execution): address review regressions * fix(execution): harden secret trace provenance * fix(execution): preserve functional state during trace projection --------- Co-authored-by: Bill Leoutsakos <billleoutsakos@Mac.localdomain> Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
This commit is contained in:
co-authored by
Bill Leoutsakos
Vikhyath Mondreti
parent
3740c62112
commit
18214158b9
@@ -24,6 +24,14 @@ describe('environment-utils mock', () => {
|
||||
conflicts: [],
|
||||
decryptionFailures: [],
|
||||
})
|
||||
await expect(environmentUtilsMock.getEffectiveEnvironmentSnapshot('user-1')).resolves.toEqual({
|
||||
personalEncrypted: {},
|
||||
workspaceEncrypted: {},
|
||||
personalDecrypted: {},
|
||||
workspaceDecrypted: {},
|
||||
conflicts: [],
|
||||
decryptionFailures: [],
|
||||
})
|
||||
await expect(environmentUtilsMock.upsertPersonalEnvVars('user-1', {})).resolves.toEqual({
|
||||
added: [],
|
||||
updated: [],
|
||||
|
||||
@@ -36,6 +36,9 @@ export const environmentUtilsMockFns = {
|
||||
mockGetPersonalAndWorkspaceEnv: vi
|
||||
.fn()
|
||||
.mockImplementation(async () => emptyPersonalAndWorkspaceEnv()),
|
||||
mockGetEffectiveEnvironmentSnapshot: vi
|
||||
.fn()
|
||||
.mockImplementation(async () => emptyPersonalAndWorkspaceEnv()),
|
||||
mockUpsertPersonalEnvVars: vi.fn().mockResolvedValue({ added: [], updated: [] }),
|
||||
mockUpsertWorkspaceEnvVars: vi.fn().mockResolvedValue([]),
|
||||
mockGetEffectiveDecryptedEnv: vi.fn().mockResolvedValue({}),
|
||||
@@ -52,6 +55,9 @@ export function resetEnvironmentUtilsMock(): void {
|
||||
environmentUtilsMockFns.mockGetPersonalAndWorkspaceEnv
|
||||
.mockReset()
|
||||
.mockImplementation(async () => emptyPersonalAndWorkspaceEnv())
|
||||
environmentUtilsMockFns.mockGetEffectiveEnvironmentSnapshot
|
||||
.mockReset()
|
||||
.mockImplementation(async () => emptyPersonalAndWorkspaceEnv())
|
||||
environmentUtilsMockFns.mockUpsertPersonalEnvVars
|
||||
.mockReset()
|
||||
.mockResolvedValue({ added: [], updated: [] })
|
||||
@@ -73,6 +79,7 @@ export const environmentUtilsMock = {
|
||||
environmentUtilsMockFns.mockInvalidateEffectiveDecryptedEnvCache,
|
||||
getEnvironmentVariableKeys: environmentUtilsMockFns.mockGetEnvironmentVariableKeys,
|
||||
getPersonalAndWorkspaceEnv: environmentUtilsMockFns.mockGetPersonalAndWorkspaceEnv,
|
||||
getEffectiveEnvironmentSnapshot: environmentUtilsMockFns.mockGetEffectiveEnvironmentSnapshot,
|
||||
upsertPersonalEnvVars: environmentUtilsMockFns.mockUpsertPersonalEnvVars,
|
||||
upsertWorkspaceEnvVars: environmentUtilsMockFns.mockUpsertWorkspaceEnvVars,
|
||||
getEffectiveDecryptedEnv: environmentUtilsMockFns.mockGetEffectiveDecryptedEnv,
|
||||
|
||||
Reference in New Issue
Block a user