Mange API keys if agent checks in with revision that is too high

michel-laterman · michel-laterman · commit e49c9d326a3b · 2025-09-22T14:31:23.000-07:00
diff --git a/internal/pkg/api/handleCheckin.go b/internal/pkg/api/handleCheckin.go
@@ -1177,7 +1177,7 @@ func (ct *CheckinT) processPolicyDetails(ctx context.Context, zlog zerolog.Logge
 	// Check if the checkin revision_idx is greater than the latest available
 	latestRev := ct.pm.LatestRev(ctx, agent.PolicyID)
 	if latestRev != 0 && revisionIDX > latestRev {
-		return 0, opts, nil
+		revisionIDX = 0 // set return val to 0 so the agent gets latest available revision.
 	}
 
 	// Update API keys if the policy has changed, or if the revision differs.

Original file line number	Diff line number	Diff line change
`@@ -1177,7 +1177,7 @@ func (ct *CheckinT) processPolicyDetails(ctx context.Context, zlog zerolog.Logge`
`1177`	`1177`	`// Check if the checkin revision_idx is greater than the latest available`
`1178`	`1178`	`latestRev := ct.pm.LatestRev(ctx, agent.PolicyID)`
`1179`	`1179`	`if latestRev != 0 && revisionIDX > latestRev {`
`1180`		`- return 0, opts, nil`
	`1180`	`+ revisionIDX = 0 // set return val to 0 so the agent gets latest available revision.`
`1181`	`1181`	`}`
`1182`	`1182`
`1183`	`1183`	`// Update API keys if the policy has changed, or if the revision differs.`