{"record":{"id":"f8c1f58c7689ce3b","repo":"nautechsystems/nautilus_trader","slug":"contract-address-should-be-set-in-logs-f8c1f5","errorCode":null,"errorMessage":"Contract address should be set in logs","messagePattern":"Contract address should be set in logs","errorType":"panic","errorClass":null,"httpStatus":null,"severity":"error","filePath":"crates/adapters/blockchain/src/exchanges/parsing/uniswap_v3/fee_protocol_update.rs","lineNumber":82,"sourceCode":"    )?;\n\n    if let Some(data) = &log.data {\n        let data_bytes = data.as_ref();\n\n        // Validate the data contains 4 parameters of 32 bytes each\n        if data_bytes.len() < 4 * 32 {\n            anyhow::bail!(\"SetFeeProtocol event data is too short\");\n        }\n\n        let decoded = match <SetFeeProtocolEventData as SolType>::abi_decode(data_bytes) {\n            Ok(decoded) => decoded,\n            Err(e) => anyhow::bail!(\"Failed to decode SetFeeProtocol event data: {e}\"),\n        };\n\n        let pool_address = Address::from_slice(\n            log.address\n                .clone()\n                .expect(\"Contract address should be set in logs\")\n                .as_ref(),\n        );\n        let pool_identifier = PoolIdentifier::Address(Ustr::from(&pool_address.to_string()));\n\n        Ok(FeeProtocolUpdateEvent::new(\n            dex,\n            pool_identifier,\n            extract_block_number(log)?,\n            extract_transaction_hash(log)?,\n            extract_transaction_index(log)?,\n            extract_log_index(log)?,\n            u32::from(decoded.fee_protocol0_new),\n            u32::from(decoded.fee_protocol1_new),\n        ))\n    } else {\n        anyhow::bail!(\"Missing data in SetFeeProtocol event log\");\n    }\n}","sourceCodeStart":64,"sourceCodeEnd":100,"githubUrl":"https://github.com/nautechsystems/nautilus_trader/blob/18893faf8b356be3320add8de2f861b0b647cf06/crates/adapters/blockchain/src/exchanges/parsing/uniswap_v3/fee_protocol_update.rs#L64-L100","documentation":"parse_fee_protocol_update_event_hypersync (Uniswap V3 variant) expects log.address to be Some: the address becomes the pool identifier of the FeeProtocolUpdateEvent. Since every Ethereum log carries its emitting contract address, the library asserts this with expect(); a None address indicates malformed input and panics, as documented in # Panics.","triggerScenarios":"Calling parse_fee_protocol_update_event_hypersync with a HypersyncLog whose address is None — from Hypersync queries omitting the address column, upstream partial rows, or test fixtures lacking address.","commonSituations":"Misconfigured Hypersync projections; Hypersync client/schema version changes; synthetic logs built in unit tests.","solutions":["Ensure the address field is selected in the Hypersync log query.","Filter out address-less logs before parsing.","Return an anyhow error instead of panicking when address is missing."],"exampleFix":"// before\nlet pool_address = Address::from_slice(\n    log.address.clone().expect(\"Contract address should be set in logs\").as_ref(),\n);\n// after\nlet raw_address = log.address.as_ref()\n    .ok_or_else(|| anyhow::anyhow!(\"missing contract address in SetFeeProtocol log\"))?;\nlet pool_address = Address::from_slice(raw_address.as_ref());","handlingStrategy":"validation","validationCode":"if log.address.is_none() {\n    log::warn!(\"skipping SetFeeProtocol log without contract address\");\n    return Ok(());\n}\nlet event = parse_fee_protocol_update_event_hypersync(dex.clone(), &log)?;","typeGuard":"fn has_address(log: &HypersyncLog) -> bool {\n    log.address.is_some()\n}","tryCatchPattern":"match parse_fee_protocol_update_event_hypersync(dex, &log) {\n    Ok(event) => handle(event),\n    Err(e) => log::error!(\"failed to parse SetFeeProtocol log: {e:#}\"),\n}\n// Note: missing address panics via expect(); pre-check log.address before calling.","preventionTips":["Include the address field in Hypersync query selection.","Apply a shared pre-parse validation step for all Hypersync logs.","After client upgrades, smoke-test that returned logs always carry addresses."],"tags":["panic","missing-field","hypersync","blockchain-logs","rust"],"backgroundTag":"null-argument","analyzedSha":"18893faf8b356be3320add8de2f861b0b647cf06","analyzedAt":"2026-09-08T20:49:34.690Z","contentChangedAt":"2026-09-08T20:49:34.690Z","schemaVersion":2},"datasetVersion":"2026-09-14T05:17:10.506Z"}