fix: aspnetcore dependency

[bruno-garcia]

On main we don't have any dependencies, so this was introduced on net8.0:

sentry-dotnet/src/Sentry.AspNetCore/Sentry.AspNetCore.csproj

Line 28 in 4d239a5

<FrameworkReference Include="Microsoft.AspNetCore.App" />

If that's the case, we don't need to add when running on net6.0/7.0.

If not, we should define the lowest version per TFM

#skip-changelog

[bruno-garcia]

If CI works without this, just delete it:

Suggested change

	<!-- <ItemGroup Condition="'$(TargetFramework)' == 'net6.0'">
	<PackageReference Include="Microsoft.Extensions.Configuration.Binder" Version="6.0.0" />
	</ItemGroup> -->
	<!-- <ItemGroup Condition="'$(TargetFramework)' == 'net6.0'">
	<PackageReference Include="Microsoft.Extensions.Configuration.Binder" Version="6.0.0" />
	</ItemGroup> -->

[jamescrosswell]

Hm, very mysterious. I am wondering how this code can possibly work in CI without that, as it depends on the source generated configuration bindings from Microsoft.Extensions.Configuration.Binder:

sentry-dotnet/src/Sentry.AspNetCore/SentryAspNetCoreOptionsSetup.cs

Line 70 in 89301cb

_config.Bind(bindable);

They are only used at design time to generate the *.g.cs classes I guess... still doesn't make sense to me though.

[bruno-garcia]

.NET 6 prob used reflection, and .NET 8 added source generator (get make this trimmable/AOT friendly)

[jamescrosswell]

That particular method (Bind) isnt' available via reflection (it's a generated method) and the codepath that uses it is when !NETSTANDARD2_0 so it still doesn't compute for me 🤷🏻‍♂️