Enable User Agent Extension

[samsharma2700]

Description

Part 3 for User Agent Feature Extension work. Following are added in this PR:-

• Enabling the pre-calculated JsonPayload to be added to the FE data.
• Adding case handling logic in case of a server ack.
• Changes to test server to force emit Ack for this feature(testing purposes).
• Two new tests to verify both Acked and Unacked pathways work.

Part 2 : #3451
Part 1 : #3582

Unit tests and functional tests are passing

[Copilot]

Pull Request Overview

This PR implements Part 3 of the User Agent Feature Extension work, adding the ability to send User Agent information in TDS login packets and handle server acknowledgments. The implementation includes TDS protocol support for the new feature extension and test coverage to verify the functionality.

• Adds UserAgentSupport feature ID to TDS protocol and enables sending User Agent JSON payload during login
• Updates both .NET Framework and .NET Core implementations to include User Agent data in feature extension requests
• Implements server acknowledgment handling with appropriate logging when unexpected ACKs are received

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/Microsoft.Data.SqlClient/tests/tools/TDS/TDS/TDSFeatureID.cs	Adds UserAgentSupport feature ID enum value
src/Microsoft.Data.SqlClient/tests/tools/TDS/TDS.Servers/GenericTDSServerSession.cs	Adds IsUserAgentSupportEnabled property to server session
src/Microsoft.Data.SqlClient/tests/tools/TDS/TDS.Servers/GenericTDSServer.cs	Implements User Agent feature extension handling in test server
src/Microsoft.Data.SqlClient/tests/tools/TDS/TDS.EndPoint/ITDSServerSession.cs	Adds User Agent support interface definition
src/Microsoft.Data.SqlClient/tests/FunctionalTests/SqlConnectionBasicTests.cs	Adds tests for User Agent feature extension behavior with and without server ACK
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/TdsParser.cs	Updates TdsLogin to include User Agent payload in feature extension data
src/Microsoft.Data.SqlClient/netfx/src/Microsoft/Data/SqlClient/TdsParser.cs	Implements User Agent feature request writing for .NET Framework
src/Microsoft.Data.SqlClient/netfx/src/Microsoft/Data/SqlClient/SqlInternalConnectionTds.cs	Adds User Agent ACK handling and property name correction
src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/TdsParser.cs	Implements User Agent feature request writing for .NET Core
src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlInternalConnectionTds.cs	Adds User Agent ACK handling and property name correction

[paulmedynski]

A few unused/unnecessary fields to remove, and need to add some checks to the tests.

[paulmedynski]

Still some cleanup to be done - mostly in the tests.

Also, we need to change the USERAGENT feature extension value from 0x0F to 0x10. Please update that value.

[paulmedynski]

Yay - all good!

[benrr101]

Overall, looks pretty ok. Couple things to look out for, but as long as @paulmedynski 's concerns are addressed, I'm ok to move forward.