Skip to content

Commit 7aebacf

Browse files
committed
Fix bug to enable SSL_SERVER_DN_MATCH set in UserConfig object to carry through when it is not set in einput easy connect string
1 parent 90428a4 commit 7aebacf

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

doc/src/release_notes.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ Common Changes
3030

3131
Thin Mode Changes
3232
+++++++++++++++++
33+
#) Fixed bug when SSL_SERVER_DN_MATCH set in UserConfig does not carry
34+
through when the input connect string is in easy connect format
35+
and SSL_SERVER_DN_MATCH value is not set in that connect string.
3336

3437
#) Add support to improve Connection Establishment Latency for DB
3538
versions 23.3 onwards.

lib/thin/sqlnet/ezConnectResolver.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -318,10 +318,11 @@ class EZConnectResolver {
318318
if (protocol != null && protocol.toLowerCase() == "tcps") {
319319
// In EZConnect format if the DN match is not specified the enable it
320320
// by default for TCPS protocol.
321-
const serverDNMatch = this.urlProps.get("SSL_SERVER_DN_MATCH") || 'TRUE';
321+
const serverDNMatch = this.urlProps.get("SSL_SERVER_DN_MATCH");
322322
const serverCertDN = this.urlProps.get("SSL_SERVER_CERT_DN");
323323
const walletDir = this.urlProps.get("MY_WALLET_DIRECTORY");
324-
securityInfo.push(`(SSL_SERVER_DN_MATCH=${serverDNMatch})`);
324+
if (serverDNMatch != null)
325+
securityInfo.push(`(SSL_SERVER_DN_MATCH=${serverDNMatch})`);
325326
if (serverCertDN != null)
326327
securityInfo.push(`(SSL_SERVER_CERT_DN=${serverCertDN}})`);
327328
if (walletDir != null)

0 commit comments

Comments
 (0)