fix(apps/hermes/server): use RpcPriceIdentifier instead of PriceIdent…

…ifier (#1662) * use RpcPriceIdentifier instead of PriceIdentifier * fix test * fix imports
pyth-network · Jun 6, 2024 · 421f349 · 421f349
1 parent 83c1e43
commit 421f349
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 7 deletions.
diff --git a/apps/hermes/Cargo.lock b/apps/hermes/Cargo.lock
diff --git a/apps/hermes/Cargo.toml b/apps/hermes/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name        = "hermes"
-version     = "0.5.11"
+version     = "0.5.12"
 description = "Hermes is an agent that provides Verified Prices from the Pythnet Pyth Oracle."
 edition     = "2021"
 

diff --git a/apps/hermes/src/api/types.rs b/apps/hermes/src/api/types.rs
@@ -327,7 +327,7 @@ impl TryFrom<PriceUpdate> for PriceFeedsWithUpdateData {
 
 #[derive(Debug, Clone, Serialize, Deserialize, ToSchema)]
 pub struct PriceFeedMetadata {
-    pub id:         PriceIdentifier,
+    pub id:         RpcPriceIdentifier,
     // BTreeMap is used to automatically sort the keys to ensure consistent ordering of attributes in the JSON response.
     // This enhances user experience by providing a predictable structure, avoiding confusion from varying orders in different responses.
     pub attributes: BTreeMap<String, String>,

diff --git a/apps/hermes/src/network/pythnet.rs b/apps/hermes/src/network/pythnet.rs
@@ -4,7 +4,10 @@
 
 use {
     crate::{
-        api::types::PriceFeedMetadata,
+        api::types::{
+            PriceFeedMetadata,
+            RpcPriceIdentifier,
+        },
         config::RunOptions,
         network::wormhole::{
             BridgeData,
@@ -30,7 +33,6 @@ use {
     },
     borsh::BorshDeserialize,
     futures::stream::StreamExt,
-    pyth_sdk::PriceIdentifier,
     pyth_sdk_solana::state::{
         load_mapping_account,
         load_product_account,
@@ -442,7 +444,7 @@ async fn fetch_price_feeds_metadata(
                             .expect("Invalid length for PriceIdentifier");
 
                         let price_feed_metadata = PriceFeedMetadata {
-                            id: PriceIdentifier::new(px_pkey_array),
+                            id: RpcPriceIdentifier::new(px_pkey_array),
                             attributes,
                         };
 

diff --git a/apps/hermes/src/state/aggregate.rs b/apps/hermes/src/state/aggregate.rs
@@ -19,6 +19,7 @@ use {
         WormholeMerkleState,
     },
     crate::{
+        api::types::RpcPriceIdentifier,
         network::wormhole::VaaBytes,
         state::{
             benchmarks::Benchmarks,
@@ -870,7 +871,7 @@ mod test {
         // Add a dummy price feeds metadata
         state
             .store_price_feeds_metadata(&[PriceFeedMetadata {
-                id:         PriceIdentifier::new([100; 32]),
+                id:         RpcPriceIdentifier::new([100; 32]),
                 attributes: Default::default(),
             }])
             .await