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

Issue with local roots which use more than one domain name #4491

Closed
bolt12 opened this issue Apr 5, 2023 · 0 comments · Fixed by #4494
Closed

Issue with local roots which use more than one domain name #4491

bolt12 opened this issue Apr 5, 2023 · 0 comments · Fixed by #4494
Assignees
Labels
bug Something isn't working DNS Issues / PRs related to DNS

Comments

@bolt12
Copy link
Contributor

bolt12 commented Apr 5, 2023

The node fails to maintain a hot connection to all configured local root peers, if more than 1 DNS name is used in a single group.

Splitting the relays up into different access points (groups) works as a workaround.

@bolt12 bolt12 self-assigned this Apr 5, 2023
@bolt12 bolt12 added networking DNS Issues / PRs related to DNS labels Apr 5, 2023
@coot coot changed the title Issue with LocalRoot configurations that use more than 1 domain names Issue with local roots which use more than one domain name Apr 5, 2023
@coot coot added the bug Something isn't working label Apr 5, 2023
@bolt12 bolt12 mentioned this issue Apr 5, 2023
iohk-bors bot added a commit that referenced this issue Apr 18, 2023
4494: Fix issue #4491 r=bolt12 a=bolt12

Updates the result local root groups TVar with up to date information instead of static one.

Also adds & fixes tests to check for this particular case.

Closes #4491

Co-authored-by: Armando Santos <armando@well-typed.com>
@iohk-bors iohk-bors bot closed this as completed in 3ab2260 Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working DNS Issues / PRs related to DNS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants