[SERVER-77723] Turn parseNamespaceFromRequest calls to deserialize in IDL Created: 01/Jun/23 Updated: 29/Oct/23 Resolved: 10/Aug/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 7.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Hugh Tong (Inactive) | Assignee: | Hugh Tong (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | ntdi_must_have | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||||||
| Sprint: | Server Serverless 2023-07-24, Server Serverless 2023-08-07, Server Serverless 2023-08-21 | ||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||
| Description |
|
We currently call NamespaceStringUtil::parseNamespaceFromRequest() to deserialize the top-level namespace string object in command requests, but are not passing in the serialization context. This prevents us from deserializing into the namespace string object correctly according to the request's expectPrefix field, so we need to address this. Tests are not currently failing around this because we have not yet introduced the expectPrefix field in our requests yet, so we will need to add some testing for this. |
| Comments |
| Comment by Githook User [ 08/Aug/23 ] |
|
Author: {'name': 'Hugh Tong', 'email': 'hugh.tong@mongodb.com', 'username': 'cortrain'}Message: |
| Comment by Sophia Tan [ 12/Jul/23 ] |
|
Please replace the `parseNamespaceFromRequest` with `NamespaceStringUtil::serialize` function. |