microsoft / HydraLab

Intelligent cloud testing made easy.
MIT License
1.09k stars 99 forks source link

Fix SQL Server connection with removing username/secret #661

Closed olivershen-wow closed 1 month ago

olivershen-wow commented 2 months ago

Description

Enable Managed Identity authentication on Azure by doing the following:

Linked GitHub issue ID:

Pull Request Checklist

Does this introduce a breaking change?

If this introduces a breaking change for Hydra Lab users, please describe the impact and migration path.

How you tested it

Please make sure the change is tested, you can test it by adding UTs, do local test and share the screenshots, etc. Tested locally with local account, allocating the same permissions in DB and run the app without error.

Please check the type of change your PR introduces:

Feature UI screenshots or Technical design diagrams

If this is a relatively large or complex change, kick it off by drawing the tech design with PlantUML and explaining why you chose the solution you did and what alternatives you considered, etc...