Skip to content

HCK-16220: Fix Oracle TNS/mTLS connections and RE error handling#252

Merged
bigorn0 merged 7 commits into
developfrom
HCK-16220/fix-tns-conenction
May 28, 2026
Merged

HCK-16220: Fix Oracle TNS/mTLS connections and RE error handling#252
bigorn0 merged 7 commits into
developfrom
HCK-16220/fix-tns-conenction

Conversation

@bigorn0
Copy link
Copy Markdown
Contributor

@bigorn0 bigorn0 commented May 22, 2026

Jira: https://hackolade.atlassian.net/browse/HCK-16220

Technical doc: https://app.gitbook.com/o/HBtg1gLTy0nw4NaX0MaV/s/k3cNzEeXGnIxGNnOQdDa/targets/rdbms/oracle/oracle-connection-modes

Content

Fixes Oracle connection behavior for TNS, Cloud Wallet, and Basic profiles, and improves reverse-engineering error reporting when a connection fails.

  • TNS / mTLS: Wallet credentials are sent only when mutual TLS is explicitly enabled (or Cloud Wallet is used). Legacy TNS profiles without mutualTLS or walletPassword keep working with a tnsnames.ora directory or file path.
  • TNS parsing: Lowercase keys in parsed tnsnames.ora data so address/service fields resolve reliably.
  • Connection modal: Optional TNS alias; mTLS checkbox and wallet password ordered before alias; clearer help text.
  • DNS is checked before connect so stopped Azure VMs (hostname resolving to 255.255.255.255) fail with a clear message instead of NJS-503 to a misleading IP.

Jira: HCK-16220

image image

bigorn0 and others added 4 commits May 22, 2026 13:48
Co-authored-by: Cursor <cursoragent@cursor.com>
Gate wallet usage on explicit mTLS while preserving legacy TNS profiles, normalize connect strings, pre-check DNS for unusable Azure hostnames, and propagate connection failures instead of empty schema lists.

Co-authored-by: Cursor <cursoragent@cursor.com>
Extract TNS, wallet, SSH, and connect-string helpers so connect stays within Sonar limits.

Co-authored-by: Cursor <cursoragent@cursor.com>
@bigorn0 bigorn0 marked this pull request as ready for review May 26, 2026 13:09
bigorn0 and others added 3 commits May 26, 2026 15:22
Co-authored-by: Cursor <cursoragent@cursor.com>
Move tnsnames resolution, mTLS checks, and connect-string setup into focused modules without behavior changes.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud
Copy link
Copy Markdown

@bigorn0 bigorn0 merged commit 176943d into develop May 28, 2026
7 checks passed
@bigorn0 bigorn0 deleted the HCK-16220/fix-tns-conenction branch May 28, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant