Skip to content

fix: default prover-db indexer to disabled unless explicitly enabled - #216

Merged
JayWhite2357 merged 1 commit into
mainfrom
fix/prover-db-indexer-enabled-gate
Aug 10, 2026
Merged

fix: default prover-db indexer to disabled unless explicitly enabled#216
JayWhite2357 merged 1 commit into
mainfrom
fix/prover-db-indexer-enabled-gate

Conversation

@JayWhite2357

Copy link
Copy Markdown
Contributor

Rationale for this change

The offchain worker logged an error every block whenever the prover-db indexer URL wasn't configured. Running offchain workers without opting into the indexer is a normal configuration, not an error state.

What changes are included in this PR?

Add a prover_db_indexer/enabled config key (default false) that gates the OCW consumer.

Are these changes tested?

Yes

@JayWhite2357
JayWhite2357 requested review from a team as code owners August 10, 2026 14:11
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

1.70.0

Bug Fixes

  • default prover-db indexer to disabled unless explicitly enabled (360803d
  • fallback to client hash when is missing (95794da
  • make on chain indexer keys saturate rather than default to (5f794de
  • use finalized block in prover-db-indexer (50438ab

Features

  • add helper that queries the client for s. (4fc3952
  • add client native interface and extenality (d9c6c4d
  • add generic config key-value store for offchain workers (117b845
  • adds (a5b581d
  • Document Snapshot Download Process (4c2608b
  • drop from prover_db_indexer (b3381c5
  • register client externality to offchain worker (338c213
  • use client for (6679c06
  • use generic key value config for prover_db_indexing instead of storage/CLI based config (faff564

@JayWhite2357

Copy link
Copy Markdown
Contributor Author

AI attribution: largely AI generated, with small manual refinements.

tlovell-sxt
tlovell-sxt previously approved these changes Aug 10, 2026
Comment thread sxt-core/src/prover_db_indexer.rs Outdated
@JayWhite2357
JayWhite2357 force-pushed the fix/prover-db-indexer-enabled-gate branch from 52369a0 to 360803d Compare August 10, 2026 22:39
@JayWhite2357
JayWhite2357 enabled auto-merge August 10, 2026 22:39
@JayWhite2357
JayWhite2357 merged commit 2443e99 into main Aug 10, 2026
9 checks passed
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.

2 participants