[SERVER-79574] Pass serialization context into makeFromFindCommand Created: 01/Aug/23 Updated: 04/Aug/23 Resolved: 04/Aug/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Hugh Tong (Inactive) | Assignee: | Hugh Tong (Inactive) |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | ntdi_must_have | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Sprint: | Server Serverless 2023-08-07 | ||||||||
| Participants: | |||||||||
| Description |
|
makeFromFindCommand is used by FindCmd and directly calls FindCommandRequest::parse() without passing in a serialization context. While we currently update the serialization context with VTS after constructing the find request, this doesn't work if we intend to check whether the supplied tenantId is received from $tenant or the security token/VTS (nonPrefixedTenantId). We need to pass in a prepared serialization context to the parse() function to ensure its values can be consumed by ::deserializeForCommands() during parsing. |
| Comments |
| Comment by Hugh Tong (Inactive) [ 04/Aug/23 ] |
|
The work for |