Skip to content

fix(cli): resolve attestation init digest mismatch with old servers#2999

Merged
javirln merged 1 commit into
chainloop-dev:mainfrom
javirln:fix/pfm-5378-fix-remote-attestations
Apr 8, 2026
Merged

fix(cli): resolve attestation init digest mismatch with old servers#2999
javirln merged 1 commit into
chainloop-dev:mainfrom
javirln:fix/pfm-5378-fix-remote-attestations

Conversation

@javirln

@javirln javirln commented Apr 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Fix inverted condition in RunCollectors that skipped state reload when the client-side digest was stale (old servers not returning digests from Save RPC)
  • Always reload crafting state after collectors run, ensuring the digest is in sync with the server regardless of server version

Closes PFM-5378

…atch with old servers

Signed-off-by: Javier Rodriguez <javier@chainloop.dev>
@javirln javirln requested review from jiparis and migmartri April 8, 2026 09:20
@javirln javirln self-assigned this Apr 8, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@javirln javirln merged commit aff2fbb into chainloop-dev:main Apr 8, 2026
15 checks passed
@migmartri

Copy link
Copy Markdown
Member

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants