check outputs

Revert clippy fix for arc_with_non_send_sync in wasm client

.github/workflows/nym-release-publish.yml

@@ -25,7 +25,7 @@ jobs:
 
     outputs:
       release_id: ${{ steps.create-release.outputs.id }}
-      release_date: ${{ fromJSON(steps.create-release.outputs.assets)[0].published_at }}
+      release_date: ${{ fromJSON(steps.create-release.outputs.assets)[0].created_at }}
       client_hash: ${{ steps.binary-hashes.outputs.client_hash }}
       mixnode_hash: ${{ steps.binary-hashes.outputs.mixnode_hash }}
       gateway_hash: ${{ steps.binary-hashes.outputs.gateway_hash }}
@@ -40,6 +40,7 @@ jobs:
       netreq_version: ${{ steps.binary-versions.outputs.netreq_version }}
       cli_version: ${{ steps.binary-versions.outputs.cli_version }}
       netstat_version: ${{ steps.binary-versions.outputs.netstat_version }}
+      platform: ${{ steps.get-platform-version.outputs.platform }}"
 
     steps:
       - uses: actions/checkout@v3
@@ -96,6 +97,11 @@ jobs:
             target/release/nym-network-statistics
             target/release/nym-cli
 
+      - id: echo-outputs
+        name: echo output for assets
+        run: |
+          echo "data from release: $ ${{ steps.create-release.outputs }}"
+
       - id: release-info
         name: Prepare release info
         run: |
@@ -124,6 +130,12 @@ jobs:
           v=$(rg '^version = "(.*)"' -or '$1' tools/nym-cli/Cargo.toml) && echo "cli_version=$v" >> "$GITHUB_OUTPUT"
           v=$(rg '^version = "(.*)"' -or '$1' service-providers/network-statistics/Cargo.toml) && echo "netstat_version=$v" >> "$GITHUB_OUTPUT"
 
+      - id: get-platform-version
+        name: get platform version
+        run: |
+          echo "::set-output name=platform::$(uname -r)"
+
+
   push-release-data-client:
     if: ${{ (startsWith(github.ref, 'refs/tags/nym-binaries-') && github.event_name == 'release') || github.event_name == 'workflow_dispatch' }}
     uses: ./.github/workflows/push-release-data.yml
@@ -139,6 +151,7 @@ jobs:
       file_hash: ${{ needs.publish-nym.outputs.client_hash }}
       name: Client
       category: binaries
+      platform: "${{ needs.publish-nym.outputs.platform }}"
     secrets: inherit
 
   push-release-data-mixnode:
@@ -156,6 +169,7 @@ jobs:
       file_hash: ${{ needs.publish-nym.outputs.mixnode_hash }}
       name: Mixnode
       category: binaries
+      platform: "${{ needs.publish-nym.outputs.platform }}"
     secrets: inherit
 
   push-release-data-gateway:
@@ -173,6 +187,7 @@ jobs:
       file_hash: ${{ needs.publish-nym.outputs.gateway_hash }}
       name: Gateway
       category: binaries
+      platform: "${{ needs.publish-nym.outputs.platform }}"
     secrets: inherit
 
   push-release-data-socks5:
@@ -190,6 +205,7 @@ jobs:
       file_hash: ${{ needs.publish-nym.outputs.socks5_hash }}
       name: Socks5 Client
       category: binaries
+      platform: "${{ needs.publish-nym.outputs.platform }}"
     secrets: inherit
 
   push-release-data-network-requester:
@@ -207,6 +223,7 @@ jobs:
       file_hash: ${{ needs.publish-nym.outputs.netreq_hash }}
       name: Network Requester
       category: binaries
+      platform: "${{ needs.publish-nym.outputs.platform }}"
     secrets: inherit
 
   push-release-data-cli:
@@ -224,6 +241,7 @@ jobs:
       file_hash: ${{ needs.publish-nym.outputs.cli_hash }}
       name: Cli
       category: binaries
+      platform: "${{ needs.publish-nym.outputs.platform }}"
     secrets: inherit
 
   push-release-data-network-stat:
@@ -241,4 +259,5 @@ jobs:
       file_hash: ${{ needs.publish-nym.outputs.netstat_hash }}
       name: Network Statistics
       category: binaries
