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

Don't use autonat or relay if relay server is enabled #82

Merged
merged 1 commit into from
May 3, 2024

Conversation

ligustah
Copy link
Collaborator

@ligustah ligustah commented May 3, 2024

Autonat is sometimes causing issues for our server nodes. If we know that we have an externally reachable address, we shouldn't rely on autonat to confirm it.

The way the implementation works autonat will always only try to connect to addresses that it can observe from the connecting node, which means it might not confirm the supposed external address of the server node.

Copy link
Collaborator Author

ligustah commented May 3, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ligustah and the rest of your teammates on Graphite Graphite

@ligustah ligustah mentioned this pull request May 3, 2024
@ligustah ligustah requested a review from erikreppel May 3, 2024 15:21
@ligustah ligustah marked this pull request as ready for review May 3, 2024 15:21
@ligustah ligustah force-pushed the logging-improvements branch from a308830 to 470c84a Compare May 3, 2024 16:07
@ligustah ligustah force-pushed the disable-autonat-with-external-address-set branch from ddfd544 to 76b200f Compare May 3, 2024 16:07
@ligustah ligustah force-pushed the logging-improvements branch from 470c84a to 5fe9b69 Compare May 3, 2024 19:06
@erikreppel erikreppel changed the base branch from logging-improvements to main May 3, 2024 19:08
@ligustah ligustah force-pushed the disable-autonat-with-external-address-set branch from 76b200f to f8d6b09 Compare May 3, 2024 19:09
Copy link
Collaborator Author

ligustah commented May 3, 2024

Merge activity

  • May 3, 3:10 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 3, 3:11 PM EDT: @ligustah merged this pull request with Graphite.

@ligustah ligustah merged commit 7b292e5 into main May 3, 2024
3 of 4 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.

2 participants