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

Fix --anonymous-inbound data leak [0.18] #9633

Open
wants to merge 1 commit into
base: release-v0.18
Choose a base branch
from

Conversation

vtnerd
Copy link
Contributor

@vtnerd vtnerd commented Dec 19, 2024

Same as #9632 but on release branch.

@vtnerd vtnerd force-pushed the fix/anonymous_inbound_0.18 branch from 9221a8d to 2b1e383 Compare December 19, 2024 23:49
@vtnerd
Copy link
Contributor Author

vtnerd commented Dec 19, 2024

See other PR for force push explanation.

@vtnerd vtnerd changed the title Fix --anonymouns-inbound data leak Fix --anonymous-inbound data leak [0.18] Dec 19, 2024
@vtnerd
Copy link
Contributor Author

vtnerd commented Dec 20, 2024

@selsta something wrong with the 0.18 branch right now? These errors look unrelated.

@selsta
Copy link
Collaborator

selsta commented Dec 20, 2024

@vtnerd you updated the trezor submodule

@vtnerd
Copy link
Contributor Author

vtnerd commented Dec 20, 2024

Yup, I see it now, thanks.

@@ -2483,26 +2484,38 @@ namespace nodetool
std::vector<peerlist_entry> local_peerlist_new;
zone.m_peerlist.get_peerlist_head(local_peerlist_new, true, max_peerlist_size);

/* Tor/I2P nodes receiving connections via forwarding (from tor/i2p daemon)
do not know the address of the connecting peer. This is relayed to them,
iff the node has setup an inbound hidden service. The other peer will have
Copy link
Collaborator

Choose a reason for hiding this comment

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

small typo here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't see the typo, can you elaborate?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I thought iff is supposed to be if but seems it's an abbreviation I wasn't aware of.

@vtnerd vtnerd force-pushed the fix/anonymous_inbound_0.18 branch from 2b1e383 to 53ffc8d Compare December 20, 2024 05:10
@vtnerd
Copy link
Contributor Author

vtnerd commented Dec 20, 2024

The last force push should fix the external submodule that was bad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants