zarf icon indicating copy to clipboard operation
zarf copied to clipboard

Fix possible infinite loops with context timeout

Open phillebaba opened this issue 9 months ago • 0 comments

Describe what should be investigated or refactored

The addition of contexts in #2405 fixed some issues with possible infinite loops using contexts. That got me thinking if there are any other examples that are out of scope for that PR. I found two cases that should be fixed.

Links to any relevant code

https://github.com/defenseunicorns/zarf/blob/48c11d99547b59ff2f10fa60078741376059f821/src/pkg/cluster/data.go#L58 https://github.com/defenseunicorns/zarf/blob/48c11d99547b59ff2f10fa60078741376059f821/src/pkg/utils/bytes.go#L67

Additional context

N/A

phillebaba avatar May 15 '24 12:05 phillebaba