Skip to content

Commit

Permalink
fix: don't scrub included state output sensitive values
Browse files Browse the repository at this point in the history
  • Loading branch information
leg100 committed Aug 7, 2023
1 parent 6f90a9c commit 478e314
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion internal/api/state_marshaler.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,8 @@ func (m *jsonapiMarshaler) toState(from *state.Version, r *http.Request) (*types
case "outputs":
var include []any
for _, out := range from.Outputs {
include = append(include, m.toOutput(out, true))
// do not scrub sensitive values for included outputs
include = append(include, m.toOutput(out, false))
}
opts = append(opts, jsonapi.MarshalInclude(include...))
}
Expand Down

0 comments on commit 478e314

Please sign in to comment.