-
Type: Improvement
-
Resolution: Unresolved
-
Priority: Major - P3
-
None
-
Affects Version/s: None
-
Component/s: IDL
-
None
-
Server Programmability
SERVER-85791 introduced a few new helper functions that commands should use to parse their command requests, since it ensures that apiStrict is validated properly. Not all commands use this path though, in particular ones that don't perform apiStrict validation. We may want to add more command-generic parsing functionality like apiStrict validation in the future, so we should ensure all commands go through this common command request parsing path.
We could help enforce this by changing the methods that the IDL-compiler generates for commands (e.g. making them private with only a specific class being a friend will ensure only that path can be used).
Alternatively, if we update all commands to use TypedCommand (SERVER-91373) or BasicCommandWithRequestParser (SERVER-53890), we will get this as part of that. They aren't strictly necessary in order for this though.
- related to
-
SERVER-53890 Revisit BasicCommandWithRequestParser commands
- Backlog
-
SERVER-85791 Only parse command request once in command processing path
- Closed
-
SERVER-91373 Update all BasicCommands to TypedCommands
- Investigating