Skip to content

Commit b913f6e

Browse files
authored
tls: move _tls_* to end-of-life
1 parent 4e1f39b commit b913f6e

11 files changed

+6
-55
lines changed

β€Ždoc/api/deprecations.mdβ€Ž

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4179,6 +4179,9 @@ npx codemod@latest @nodejs/repl-builtin-modules
41794179

41804180
<!-- YAML
41814181
changes:
4182+
- version: REPLACEME
4183+
pr-url: https://github.com/nodejs/node/pull/60658
4184+
description: End-of-Life.
41824185
- version:
41834186
- v24.2.0
41844187
- v22.17.0
@@ -4187,7 +4190,7 @@ changes:
41874190
description: Runtime deprecation.
41884191
-->
41894192

4190-
Type: Runtime
4193+
Type: End-of-Life
41914194

41924195
The `node:_tls_common` and `node:_tls_wrap` modules are deprecated as they should be considered
41934196
an internal nodejs implementation rather than a public facing API, use `node:tls` instead.

β€Žlib/_tls_common.jsβ€Ž

Lines changed: 0 additions & 10 deletions
This file was deleted.

β€Žlib/_tls_wrap.jsβ€Ž

Lines changed: 0 additions & 11 deletions
This file was deleted.

β€Žlib/eslint.config_partial.mjsβ€Ž

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,6 @@ export default [
531531
{
532532
files: [
533533
'lib/_http_*.js',
534-
'lib/_tls_*.js',
535534
'lib/http.js',
536535
'lib/http2.js',
537536
'lib/internal/http.js',

β€Žlib/internal/main/mksnapshot.jsβ€Ž

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,6 @@ const supportedModules = new SafeSet(new SafeArrayIterator([
4848
'_stream_transform',
4949
'_stream_wrap',
5050
'_stream_writable',
51-
// '_tls_common',
52-
// '_tls_wrap',
5351
'assert',
5452
'assert/strict',
5553
// 'async_hooks',

β€Žsrc/node_builtins.ccβ€Ž

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,8 @@ BuiltinLoader::BuiltinCategories BuiltinLoader::GetBuiltinCategories() const {
142142
#endif // !NODE_USE_V8_PLATFORM || !defined(NODE_HAVE_I18N_SUPPORT)
143143

144144
#if !HAVE_OPENSSL
145-
"crypto", "crypto/promises", "https", "http2", "tls", "_tls_common",
146-
"_tls_wrap", "internal/tls/parse-cert-string", "internal/tls/common",
145+
"crypto", "crypto/promises", "https", "http2", "tls",
146+
"internal/tls/parse-cert-string", "internal/tls/common",
147147
"internal/tls/wrap", "internal/tls/secure-context",
148148
"internal/http2/core", "internal/http2/compat",
149149
"internal/streams/lazy_transform",

β€Žtest/parallel/test-internal-module-require.jsβ€Ž

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,6 @@ const expectedPublicModules = new Set([
3030
'_stream_transform',
3131
'_stream_wrap',
3232
'_stream_writable',
33-
'_tls_common',
34-
'_tls_wrap',
3533
'assert',
3634
'async_hooks',
3735
'buffer',

β€Žtest/parallel/test-process-get-builtin.mjsβ€Ž

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,6 @@ if (!isMainThread) {
2525
if (!hasCrypto) {
2626
publicBuiltins.delete('crypto');
2727
publicBuiltins.delete('tls');
28-
publicBuiltins.delete('_tls_common');
29-
publicBuiltins.delete('_tls_wrap');
3028
publicBuiltins.delete('http2');
3129
publicBuiltins.delete('https');
3230
publicBuiltins.delete('inspector');

β€Žtest/parallel/test-warn-tls-common-deprecation.jsβ€Ž

Lines changed: 0 additions & 11 deletions
This file was deleted.

β€Žtest/parallel/test-warn-tls-wrap-deprecation.jsβ€Ž

Lines changed: 0 additions & 11 deletions
This file was deleted.

0 commit comments

Comments
Β (0)