Skip to content

Commit

Permalink
fix(core): omit output.detail from graph events
Browse files Browse the repository at this point in the history
The values for this field can sometimes be too large to be reliably
streamed.
  • Loading branch information
thsig authored and edvald committed Sep 24, 2021
1 parent a4fa362 commit e81a497
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion core/src/events.ts
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,8 @@ export interface ServiceStatusPayload extends Omit<ServiceStatus, "detail"> {
export function toGraphResultEventPayload(result: GraphResult): GraphResultEventPayload {
const payload = omit(result, "dependencyResults")
if (result.output) {
payload.output = omit(result.output, "dependencyResults", "log", "buildLog")
// TODO: Use a combined blacklist of fields from all task types instead of hardcoding here.
payload.output = omit(result.output, "dependencyResults", "log", "buildLog", "detail")
if (result.output.version) {
payload.output.version = result.output.version.versionString || null
}
Expand Down

0 comments on commit e81a497

Please sign in to comment.