allow fastrouter subscriptions to use hostnames - #2012
Open
terencehonles wants to merge 1 commit into
Open
terencehonles wants to merge 1 commit into
terencehonles wants to merge 1 commit into
Conversation
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
2 times, most recently
from
May 8, 2019 17:18
3b4739e to
310951c
Compare
Contributor
Author
|
@xrmx Any chance this can get reviewed? |
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
July 8, 2020 23:58
310951c to
4f81d6f
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
November 1, 2022 10:52
4f81d6f to
3604ab0
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
September 7, 2023 11:51
3604ab0 to
d639098
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
November 16, 2023 02:50
d639098 to
8396089
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
February 9, 2024 09:31
8396089 to
3d71183
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
April 26, 2024 08:44
3d71183 to
fb38663
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
June 5, 2024 18:57
fb38663 to
8322fa0
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
October 10, 2024 07:58
8322fa0 to
8055cda
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
January 16, 2025 19:01
8055cda to
ef8aed0
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
June 30, 2025 22:55
ef8aed0 to
9d8777d
Compare
terencehonles
force-pushed
the
update-fastrouter-subscriptions-to-allow-hostnames
branch
from
December 3, 2025 08:05
9d8777d to
bb7d72c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update: Backported in #2013
The following patch allows the following hostname subscriptions:
--subscribe-to NODE_HOSTNAME:3031=ROUTER_HOSTNAME:2626:FASTROUTER_KEY--subscribe2 addr=NODE_HOSTNAME:3031,server=ROUTER_HOSTNAME:2626,key=FASTROUTER_KEYinstead of manually resolving the IP addresses:
--subscribe-to 10.0.0.2:3031=10.0.0.1:2626:FASTROUTER_KEY--subscribe2 addr=10.0.0.2:3031,server=10.0.0.1:2626,key=FASTROUTER_KEYAddresses are resolved by the fastrouter client rather than the server so I didn't have to update the request verification on the server (signing). This also means that new clients can understand hostnames without having to restart the server. If server side address resolution is preferred I can look at updating fastrouter/corerouter, but an initial investigation suggested it was not a simple change.
In addition to above, I also made the following changes:
subscribe-toparsing by noting thataddrcan actually be specified as long as=isn't the first character of the "named socket" (and as mentioned above I added hostname resolution on that code path). NOTE: should this be updated in the documentation? Currently the docs suggest you need to usesubscribe2, but instead of closing the undocumented behavior I updated it to match the handling I was adding tosubscribe2