ssl: help message refers to nonexistent function

[lemenkov]

No such function - ssl_handshake. Obviously the intention was to refer to handshake

[github-actions]

CT Test Results

    5 files    202 suites   1h 34m 41s ⏱️
2 374 tests 2 273 ✅ 101 💤 0 ❌
6 331 runs  5 337 ✅ 994 💤 0 ❌

Results for commit 86da67e.

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

• Complete CT logs (Download Logs)
• HTML Documentation (Download HTML Docs)
• Windows Installer

// Erlang/OTP Github Action Bot

[IngelaAndin]

@lemenkov could you please rebase it on maint so that this can be fixed in OTP-27 track too.

[lemenkov]

@IngelaAndin sure will do in a moment

[IngelaAndin]

@lemenkov Thanks, when I was checking this for merge I however realized that the typo was actually that there was an _ instead of a : and that ssl: was missing in the the clause for cipher_suites. We think that including the module always even when it is the same is clearer and the information is sometimes also used in a non module context and then it is really nice that it exists. So could you please make your PR like this instead?

-removed({ssl_accept, '_', 
          "use ssl:handshake/1,2,3 instead"}).
-removed({cipher_suites, 0, 
          "use ssl:cipher_suites/2,3 instead"}).
-removed({cipher_suites, 1, 
          "use ssl:cipher_suites/2,3 instead"}).
-removed([{negotiated_next_protocol,1,
           "use ssl:negotiated_protocol/1 instead"}]).
-removed([{connection_info,1,
           "use ssl:connection_information/[1,2] instead"}]).

Where the three first clauses are the changed ones.

[lemenkov]

Heh, @IngelaAndin you already invested more time on this task than me :)
Sure, I'll fix it in a moment.

[IngelaAndin]

@lemenkov thanks for fixing all the places.