Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
From denisenkom#533
We are adding a plugin to the
secretless-broker
project (https://github.com/cyberark/secretless-broker) and will be using this project to communicate with SQL Server. Secretless is written in go too and needs a bit more access to some of the internals of the project. In this PR we expose the constants & functions we need to use. Other than that, the logic stays the same.The Secretless Broker lets your applications connect securely to services - without ever having to fetch or manage passwords or keys. It does so by acting as a connector between the application and the target, retrieving the credentials for authentication from a credentials provider. This way, it opens an authenticated stream between the application and the target, so the application doesn't need to manage credentials in its code. To do so with mssql, we need to get the underlying channel and and the credentials to the message. We needed access to some constants and structs to do so.
Note: It would be great if these minor changes will be merged to the original go-mssqldb project so we can consume it as a 3rd party to our project, rather than relying on our own fork - but for now we are putting this in our own fork.