search

Beldex-Coin / beldex

Beldex: the secure, private, untraceable cryptocurrency
https://beldex.io
Other
27 stars 21 forks source link

Connected peers exceeded max count #74

Closed codeman-crypto closed 2 years ago

codeman-crypto commented 2 years ago

The max peers(in and out) connected to a node should be 40 but, the nodes are connect to more than 40 peers.

Screenshot 2022-06-09 at 3 41 05 PM

The above screenshot shows 36 out and 32 in so, totally there are 69 peers connected which is higher than the max peer connection count

logs

2022-05-31 05:34:33.987 [P2P7]  ERROR   net contrib/epee/include/epee/net/levin_protocol_handler_async.h:342    [15.207.44.251:19090 OUT] [levin_protocol] -->> start_outer_call failed
2022-05-31 05:34:33.987 [P2P7]  ERROR   net contrib/epee/include/epee/net/levin_protocol_handler_async.h:342    [49.205.84.124:31636 INC] [levin_protocol] -->> start_outer_call failed

Note: Looks like the dead connections are not being dropped.

victor-tucci commented 2 years ago

In the function try_ping LOG_PRINT_CC_L0(context,"ping handler before callback" ); cb(); //zone.m_net_server.get_config_object().close(ping_context.m_connection_id); in net_node.inl this may can't drop the connection id. i enable this close() function and looking into the logs.

victor-tucci commented 2 years ago

checked with the close(). the timeout peers handshake are closed. #75

codeman-crypto commented 2 years ago

Resolved in #75