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

Make the node name separator in short link format configurable #1755

Open
ipspace opened this issue Jan 4, 2025 · 0 comments
Open

Make the node name separator in short link format configurable #1755

ipspace opened this issue Jan 4, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@ipspace
Copy link
Owner

ipspace commented Jan 4, 2025

Based on a fun discussion with @ddutt:

  • You cannot use the short link format if the hostnames contain '-'
  • We could use some other character as the separator, for example A↔B↔C, but that would break almost any topology out there

However, if we define defaults.const.link.node_separator to be '-' and use that when splitting node names, users could use whatever separator they want ;)

@ipspace ipspace added the enhancement New feature or request label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant