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

Skip to content

Conversation

@sverker
Copy link
Contributor

@sverker sverker commented Jan 3, 2023

Fixes #6595.

Make OTP-25 accept old send_name handshake messages used by OTP-22 and older. OTP-23 or OTP-24 nodes, not using epmd, may send the old message to us if they do not know our OTP version.
Hence, we accept the old send_name and the accompanying send_complement, but we do not send them ourself.

This was removed prematurely in OTP-25.0 by commit 722219f.
Therefore versions 25.0 to 25.2.* are broken in this regard and may reject connection setup from OTP-23 and 24 not using epmd.

@sverker sverker added team:VM Assigned to OTP team VM fix labels Jan 3, 2023
@sverker sverker self-assigned this Jan 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2023

CT Test Results

       6 files     192 suites   1h 38m 12s ⏱️
2 924 tests 2 647 ✔️ 276 💤 1
3 436 runs  3 109 ✔️ 326 💤 1

For more details on these failures, see this check.

Results for commit 2e9b56d.

♻️ 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

@sverker sverker added the testing currently being tested, tag is used by OTP internal CI label Jan 10, 2023
We accept old 'send_name' handshake messages used by OTP-22 and older.
OTP-23 or OTP-24 nodes, not using epmd, may send the old message
to us if they do not know our OTP version.
Hence, we accept the old 'send_name' and the accompanying
'send_complement', but we do not send them ourself.

This was removed prematurely in OTP-25.0 by
commit 722219f.
Therefore versions 25.0 to 25.2.* are broken in this regard and may
reject connection setup from OTP-23 and 24 not using epmd.
@sverker sverker force-pushed the sverker/kernel/accept-ver5-connection-setup branch from fd94665 to 2e9b56d Compare January 17, 2023 20:06
@sverker sverker merged commit 1976254 into erlang:maint Jan 18, 2023
@sverker sverker deleted the sverker/kernel/accept-ver5-connection-setup branch November 4, 2024 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 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.

1 participant