Update Azure.Identity to 1.6.0.

[AraHaan]

Azure.Identity's latest version is 1.6.0 but there is plans for some work for it to remove some more dependencies with it adding an .NET 6 TFM for it so it can require less packages which would be great for everyone using SqlClient directly or indirectly from efcore.

[JRahnama]

@AraHaan This is in our radar. We have recently updated the version, but the latest version will be used when Net6 support is added.

[AraHaan]

So I can basically add it in my PR that basically removes the needless dependencies.

Speaking of that PR, VS seems to be having issues with it as well for some reason.

[JRahnama]

we will do some testing and if everything works as expected we will upgrade the version in our next preview release.

[AraHaan]

I experimented with it in my pr to remove some dependencies and it compiles fine with the updated packages. I even pushed them to that PR as well.

Feel free to cherry-pick the dependency update commit from that pr if needed.

[JRahnama]

Can you make a separate PR for that. We try to avoid multiple changes in one PR to make it easier to find the source any regression? or we can do it.

Ignore my comment. I saw that is a separate PR.

[lcheunglci]

@AraHaan already has PR #1536 opened for the Azure Identity upgrade, so I closed my PR opened earlier.

[AraHaan]

I commented on your PR. #1536 had to have the dependency updates temporarily in order to get my local build to pass.

However the updates could be moved from #1536 as they can also be used by all the other TFMs.

[lcheunglci]

I added a comment in the merged PR #1611 , regarding the Microsoft.IdentityModel.* dependency i.e. upgrading from 6.8.0 to 6.18.0, where it succeeded in the netcore project but failed on netfx project because the nuget package Microsoft.IdentityModel.Abstractions doesn't exist for netfx from the solution/project's nuget repo: https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json and the solution to fix it was to add the repo https://api.nuget.org/v3/index.json to it. However, I'm not sure if there was a reason behind using the azure repo vs the official nuget repo, so perhaps, one of the maintainers could chime in.

[JRahnama]

@lcheunglci and @AraHaan the issue with the feed should be fixed now. It is added to the public feed.

[JRahnama]

Closing this as done in PR #1611

[AraHaan]

@JRahnama, @lcheunglci was the followup pr created for the other Microsoft.IdentityModel.* dependencies?

[lcheunglci]

I have a branch with the change for the upgrade, I'll open a PR shortly.