jsonrpc-feature returned #417

Includes: - re-purposing of feature flags * metrics is always a thing (on) * monitoring is the gated feature * websocket-server flag is gone, we only gate push notifications - jsonrpc setup and rpc method register - prometheus exposition format to json parser - Added todo around split networking config Notes: - will not merge this in until example app has been restored with the client changes

# Description This pull request implements the following changes: - [x] Add event notifications - [x] Add `emit_event` notification utility function - [x] Add `emit_receipt` notification utility function - [x] Move receipt notifications to event handler notifications - [x] Add `ConnnectionEstablished`, `ConnnectionClosed`, `ListeningOn`, `OutgoingConnectionError`, and `IncomingConnectionError` network notifications - [x] Test JSON event notification bytes roundtrip - [x] Test JSON event notification string roundtrip - [x] Integration test connection notifications with two Homestar nodes ## Link to issue Closes #407 ## Type of change - [x] New feature (non-breaking change that adds functionality) ## Test plan (required) We have included unit tests to check roundtrip conversions between JSON bytes and strings. In addition, we have included an integration test that subscribes and listens for connection messages between Homestar nodes. --------- Co-authored-by: Zeeshan Lakhani <[email protected]>

# Description This PR implements the following changes: - [x] Add receipt published and received notifications - [x] Update receipt sharing log messages - [x] Add receipt sharing integration test - [x] Add `remove_db` and `wait_for_socket_connection` test utilities - [x] Update JSON `peer_id` key to `peerId` ## Link to issue Implements #131 ## Type of change - [x] New feature (non-breaking change that adds functionality) - [x] Refactor (non-breaking change that updates existing functionality) ## Test plan (required) This PR includes an integration test that checks for gossiped receipts in websocket notifications and logs. --------- Co-authored-by: Zeeshan Lakhani <[email protected]>

# Description - [x] Fix the connection notification integration test - [x] Avoid event hooks to IPFS in tests ## Type of change - [x] Refactor (non-breaking change that updates existing functionality) ## Test plan (required) Improved connection notification test included to move away from arbitrary waits to event-driven notification tests. --------- Co-authored-by: Zeeshan Lakhani <[email protected]>

# Description This PR includes the following changes: - [x] Set gossip `max_transmit_size` - [x] Handle Kademlia `InboundRequest` events ## Type of change - [x] Refactor (non-breaking change that updates existing functionality)

Includes: - also, add starter functions for default .pem/settings initialization - error handling around retry, wasms - ups the transport timeout

# Description This PR implements the following changes: - [x] Add gosssipsub message wrapper - [x] Rename `store_and_notify` to `publish_and_notify` The message wrapper includes a header with a nonce to force the gossip of duplicate receipts. We will likely expand on the header in future work and make the nonce optional. ## Link to issue Implements #421. ## Type of change - [x] Refactor (non-breaking change that updates existing functionality) ## Test plan (required) We've added a unit test to roundtrip a gossiped message to bytes and back again. We also have a gossip notifications integration test to confirm messages are still sent. --------- Co-authored-by: Zeeshan Lakhani <[email protected]>

Fix: remove unnecessary decode of already utf8-ed string from wasmval.

# Description This PR implements the following changes: - [x] Remove temporary databases after test runs ## Type of change - [x] Refactor (non-breaking change that updates existing functionality) ## Test plan (required) The tests should pass and leave no databases behind.

Includes: - a rename of `asyncboundedchannel*` to `asyncchannel` to accomodate unbounded channels - remove receipt logging with too much output

# Description This PR implements the following changes: - [x] Break up `[node.network]` settings into smaller groups - [x] Move `[monitoring]` settings to `[node.monitoring]` - [x] Add `defaults.toml` with complete settings matching the defaults - [x] Reduce the settings passed to consumers to the minimum necessary - [x] Update test fixtures using the new settings - [x] Update and add comments A complete version of the updated settings will look something like: ```toml [node] gc_interval = 1800 shutdown_timeout = 20 [node.database] url = "homestar.db" max_pool_size = 100 [node.monitoring] process_collector_interval = 5000 console_subscriber_port = 6669 [node.network] events_buffer_len = 1024 poll_cache_interval = 1000 [node.network.ipfs] host = "127.0.0.1" port = 5001 [node.network.libp2p] listen_address = "/ip4/0.0.0.0/tcp/0" node_addresses = [] announce_addresses = [] transport_connection_timeout = 60 max_connected_peers = 32 max_announce_addresses = 10 [node.network.libp2p.mdns] enable = true enable_ipv6 = false query_interval = 300 ttl = 540 [node.network.libp2p.rendezvous] enable_client = true enable_server = false registration_ttl = 7200 discovery_interval = 600 [node.network.libp2p.pubsub] enable = false duplication_cache_time = 1 heartbeat = 60 idle_timeout = 86400 max_transmit_size = 10485760 mesh_n_low = 1 mesh_n_high = 10 mesh_n = 2 mesh_outbound_min = 1 [node.network.libp2p.dht] p2p_provider_timeout = 30 receipt_quorum = 2 workflow_quorum = 3 [node.network.keypair_config] random = { } [node.network.metrics] port = 4000 [node.network.rpc] host = "::1" port = 3030 max_connections = 10 server_timeout = 120 [node.network.webserver] host = "127.0.0.1" port = 1337 timeout = 120 websocket_capacity = 2048 websocket_receiver_timeout = 30000 ``` ## Link to issue Closes #442 ## Type of change - [x] Refactor (non-breaking change that updates existing functionality) - [ ] This change requires a documentation update. (It does. Will do.) - [x] Comments have been added/updated. ## Test plan (required) All existing tests should pass with the updated fixtures.

This PR implements the following changes: - [x] Update example to use Homestar client: https://www.npmjs.com/package/@fission-codes/homestar - [x] Remove emulation mode Implements #390 - [x] Refactor (non-breaking change that updates existing functionality) The example should be tested manually to verify it works. --------- Signed-off-by: Brian Ginsburg <[email protected]> Co-authored-by: Hugo Dias <[email protected]>

- adds npm packages to homestar-runtime. - adds CI/CD to build and publish wrapper package and os specific packages.

# Description Remove many TODOs, add comments on *pub* (*pub* crate), and work through logs. - Includes a few other minor fixins and additions. ## Type of change - [X] Refactor (non-breaking change that updates existing functionality) Closes #262. --------- Signed-off-by: Zeeshan Lakhani <[email protected]> Co-authored-by: Brian Ginsburg <[email protected]>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

jsonrpc-feature returned #417

jsonrpc-feature returned #417

Commits on Nov 29, 2023