Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor SQLServerColumnEncryptionAzureKeyVaultProvider usage in SQLServerConnection #1774

Merged
merged 1 commit into from
Mar 31, 2022
Merged

Refactor SQLServerColumnEncryptionAzureKeyVaultProvider usage in SQLServerConnection #1774

merged 1 commit into from
Mar 31, 2022

Conversation

gastaldi
Copy link
Contributor

This refactors the usage of SQLServerColumnEncryptionAzureKeyVaultProvider into private methods.

This is useful for the native-image compilation GraalVM's optimizations, allowing these methods to be dynamically substituted when the Azure classes are not in the classpath

See quarkusio/quarkus#24010 for more information

…erverConnection

This refactors the usage of SQLServerColumnEncryptionAzureKeyVaultProvider into private methods.
This is useful for the native-image compilation GraalVM's optimizations, allowing these methods to be dynamically substituted when the Azure classes are not in the classpath

See quarkusio/quarkus#24010 for more information
@lilgreenbird
Copy link
Member

hi @gastaldi

thank you for this the team will take a look.

@lilgreenbird lilgreenbird added this to Under Investigation in MSSQL JDBC via automation Mar 26, 2022
@lilgreenbird lilgreenbird moved this from Under Investigation to Under Peer Review in MSSQL JDBC Mar 26, 2022
@lilgreenbird lilgreenbird added this to the 11.1.0 milestone Mar 29, 2022
@lilgreenbird lilgreenbird merged commit 8e66ac0 into microsoft:main Mar 31, 2022
MSSQL JDBC automation moved this from Under Peer Review to Closed/Merged PRs Mar 31, 2022
@gastaldi gastaldi deleted the refactor_to_graalvm branch March 31, 2022 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
MSSQL JDBC
  
Closed/Merged PRs
Development

Successfully merging this pull request may close these issues.

None yet

5 participants