Skip to content

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

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

Merged
merged 1 commit into from
Apr 1, 2025

Conversation

nahuhh
Copy link
Contributor

@nahuhh nahuhh commented Sep 3, 2024

No description provided.

Copy link

netlify bot commented Sep 3, 2024

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit d6d52fc
🔍 Latest deploy log https://app.netlify.com/sites/barolo-time-757cf9/deploys/67e9d3847be06d00096d45fb
😎 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 nahuhh force-pushed the nodes branch 5 times, most recently from 92df44a to c04cfac Compare September 3, 2024 04:41
@nahuhh
Copy link
Contributor Author

nahuhh commented Sep 3, 2024

todo:

  • fix image extension or replace with an actual png
  • translations..?

@plowsof
Copy link
Collaborator

plowsof commented Sep 4, 2024

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
Copy link
Contributor

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

@nahuhh
Copy link
Contributor Author

nahuhh commented Sep 7, 2024

They are on p2p as well, to help defeat dandelion

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

@nahuhh
Copy link
Contributor Author

nahuhh commented Sep 7, 2024

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

@HardenedSteel
Copy link
Contributor

HardenedSteel commented Sep 7, 2024

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
Copy link
Contributor Author

nahuhh commented Sep 8, 2024

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

Added a warning

Copy link
Contributor

@HardenedSteel HardenedSteel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May be too technical for users otherwise OK.

@plowsof
Copy link
Collaborator

plowsof commented Sep 8, 2024

May be too technical for users otherwise OK.

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

@plowsof plowsof merged commit d41c9a7 into monero-project:master Apr 1, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants