@ChALkeR 포인터 주셔서 감사합니다. 이것은 우리가 이 문제에 접근할 때 도움이 될 것입니다. 영감을 받았다면 자유롭게 PR을 보내주세요 :-)

@tvrprasad : @ChALkeR 는 Node.js 팀의 일원이며 이 게시물은 대부분 이 기능이 더 이상 사용되지 않는다는 사실을 알리기 위한 것입니다. 나는 그가 실제로 tedious 사용한다고 생각하지 않습니다. 😄

@ChALkeR : 이 지원 중단에 대해 잠시 알고 있었지만 이 지원 중단에 대한 문서는 정말 희박합니다. 내가 지금까지 본 예제(예: node-mysql 대한 PR)는 연결이 비 TLS에서 TLS로 전환되는 "단순한" TLS 업그레이드에 중점을 둡니다.

TDS에서는 TLS 업그레이드가 훨씬 더 복잡하고 모든 핸드셰이킹 패킷을 TDS 메시지로 래핑해야 하며 연결이 설정되면 더 이상 모든 TLS 패킷을 래핑할 필요가 없습니다. tls.createSecurePair 를 통해 이것을 구현하는 것은 매우 간단했지만 TLSSocket 를 통해 이것을 마지막으로 시도했을 때 작동하지 못했습니다. 😞

시간이 되면 다시 해봐야 겠지만 크게 달라진 점은 없는 것 같아요.

내가 우연히 또 다른 문제는에서 소켓 포장이다 readable-stream 로 패키지를 TLSSocket 세그먼트 폴트 (!)가 Node.js를 프로세스의 원인이됩니다. 불행히도 지금 당장은 재생산 단계가 없습니다. 😞

@arthurschreiber 정보 감사합니다 :-)

그러나 마지막으로 TLSSocket을 통해 이것을 시도했을 때 작동하지 못했습니다.

이 작업이 github에 있다면 포인터를 보내주세요. 궁금해서 확인합니다.

@tvrprasad @arthurschreiber tls.createSecurePair 를 대체하는 방법에 대한 정보입니다. https://nodejs.org/dist/latest-v6.x/docs/api/tls.html#tls_class_securepair

Node v8.0.1에서 동일한 문제가 발생했습니다. 내 앱은 암호화된 연결을 설정하려고 시도한 후 다음 오류를 보고합니다.

(node:6344) [DEP0064] DeprecationWarning: tls.createSecurePair() is deprecated. Please use tls.Socket instead.

저는 patriksimek/node-mssql 및 tedious 드라이버( node-mssql 의 기본 드라이버)를 사용하여 Microsoft SQL Server 데이터베이스에 액세스하고 있습니다.

지루한 드라이버와 함께 node-mssql을 사용하는 @olange 와 동일한 문제입니다.

Listening on port 4000
(node:4402) [DEP0064] DeprecationWarning: tls.createSecurePair() is deprecated. Please use tls.Socket instead.

저를 오해하지 마세요. 이 모듈을 계속 유지하는 당신의 노력에 정말 감사드립니다. 하지만 언제 수정될지 예측할 수 있습니까?

패키지 'mssql' 사용 및 옵션 사용:
{
암호화: 참
}

(node:13508) [DEP0064] DeprecationWarning: tls.createSecurePair()는 더 이상 사용되지 않습니다. 대신 tls.Socket을 사용하십시오.

Node 팀이 이와 같은 것을 더 이상 사용하지 않을 때 Node에서 제거하기 위한 기간은 어떻게 됩니까? 그냥 궁금합니다.

아마도 노드 9 또는 10에서 제거될 것입니다. 이 기능은 최소 4년 동안 문서에서 더 이상 사용되지 않습니다. :-/ 이것은 여전히 ​​그것을 사용하는 유일한 인기 있는 NPM 패키지 중 하나입니다.

공동 작업자 중 변경 사항이 출시될 일정이 있는 사람이 있습니까?

방금 시스템을 노드 8.9.0 lts로 업데이트했기 때문에 이것을 얻었습니다. 지금 두 개의 무거운 프로젝트가 있지만 둘 다 완화되면 이 문제를 해결하는 데 시간을 할애할 수 있는지 알아보겠습니다.

@Iiridayn 굉장
나는 그것을 보았지만 TLSSocket에서 핸드 셰이크 패킷을 얻는 방법을 알 수 없었습니다. https://github.com/tediousjs/tedious/issues/515#issuecomment -283191248 :confused:
그것에 대한 아이디어가 있다면 듣고 싶습니다.

