Merged
Conversation
Member
Author
|
I am running a local sanity test. |
GrpcIndexer::new() now returns Result, get_trees() was renamed to get_tree_state() taking BlockId, and set_indexer_uri() now returns Result since GrpcIndexer is immutable after construction. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Both ring and aws-lc-rs get unified into the dependency tree via transitive deps, preventing rustls from auto-selecting a provider. Also fix test helper to supply a valid indexer URI since GrpcIndexer::new now validates the scheme eagerly. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
ClientConfigBuilder::build() now defaults to DEFAULT_INDEXER_URI instead of an empty URI, since GrpcIndexer::new validates the scheme eagerly in netutils 4.0.0. Also install ring as the default rustls CryptoProvider in LightClient::new to handle both ring and aws-lc-rs being unified into the dep tree via transitive dependencies. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
pacu
approved these changes
Mar 29, 2026
pacu
approved these changes
Mar 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes: #2312
This updates to the modern pattern where "Indexer" is a first class Rust abstraction that is maintained in the zingo-netutils package.