-
Notifications
You must be signed in to change notification settings - Fork 37.7k
net: Add -allowinbound config option #25718
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
Conversation
@mzumsande has suggested having separate |
I think this can and should be resolved in a simpler way by removing The new option |
I like the |
What would |
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
Maybe I don't understand the purpose of the PR, but I thought it would make changes such as: |
🐙 This pull request conflicts with the target branch and needs rebase. Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft". |
There hasn't been much activity lately and the patch still needs rebase. What is the status here?
|
1 similar comment
There hasn't been much activity lately and the patch still needs rebase. What is the status here?
|
Closing this for now as I currently don't have time to continue with this. I hope I can revisit it in the future. |
(This is just a very rough draft, I am only looking for conceptual feedback right now, so this is the bare minimum code to understand what it should do. I will add tests and fix docs when the conceptual review hurdle is cleared.)
This is an alternative approach to what was already attempted with #25690 and #24835. Please have a look there for more background information.
The
-allowinbound
option lets users explicitly mark networks as available for inbound connections, even if that network may be explicitly deactivated for outbound connections or if it is blocked for other reasons that are out of the user's control. The effect is that the-externalip
addresses of that network may be announced and the-getnetworkinfo
RPC will show these addresses. These several users seem to have run into these issues, see for example #25669.