Skip to content

Conversation

MattB-msft
Copy link
Member

@MattB-msft MattB-msft commented Feb 6, 2025

RequestId on OrganizationRequest is not overridden by random Guid if present. RequestId from ClientRequestBuilder still takes precedence over OrganizationRequest.RequestId
Dependency changes:
Microsoft.Identity.Client updated to 4.66.1
System.Text.Json updated to 8.0.5
Azure.Identity updated to 1.31.1

Dependency Updates:

  • Updated System.Formats.Asn1 package version to be controlled by PackageVersion_System_Formats_Asn1 in src/GeneralTools/DataverseClient/Client/Microsoft.PowerPlatform.Dataverse.Client.csproj and src/GeneralTools/DataverseClient/Extensions/DynamicsExtension/Microsoft.PowerPlatform.Dataverse.Client.Dynamics.csproj [1] [2].
  • Added System.Text.Json package reference in src/GeneralTools/DataverseClient/Extensions/Microsoft.PowerPlatform.Dataverse.ServiceClientConverter/Microsoft.PowerPlatform.Dataverse.ServiceClientConverter.csproj.
  • Updated multiple package versions in src/Packages.props, including MSAL, CdsSdk, CrmProxy, SystemTextJson, and Azure_Identity [1] [2].

Code Improvements:

  • Improved handling of RequestId in internal OrganizationRequest BuildRequest(OrganizationRequest request) in src/GeneralTools/DataverseClient/Client/Builder/AbstractClientRequestBuilder.cs to avoid overriding RequestId if it is already present.

Release Notes:

  • Updated release notes in src/nuspecs/Microsoft.PowerPlatform.Dataverse.Client.AzAuth.ReleaseNotes.txt to reflect dependency changes including Azure.Identity update.
  • Updated release notes in src/nuspecs/Microsoft.PowerPlatform.Dataverse.Client.Dynamics.ReleaseNotes.txt to remove support for .NET 3.1 and .NET 5.0.
  • Updated release notes in src/nuspecs/Microsoft.PowerPlatform.Dataverse.Client.ReleaseNotes.txt to document changes in RequestId handling and multiple dependency updates.

@MattB-msft MattB-msft merged commit dab25f4 into master Feb 6, 2025
1 check passed
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.

1 participant