이에 대한 진전이 있습니까? 도움이 되는 경우에만 Azure에 연결하려고 하는 것과 동일한 문제에 직면하고 있습니다.
누군가가 이것이 더 이상 사용되지 않는 동안 앱을 실행한 결과가 무엇인지 신속하게 설명하고 내가 요청한 SQL 쿼리가 실행되지 않고 이것이 원인인지 알아야 하지만 이에 대한 텍스트를 알려 주시면 감사하겠습니다. ve read는 그다지 웅변적이지 않습니다.

미리 감사드립니다.

@Jorgeee1 이것은 경고일 뿐, 문제의 원인은 아닐 것입니다. Azure DB에 연결하는 데도 지루한 것을 사용하고 있으며 잘 작동합니다.

createSecurePair 를 TLSSocket + TCP 서버로 대체하여 원시 핸드셰이크 데이터를 얻는 개념의 빠른 증명을 작성했습니다. 여기에서 살펴볼 수 있습니다 . 실행할 수 있는 통합 테스트를 통과했습니다.

이것은 분명히 당면한 문제에 대한 실제 솔루션이 아닙니다(추가 소켓 등을 먹는 것이 좋지 않은 경우). 이것은 토론을 시작하는 것 이상입니다. https://github.com/nodejs/help/issues/1010 에서 실제 제안과 토론이 있으면 좋을 것

@joux3 굉장하네요 😃 PR 만들어주실 수 있나요? 그곳에서 검토하고 논의하는 것이 더 쉬울 것입니다.

무거운 프로젝트는 완화되지 않았고 세 번째가 추가되었으며 MySQL도 마이그레이션했습니다. 죄송합니다. 이 문제를 해결할 수 없습니다. 행운을 빕니다 :).

고쳐주세요 중요한 문제입니다

여러분,

여기에서 현재 상태에 대한 빠른 업데이트를 제공하겠습니다. 지원 중단 메시지가 귀찮을 수 있다는 것을 이해하지만 실제로 문제가되지는 않습니다. tls.createSecurePair 는 더 이상 사용되지 않을 수 있지만 단기 및 중기 미래에는 아무데도 가지 않을 것입니다. https://github.com/nodejs/node/pull/17882 에 따르면 곧 제거되지 않습니다.

또한 https://github.com/nodejs/node/pull/17882 는 공개 node.js API 측면에서 tls.createSecurePair 의 재구현을 제안하여 (현재) 내부 DuplexSocket 도입합니다 tls.createSecurePair 에서 벗어나는 데 필요한 것입니다. 지금은 우리만의 DuplexSocket 버전을 만들어야 하지만 적어도 tls.createSecurePair 가 실제로 제거되는 시점이 명확해지면 이 문제를 지루하게 해결하는 방법에 대한 좋은 아이디어가 있습니다. .

당신의 생각을 알려주세요! 😄

@arthurschreiber Fwiw, 저도 방금 DuplexPair 구현을 https://www.npmjs.com/package/duplexpair에 게시했습니다 tls.createSecurePair() 유사하게 만드는 래퍼를 꽤 쉽게 만들 수 있다고 생각합니다.

@adddaleax 스윗! ❤️ 이것은 훌륭하며 나중에 연결하여 우리를 tls.createSecurePair 빨리 이동할 수 있는지 확인하겠습니다! 정말 고맙습니다! 🙇

@arthurschreiber 이 패치가 다음과 같이 간단해 보일 수 있다고 생각합니다.

diff --git a/package.json b/package.json
index bca5b266849d..8cc0ed957577 100644
--- a/package.json
+++ b/package.json
@@ -43,6 +43,7 @@
     "babel-runtime": "^6.26.0",
     "big-number": "0.3.1",
     "bl": "^1.2.0",
+    "duplexpair": "^1.0.1",
     "iconv-lite": "^0.4.11",
     "readable-stream": "^2.2.6",
     "sprintf": "0.1.5"
diff --git a/src/message-io.js b/src/message-io.js
index 6c2a15c32a9f..a1de3a1e91c0 100644
--- a/src/message-io.js
+++ b/src/message-io.js
@@ -1,7 +1,9 @@
+'use strict';
 const tls = require('tls');
 const crypto = require('crypto');
 const EventEmitter = require('events').EventEmitter;
 const Transform = require('readable-stream').Transform;
+const DuplexPair = require('duplexpair');

 const Packet = require('./packet').Packet;
 const TYPE = require('./packet').TYPE;
