Skip to content

[Reputation Oracle] Use manifest encryption state for final results#3898

Merged
flopez7 merged 1 commit into
developfrom
flopez/encrypt-final-results
May 19, 2026
Merged

[Reputation Oracle] Use manifest encryption state for final results#3898
flopez7 merged 1 commit into
developfrom
flopez/encrypt-final-results

Conversation

@flopez7
Copy link
Copy Markdown
Contributor

@flopez7 flopez7 commented May 12, 2026

Issue tracking

Freestyle

Context behind the change

Final results were always encrypted during escrow completion. This could block completion when the required PGP public keys were missing. The agreed behavior is to follow the manifest encryption state instead:

  • encrypted manifest -> encrypt final results
  • plaintext manifest -> upload final results without encryption

This keeps result handling consistent with how the job launcher created the escrow manifest.

How has this been tested?

TBT

Release plan

None

Potential risks; What to monitor; Rollback plan

None

@flopez7 flopez7 requested a review from portuu3 May 12, 2026 14:26
@flopez7 flopez7 self-assigned this May 12, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

5 Skipped Deployments
Project Deployment Actions Updated (UTC)
faucet-frontend Ignored Ignored May 12, 2026 2:26pm
faucet-server Ignored Ignored May 12, 2026 2:26pm
human-app Skipped Skipped May 12, 2026 2:26pm
human-dashboard-frontend Skipped Skipped May 12, 2026 2:26pm
staking-dashboard Skipped Skipped May 12, 2026 2:26pm

Request Review

@flopez7 flopez7 merged commit 6150621 into develop May 19, 2026
13 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Core tech - 2026 (H1) May 19, 2026
@portuu3 portuu3 deleted the flopez/encrypt-final-results branch May 20, 2026 09:31
@portuu3 portuu3 mentioned this pull request May 20, 2026
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants