Skip to content

Conversation

@virzak
Copy link

@virzak virzak commented Mar 1, 2024

This requires less effort for the user since they can simply reference a nuget package.

If you like the approach we can remove the nuget.config file and use nuget instead of myget.

@0xced
Copy link
Owner

0xced commented Jun 18, 2025

Hey Viktor, sorry I completely forgot about this pull request!

Note that I have now released Chisel. Getting rid of the Azure authentication libraries from Microsoft.Data.SqlClient is now really easy. And transitive dependencies are properly handled!

<ItemGroup>
  <ChiselPackage Include="Azure.Identity" />
  <ChiselPackage Include="Microsoft.IdentityModel.JsonWebTokens" />
  <ChiselPackage Include="Microsoft.IdentityModel.Protocols.OpenIdConnect" />
</ItemGroup>

<ItemGroup>
  <PackageReference Include="Chisel" Version="1.1.2" PrivateAssets="all" />
  <PackageReference Include="Microsoft.Data.SqlClient" Version="6.0.2" />
</ItemGroup>

I also updated the README in this repository to mention Chisel.

And thanks to the combination of dotnet/SqlClient#2577 and dotnet/SqlClient#2636, getting rid of Microsoft.Identity.Client doesn't require any special handling anymore. 🥳

@0xced 0xced closed this Jun 18, 2025
@virzak
Copy link
Author

virzak commented Jun 18, 2025

No worries at all.

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.

2 participants