+      platform: "${{ needs.publish-nym.outputs.platform }}"
     secrets: inherit

clients/webassembly/src/lib.rs

@@ -1,6 +1,10 @@
 // Copyright 2021-2023 - Nym Technologies SA <contact@nymtech.net>
 // SPDX-License-Identifier: Apache-2.0
 
+// After reading https://github.com/rust-lang/rust-clippy/issues/11382
+// I suspect we *maybe* have hit a false positive, but I'm not sure.
+#![allow(clippy::arc_with_non_send_sync)]
+
 use wasm_bindgen::prelude::*;
 
 #[cfg(target_arch = "wasm32")]

clients/webassembly/src/storage/mod.rs

@@ -1,8 +1,6 @@
 // Copyright 2023 - Nym Technologies SA <contact@nymtech.net>
 // SPDX-License-Identifier: Apache-2.0
 
-use std::rc::Rc;
-
 use crate::client::config::Config;
 use crate::storage::error::ClientStorageError;
 use js_sys::Promise;
@@ -10,6 +8,7 @@ use nym_client_core::client::base_client::storage::gateway_details::PersistedGat
 use nym_crypto::asymmetric::{encryption, identity};
 use nym_gateway_client::SharedKeys;
 use nym_sphinx::acknowledgements::AckKey;
+use std::sync::Arc;
 use wasm_bindgen::prelude::*;
 use wasm_bindgen_futures::future_to_promise;
 use wasm_utils::storage::{IdbVersionChangeEvent, WasmStorage};
@@ -44,7 +43,7 @@ mod v1 {
 pub struct ClientStorage {
     #[allow(dead_code)]
     pub(crate) name: String,
-    pub(crate) inner: Rc<WasmStorage>,
+    pub(crate) inner: Arc<WasmStorage>,
 }
 
 #[wasm_bindgen]
@@ -89,7 +88,7 @@ impl ClientStorage {
         .await?;
 
         Ok(ClientStorage {
-            inner: Rc::new(inner),
+            inner: Arc::new(inner),
             name,
         })
     }

clients/webassembly/src/tester/mod.rs

@@ -29,7 +29,6 @@ use nym_validator_client::client::IdentityKey;
 use nym_validator_client::QueryReqwestRpcNyxdClient;
 use rand::rngs::OsRng;
 use std::collections::HashSet;
-use std::rc::Rc;
 use std::sync::atomic::{AtomicBool, AtomicU32, Ordering};
 use std::sync::{Arc, Mutex as SyncMutex};
 use std::time::Duration;
@@ -43,7 +42,7 @@ pub(crate) mod helpers;
 
 pub type NodeTestMessage = TestMessage<WasmTestMessageExt>;
 type LockedGatewayClient =
-    Rc<AsyncMutex<GatewayClient<QueryReqwestRpcNyxdClient, EphemeralStorage>>>;
+    Arc<AsyncMutex<GatewayClient<QueryReqwestRpcNyxdClient, EphemeralStorage>>>;
 
 pub(crate) const DEFAULT_TEST_TIMEOUT: Duration = Duration::from_secs(10);
 pub(crate) const DEFAULT_TEST_PACKETS: u32 = 20;
@@ -211,7 +210,7 @@ impl NymNodeTesterBuilder {
             test_in_progress: Arc::new(AtomicBool::new(false)),
             current_test_nonce: Default::default(),
             tester: Arc::new(SyncMutex::new(tester)),
-            gateway_client: Rc::new(AsyncMutex::new(gateway_client)),
+            gateway_client: Arc::new(AsyncMutex::new(gateway_client)),
             processed_receiver: ReceivedReceiverWrapper::new(processed_receiver),
             _task_manager: task_manager,
         })
@@ -344,7 +343,7 @@ impl NymNodeTester {
         ));
 
         let processed_receiver_clone = self.processed_receiver.clone();
-        let gateway_client_clone = Rc::clone(&self.gateway_client);
+        let gateway_client_clone = Arc::clone(&self.gateway_client);
         let tester_marker = TestMarker::new(Arc::clone(&self.test_in_progress));
 
         // start doing async things (send packets and watch for anything coming back)