Skip to content

Commit b7adaa2

Browse files
jmcauleyclaude
andcommitted
Fix non-exhaustive pattern match in rsky-firehose binary
Add missing Ok(None) arm to the match on firehose::read() in process(). The function returns Ok(None) for intentionally skipped message types (#info, #sync, unknown types) and for recoverable decode errors that the library already logs. The Err arm is preserved to catch genuine CBOR header parse failures. Fixes #150 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 477e72a commit b7adaa2

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

rsky-firehose/src/main.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -268,6 +268,7 @@ async fn process(message: Vec<u8>, client: &reqwest::Client) {
268268
};
269269
}
270270
}
271+
Ok(None) => (),
271272
Err(error) => eprintln!(
272273
"@LOG: Error unwrapping message and header: {}",
273274
error.to_string()

0 commit comments

Comments
 (0)