nim-libp2p icon indicating copy to clipboard operation
nim-libp2p copied to clipboard

Various cleanups

Open Menduist opened this issue 3 years ago • 1 comments

Some groundwork for V1:

  • [ ] Mount PushIdentify by default in newStandardSwitch
  • [ ] Try to remove as much exported procs & fields as possible
  • [x] Remove raise without explicit exception (should be raise exc)
  • [x] Check that we don't try to catch RangeError (or other defects)
  • [x] Remove the need for stdlib random
  • [x] Check the new/init procs
  • [x] Remove deprecated procs
  • [ ] Fix compilation warnings
  • [ ] Try to lower the recursive dependency count as low as possible

Menduist avatar Aug 31 '21 15:08 Menduist

Mount PushIdentify by default in newStandardSwitch

we've avoided any strictly unnecessary protocols so as to reduce the security surface area and information leaks in the "default" setup - by not having them enabled, users of libp2p are made aware of each protocol that they're enabling so they can analyze its usefulness for their application - eth2 for example likely shouldn't have it enabled

arnetheduck avatar Sep 21 '21 10:09 arnetheduck

Done except the Push Identify

Menduist avatar Oct 12 '23 09:10 Menduist