Aimeedeer / realtps

https://realtps.net
MIT License
93 stars 25 forks source link

Panic in esplora client #71

Open brson opened 1 year ago

brson commented 1 year ago
thread 'main' panicked at 'u64: ParseIntError { kind: InvalidDigit }', src/realtps_import/src/clients/esplora.rs:51:41

This crashed the entire import process. Need to handle panics better: https://github.com/Aimeedeer/realtps/issues/65

I'm disabling bitcoin for now.

brson commented 1 year ago
    async fn get_latest_block_number(&self) -> Result<u64> {
        let block_number = reqwest::get(format!("{}/{}", self.url, "blocks/tip/height"))
            .await?
            .text()
            .await?;

        Ok(u64::from_str(&block_number).expect("u64"))
    }

This code should error if the block can't parse.

brson commented 1 year ago

In general it looks like we never check that status of HTTP reqwest responses: https://github.com/Aimeedeer/realtps/issues/72