search

monero-project / monero-site

https://getmonero.org
BSD 3-Clause "New" or "Revised" License
275 stars 384 forks source link

user-guides: don't recommend remote nodes #2367

Open nahuhh opened 3 weeks ago

netlify[bot] commented 3 weeks ago

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
Latest commit 77030ecfc8d4fb82ca0d37400bbfa267025c6281
Latest deploy log https://app.netlify.com/sites/barolo-time-757cf9/deploys/66ddffb8bedcb000086cd3c8
Deploy Preview https://deploy-preview-2367--barolo-time-757cf9.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

nahuhh commented 3 weeks ago

todo:

plowsof commented 3 weeks ago

todo:

  • translations..?

Pulled into my branch with +100k/-100k diff PR to be thrown into the memory hole when new site comes.

OR if this is security sensitive i could make a specific PR for only this as its new.

HardenedSteel commented 3 weeks ago

We should warn about there are nodes run by malicious actors which are setup solely for de-anonymizing people.

nahuhh commented 3 weeks ago

They are on p2p as well, to help defeat dandelion

https://b10c.me/observations/06-linkinglion/

nahuhh commented 3 weeks ago

Is this PR concept ok? (should i proceed with the final touches on the english page and image?)

HardenedSteel commented 3 weeks ago

when we just simply say discouraged user may wonder why, its better to include the warning I stated above imo.

the new screenshot is ok for this PR.

new PR can be made for better quality. current screenshots look blurry to me.

nahuhh commented 3 weeks ago

We should warn about there are nodes run by malicious actors which are setup solely for de-anonymizing people.

Added a warning

plowsof commented 3 weeks ago

May be too technical for users otherwise OK.

Can we improve? which are in particular? the real spends / outputs point?