🧹 Don't unnecessarily recalculate framework dep checksums

This causes us to load frameworks to calculate checksums we already have. If checksums need to be recalculated, they get cleared first
mondoohq · Jul 26, 2023 · 41a5464 · 41a5464
1 parent 9f15ecf
commit 41a5464
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/policy/framework.go b/policy/framework.go
@@ -246,8 +246,10 @@ func (f *Framework) updateGraphChecksums(
  depObj.FrameworkMaps = frameworkMaps
  }
 
- if err := depObj.UpdateChecksums(ctx, getFramework, getFrameworkMaps, bundle); err != nil {
- return err
+ if depObj.LocalExecutionChecksum == "" || depObj.LocalContentChecksum == "" || depObj.GraphExecutionChecksum == "" || depObj.GraphContentChecksum == "" {
+ if err := depObj.UpdateChecksums(ctx, getFramework, getFrameworkMaps, bundle); err != nil {
+ return err
+ }
  }
 
  graphExecutionChecksum = graphExecutionChecksum.