[migration] Migrate project to SDK 10 and fix deprecations#284
Open
marwannettour wants to merge 6 commits intomasterfrom
Open
[migration] Migrate project to SDK 10 and fix deprecations#284marwannettour wants to merge 6 commits intomasterfrom
marwannettour wants to merge 6 commits intomasterfrom
Conversation
…onTests/ByteSync.Client.IntegrationTests.csproj)
… is correctly executed
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Description
Upgrades the entire ByteSync solution from .NET 8 to .NET 10, bringing in the latest runtime features, performance improvements, and long-term support.
All four library/application projects and all six test projects are updated in a single atomic change so the solution remains consistently targeted.
🛠️ Changes Made
Target framework bump (
net8.0→net10.0)ByteSync.ClientByteSync.CommonByteSync.FunctionsByteSync.ServerCommonByteSync.Client.UnitTestsByteSync.Client.IntegrationTestsByteSync.Common.TestsByteSync.Functions.UnitTestsByteSync.Functions.IntegrationTestsByteSync.ServerCommon.TestsByteSync.TestsCommonNuGet package upgrades
All Microsoft packages that are versioned alongside the runtime were updated to their .NET 10 equivalents:
Microsoft.AspNetCore.Authentication.JwtBearerMicrosoft.AspNetCore.SignalR.ClientMicrosoft.Extensions.Configuration/.JsonMicrosoft.Extensions.DependencyInjectionMicrosoft.Extensions.HttpMicrosoft.Extensions.Logging.AbstractionsMicrosoft.Azure.Functions.WorkerMicrosoft.Azure.Functions.Worker.SdkMicrosoft.Azure.Functions.Worker.ApplicationInsightsMicrosoft.Azure.AppConfiguration.AspNetCoreMicrosoft.Azure.SignalR/.ManagementBreaking change adaptations (Azure Functions Worker v2 API)
The
WriteAsJsonAsyncoverload that accepted an explicitHttpStatusCodeargument was removed in the Azure Functions Worker v2 SDK. All affected HTTP function handlers were updated to the new pattern: set the status code on the response object separately, then callWriteAsJsonAsyncwithout the status parameter.Files updated:
src/ByteSync.Functions/Http/AuthFunction.cssrc/ByteSync.Functions/Http/AnnouncementFunction.cssrc/ByteSync.Functions/Http/CloudSessionFunction.cssrc/ByteSync.Functions/Http/CloudSessionProfileFunction.cssrc/ByteSync.Functions/Http/FileTransferFunction.cssrc/ByteSync.Functions/Http/InventoryFunction.cssrc/ByteSync.Functions/Http/LobbyFunction.cssrc/ByteSync.Functions/Http/SessionMemberFunction.cssrc/ByteSync.Functions/Http/TrustFunction.cssrc/ByteSync.Functions/Helpers/Middlewares/JwtMiddleware.cs🧪 How to Test?
📌 Additional Notes
Microsoft.Extensions.Http.Pollypackage (8.0.22) was intentionally left at its current version as no .NET 10-specific release is yet available; it remains compatible.2.xSDK and Worker packages.