Sslprotocol

[v-nisidh]

Please DO NOT MERGE. This Pull Request is only for review for approach taken for IBM FIPS & SSL Protocol

[v-nisidh]

Please find attach files for test purpose.

Shared_Files.zip

[codecov-io]

Codecov Report

Merging #325 into dev will decrease coverage by 0.11%.
The diff coverage is 52.38%.

@@             Coverage Diff              @@
##                dev     #325      +/-   ##
============================================
- Coverage     37.26%   37.14%   -0.12%     
+ Complexity     1764     1762       -2     
============================================
  Files           107      107              
  Lines         24491    24507      +16     
  Branches       4000     4001       +1     
============================================
- Hits           9127     9104      -23     
- Misses        13653    13698      +45     
+ Partials       1711     1705       -6

Flag	Coverage Δ	Complexity Δ
#JDBC41	37.05% <52.38%> (-0.11%)	1757 <0> (-3)
#JDBC42	37.01% <52.38%> (-0.14%)	1753 <0> (-5)

Impacted Files	Coverage Δ	Complexity Δ
...om/microsoft/sqlserver/jdbc/SQLServerResource.java	100% <ø> (ø)	4 <0> (ø)	⬇️
.../microsoft/sqlserver/jdbc/SQLServerDataSource.java	43.51% <0%> (-3.9%)	59 <0> (-7)
.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java	74.18% <100%> (+0.23%)	25 <0> (ø)	⬇️
.../microsoft/sqlserver/jdbc/SQLServerConnection.java	44.62% <28.57%> (+0.03%)	255 <0> (+1)	⬆️
...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java	43.45% <75%> (-0.87%)	0 <0> (ø)
...java/com/microsoft/sqlserver/jdbc/StringUtils.java	0% <0%> (-60%)	2% <0%> (-2%)
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java	29.24% <0%> (-1.12%)	55% <0%> (-1%)
... and 3 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a12a39...11ea2c7. Read the comment docs.

[v-nisidh]

This pull request consists of adding configurable SSLProtocol and removing FIPS validation.

Please create 2 different pull requests for SSLProtocol & removing FIPS validation

[ulvii]

Hi @v-nisidh/ @nsidhaye

I created a new PR that adds sslProtocol connection property. Planing to create another one for FIPS validation. Please feel free to leave your feedback :)