Skip to content

[Internal]Thin Client Integration: Adds containerId to ThinClient request.#5109

Merged
kirankumarkolli merged 2 commits intomasterfrom
users/aavasthy/updateheaders
Apr 5, 2025
Merged

[Internal]Thin Client Integration: Adds containerId to ThinClient request.#5109
kirankumarkolli merged 2 commits intomasterfrom
users/aavasthy/updateheaders

Conversation

@aavasthy
Copy link
Copy Markdown
Contributor

@aavasthy aavasthy commented Apr 4, 2025

Pull Request Template

Description

Adding ResourceId as the proxy expects the ResourceId in the RNTBD headers otherwise the requests will start to fail.
By seting request.ResourceId = containerRid, TransportSerialization.cs will automatically add the ResourceID header.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Closing issues

To automatically close an issue: closes #5107

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good!

@aavasthy aavasthy changed the title [Internal]Thin Client Integration: Add containerId to ThinClient request [Internal]Thin Client Integration: Adds containerId to ThinClient request. Apr 4, 2025
@kirankumarkolli kirankumarkolli merged commit 187e164 into master Apr 5, 2025
26 checks passed
@kirankumarkolli kirankumarkolli deleted the users/aavasthy/updateheaders branch April 5, 2025 02:40
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.

[Thin Client Integration] Add containerId and resourceId to ThinClient request

2 participants