network: fix publicKeyIdentTracker data race in hybrid mode

[algorandskiy]

Summary

Fix data race in hybrid mode when a shared publicKeyIdentTracker.peersByID is accessed from both p2p and ws net instances - added a mutex to publicKeyIdentTracker. Since net identity is called only on connect/disconnect (i.e. not very frequent), mutex is OK to use.

Test Plan

Existing tests

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.25%. Comparing base (8e2e2e4) to head (6dd7fd4).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6110      +/-   ##
==========================================
+ Coverage   56.22%   56.25%   +0.03%     
==========================================
  Files         490      490              
  Lines       69705    69710       +5     
==========================================
+ Hits        39191    39217      +26     
+ Misses      27840    27822      -18     
+ Partials     2674     2671       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gmalouf]

Makes sense