@@ -84,10 +86,14 @@ module.exports = class MessageIO extends EventEmitter {
   startTls(credentialsDetails, hostname, trustServerCertificate) {
     const credentials = tls.createSecureContext ? tls.createSecureContext(credentialsDetails) : crypto.createCredentials(credentialsDetails);

-    this.securePair = tls.createSecurePair(credentials);
+    const duplexpair = new DuplexPair();
+    this.securePair = {
+      cleartext: new tls.TLSSocket(duplexpair.socket1),
+      encrypted: duplexpair.socket2
+    };
     this.tlsNegotiationComplete = false;

-    this.securePair.on('secure', () => {
+    this.securePair.cleartext.on('secure', () => {
       const cipher = this.securePair.cleartext.getCipher();

       if (!trustServerCertificate) {

그러나 이것은 일반 테스트 모음의 일부로 테스트되지 않은 것 같으므로 어떻게 해야 할지 잘 모르겠습니다...

나는 SecurePair를 교체하는 또 다른 기회를 주었다. Node 9에서 테스트한 통합 테스트가 괜찮은 것 같습니다. https://github.com/joux3/tedious/commit/6d255fbdaa2f9afd388617e6e34815cae95292a0

그러나 몇 가지 관찰 사항이 있습니다(하지만 패치가 PR이 문제를 논의하기에 충분할 수 있습니까?).

• Node.js 6은 Assertion failed: (uv__stream_fd(stream) >= 0), function uv_read_start, file ../deps/uv/src/unix/stream.c, line 1517. 에서 완전히 충돌하는 것 같습니다.
• 재시도 시간 초과 테스트에서 가짜 타이머를 제거해야 했습니다. 가짜 타이머는 내부적으로 DuplexPair를 중단시키는 것 같습니다.
• 암호화되지 않은 연결은 테스트하지 않았습니다. 그것들은 논리 흐름 변경의 영향을 받을 수 있습니다(신호가 파이프를 통해 올바르게 전파되지 않는 것 같아서 소켓에 수동 close 핸들러가 있음)'

작업에 대해 @addaleax 에게 큰 감사를 드립니다!

@joux3 포크는 현재 LTS 릴리스인 노드 8에서 사용할 준비가 되었습니까? 풀 리퀘스트를 여는 것이 가능합니까? https://github.com/patriksimek/node-mssql/ 과 함께 사용하는 문제가 해결되는지 확인하고 싶습니다. 감사합니다!

재시도 시간 초과 테스트에서 가짜 타이머를 제거해야 했습니다. 가짜 타이머는 내부적으로 DuplexPair를 중단시키는 것 같습니다.

@joux3 어떤 테스트가 될까요? (죄송합니다. 저는 이 문제에 대해 살펴본 것 외에 이 코드 기반에 대해 특별히 익숙하지 않습니다.)

@addaleax , 실패한 테스트는 https://github.com/tediousjs/tedious/blob/f5a0f95a5a522ad6a312170201b829e70c866976/test/integration/connection-retry-test.js#L73 -L104입니다.
타이머 비활성화: https://github.com/tediousjs/tedious/pull/689/files#diff -ed1dc3ba8a10a8fb4b18a528b35bad00.
편집: setTimeout https://github.com/tediousjs/tedious/pull/689/commits/ff5d5eb1a3603db05266306a823fa49a98b50209 만 가짜 타이머 가짜를 복원했습니다

@jimmont , https://github.com/tediousjs/tedious/pull/689 에서 pull request를 열었습니다(PR 이름에 문제 ID를 참조하는 것만으로는 충분하지 않은 것 같습니다...). 통합 테스트는 노드 8을 통과했지만 아직 해야 할 일이 있습니다.

👍 이 문제를 최대한 빨리 해결하기 위해!

이것에 진전이 있습니까? 이 문제에 대한 업데이트를 얻을 수 있다면 좋을 것입니다.

추적을 위해 PR #689를 연결합니다.

경고가 stdout 대신 stderr로 전송되는 세상에서. <fail/>

모든 업데이트?

738에는 수정 사항이 포함되어 있으며 @tediousjs/microsoft-contributors 팀의 누군가가 검토한 후 병합 및 릴리스됩니다.

오랜 시간이 지난 후, 이것은 https://github.com/tediousjs/tedious/pull/738을 통해 마침내 (그리고 적절하게) 수정되었습니다 [email protected] 은(는) 이제 npm에서 사용할 수 있습니다. 현재 next 로 태그가 지정되어 있으며 곧 latest 로 태그가 지정되도록 하겠습니다.

인내심을 갖고 이 문제를 해결하는 데 도움을 주신 모든 분들께 감사드립니다! 🙇