Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

security brainstorming: how do we stop pairing IP address with wallet address? #23

Open
arilotter opened this issue Jan 4, 2022 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Comments

@arilotter
Copy link
Owner

Broadcasting IPs to everyone is dangerous. Perhaps we need to move off webtorrent trackers, and to a more centralized but less dangerous service.
Or, figure out a way to publish some information to a WebTorrent tracker without publishing an IP - maybe the Vaportrade site itself can run a small backend and pin a magnet link to each tracker, and then we can just publish addresses, no IPs, and the Vaportrade site can ferry requests to & from the trackers, just until users trust eachother enough to accept a trade request?

Major kudos to @collabshard on Twitter for these thoughts:

Exposing public IP of a machine with assets on it to untrusted 3rd parties (by design - that's how WebRTC works), what could go wrong.

New iterations should keep Joe in mind.

Joe average probably doesn't
a) Use a VPN (recommend one?)
b) Have a hot/cold wallet setup (Educate him?)
c) Keep his system updated all the time (Educate)
d) Want to get rekt / become an easy target for hackers

IP is the critical piece of information here. If it gets exposed and it's known that it's a Vaportrade user, crypto is at risk.

See thread https://twitter.com/usevaportrade/status/1478173602184847361

@arilotter arilotter added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant