Make AptosErrorCode compatible with serde deserialization (aptos-labs…

…#3013)

api/types/src/error.rs

@@ -47,6 +47,7 @@ impl From<anyhow::Error> for AptosError {
 // Make sure the integer codes increment one by one.
 #[derive(Debug, Deserialize, Enum)]
 #[oai(rename_all = "snake_case")]
+#[serde(rename_all = "snake_case")]
 pub enum AptosErrorCode {
     /// The API failed to read from storage for this request, not because of a
     /// bad request, but because of some internal error.

crates/aptos-rest-client/Cargo.toml

@@ -17,6 +17,7 @@ dpn = []
 anyhow = "1.0.57"
 bcs = "0.1.3"
 hex = "0.4.3"
+poem-openapi = { git = "https://github.com/poem-web/poem", rev = "f39eba95cbfb52989e0eff516dad86719dc7dcba", features = ["url"] }
 reqwest = { version = "0.11.10", features = ["json", "cookies", "blocking"] }
 serde = { version = "1.0.137", features = ["derive"] }
 serde_json = "1.0.81"

crates/aptos-rest-client/src/lib.rs

@@ -28,6 +28,7 @@ use aptos_types::{
     account_config::{NewBlockEvent, CORE_CODE_ADDRESS},
     transaction::SignedTransaction,
 };
+use poem_openapi::types::ParseFromJSON;
 use reqwest::{header::CONTENT_TYPE, Client as ReqwestClient, StatusCode};
 use serde::{de::DeserializeOwned, Deserialize, Serialize};
 use serde_json::{json, Value};
@@ -510,7 +511,7 @@ impl Client {
         let response = self.inner.get(url.clone()).send().await?;
 
         if !response.status().is_success() {
-            let error_response = response.json::<AptosError>().await?;
+            let error_response = AptosError::parse_from_json(Some(response.json().await?));
             return Err(anyhow::anyhow!("Request failed: {:?}", error_response));
         }
 
@@ -525,7 +526,7 @@ impl Client {
         response: reqwest::Response,
     ) -> Result<(reqwest::Response, State)> {
         if !response.status().is_success() {
-            let error_response = response.json::<AptosError>().await?;
+            let error_response = AptosError::parse_from_json(Some(response.json().await?));
             return Err(anyhow::anyhow!("Request failed: {:?}", error_response));
         }
         let state = State::from_headers(response.headers())?;