Skip to content

Access token callback#1940

Merged
tkyc merged 12 commits intomainfrom
access-token-callback
Nov 3, 2022
Merged

Access token callback#1940
tkyc merged 12 commits intomainfrom
access-token-callback

Conversation

@tkyc
Copy link
Copy Markdown
Contributor

@tkyc tkyc commented Oct 26, 2022

Allows client code to register a callback for DataSource that returns an access token.

@tkyc tkyc force-pushed the access-token-callback branch from 1926cbe to c3c3fd6 Compare October 26, 2022 23:40
.format(SQLServerException.getErrString("R_CustomKeyStoreProviderValueNull"), providerName),
null, 0, false);
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything above this comment is just formatting changes.

@lilgreenbird lilgreenbird added this to the 12.1.0 milestone Nov 2, 2022
Comment thread src/main/java/com/microsoft/sqlserver/jdbc/SQLServerConnection.java
Comment thread src/main/java/com/microsoft/sqlserver/jdbc/SqlAuthenticationToken.java Outdated
@Jeffery-Wasty
Copy link
Copy Markdown
Contributor

Multiple comments have leading hyphens, I don't think we do that anywhere. Please remove them for consistency.

Comment thread src/main/java/com/microsoft/sqlserver/jdbc/ISQLServerConnection.java Outdated
Comment thread src/main/java/com/microsoft/sqlserver/jdbc/ISQLServerDataSource.java Outdated
Jeffery-Wasty
Jeffery-Wasty previously approved these changes Nov 3, 2022
lilgreenbird
lilgreenbird previously approved these changes Nov 3, 2022
Jeffery-Wasty
Jeffery-Wasty previously approved these changes Nov 3, 2022
@tkyc tkyc merged commit c79db60 into main Nov 3, 2022
@tkyc tkyc deleted the access-token-callback branch November 3, 2022 22:22
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.

4 participants