Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Document restrictions on node names#10473

Merged
jhogberg merged 1 commit intoerlang:masterfrom
nabijaczleweli:nodename
Jan 12, 2026
Merged

Document restrictions on node names#10473
jhogberg merged 1 commit intoerlang:masterfrom
nabijaczleweli:nodename

Conversation

@nabijaczleweli
Copy link
Contributor

I just got some errors when automatically deriving the nodename from other configuration and the documentation doesn't say what the restrixion actually is.

lib/kernel/src/net_kernel.erl:

valid_name_head(Head) ->
    {ok, MP} = re:compile("^[0-9A-Za-z_\\-]+$", [unicode]),
        case re:run(Head, MP) of
            {match, _} ->
                true;
            nomatch ->
                false
    end.

lib/kernel/src/net_kernel.erl:
  valid_name_head(Head) ->
      {ok, MP} = re:compile("^[0-9A-Za-z_\\-]+$", [unicode]),
          case re:run(Head, MP) of
              {match, _} ->
                  true;
              nomatch ->
                  false
      end.

Sponsored-by: https://beaverlabs.net
@github-actions
Copy link
Contributor

github-actions bot commented Dec 12, 2025

CT Test Results

  1 files   11 suites   2m 50s ⏱️
 95 tests  91 ✅ 4 💤 0 ❌
111 runs  107 ✅ 4 💤 0 ❌

Results for commit aeadf52.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@rickard-green rickard-green added the team:VM Assigned to OTP team VM label Dec 15, 2025
@jhogberg jhogberg self-assigned this Dec 15, 2025
@jhogberg jhogberg added the testing currently being tested, tag is used by OTP internal CI label Jan 7, 2026
@jhogberg jhogberg merged commit c929d98 into erlang:master Jan 12, 2026
24 of 25 checks passed
@jhogberg
Copy link
Contributor

Merged, thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants