Quicker Builds & Two Small Errors (#180)

Author: bh2smith

data-store/src/store.rs

@@ -81,7 +81,7 @@ impl DataStore {
                     .execute(conn);
                 handle_insert_result(result, 1, format!("insert_metadata: {}", token));
 
-                // one of the following two tables will be updated (depending on token type.)
+                // one of the following two tables should be updated (depending on token type.)
                 let erc721_res = update(nfts::dsl::nfts)
                     .set(nfts::metadata_id.eq::<Vec<u8>>((&*uid).into()))
                     .filter(nfts::contract_address.eq(&token.db_address()))
@@ -92,10 +92,8 @@ impl DataStore {
                     .filter(erc1155s::contract_address.eq(&token.db_address()))
                     .filter(erc1155s::token_id.eq(&token.db_token_id()))
                     .execute(conn);
-                // Exactly one of the two tables should result in an update!
-                assert_eq!(
-                    handle_query_result(erc721_res) + handle_query_result(erc1155_res),
-                    1,
+                assert!(
+                    handle_query_result(erc721_res) + handle_query_result(erc1155_res) > 0,
                     "invalid token update on metadata insertion {} - {:?}",
                     token,
                     uid

docker/Dockerfile.binary

@@ -4,14 +4,14 @@ WORKDIR /src/
 
 # Copy & Build
 COPY . .
-RUN cargo build --release
+RUN cargo build
 
 # Extract Binary
 FROM docker.io/debian:bookworm-slim
 
 RUN apt-get update && apt-get install -y ca-certificates tini libpq-dev
-COPY --from=builder /src/target/release/event-handler /usr/local/bin/event-handler
-COPY --from=builder /src/target/release/metadata-retriever /usr/local/bin/metadata-retriever
+COPY --from=builder /src/target/debug/event-handler /usr/local/bin/event-handler
+COPY --from=builder /src/target/debug/metadata-retriever /usr/local/bin/metadata-retriever
 
 CMD echo "Specify binary..."
 ENTRYPOINT ["/usr/bin/tini", "--"]

metadata-retriever/src/routes/contract/abi.rs

@@ -46,16 +46,10 @@ impl EtherscanApi {
     fn handle_response(
         ApiResponse {
             status,
-            message,
+            message: _,
             result,
         }: ApiResponse<Value>,
     ) -> Result<Option<Value>> {
-        tracing::debug!(
-            "Status {}, message: {:?}, result: {:?}",
-            status,
-            message,
-            result
-        );
         if status == "1" && result.is_some() {
             // The request was successful, return the result
             Ok(

metadata-retriever/src/routes/token/metadata/mod.rs

@@ -76,7 +76,7 @@ impl FetchedMetadata {
 
     pub fn error(text: &str) -> Self {
         Self {
-            hash: vec![0],
+            hash: md5::compute(text.as_bytes()).0.to_vec(),
             raw: Some(text.into()),
             json: None,
         }