Skip to content

Fixed ActiveDirectoryInteractive authentication timeout issue#1696

Merged
lilgreenbird merged 3 commits intomicrosoft:mainfrom
lilgreenbird:interact
Nov 25, 2021
Merged

Fixed ActiveDirectoryInteractive authentication timeout issue#1696
lilgreenbird merged 3 commits intomicrosoft:mainfrom
lilgreenbird:interact

Conversation

@lilgreenbird
Copy link
Copy Markdown
Contributor

Use 10 * loginTimeout timeout value which is consistent with SqlClient behaviour

@lilgreenbird lilgreenbird added this to the 10.1.0 Preview milestone Nov 24, 2021
Comment thread src/main/java/com/microsoft/sqlserver/jdbc/SQLServerConnection.java Outdated
@tkyc
Copy link
Copy Markdown
Contributor

tkyc commented Nov 24, 2021

Was just reading the files, and noticed this alot in project. So the convention for null checks is "null == someVar"? And not the other way around? eg. "someVar == null" or "someVar != null"

@lilgreenbird
Copy link
Copy Markdown
Contributor Author

/azp run CI-MacOS

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lilgreenbird
Copy link
Copy Markdown
Contributor Author

/azp run CI-MacOS

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lilgreenbird
Copy link
Copy Markdown
Contributor Author

/azp run CI-MacOS

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lilgreenbird lilgreenbird merged commit 6ca48b3 into microsoft:main Nov 25, 2021
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.

3 participants