-
Type:
Improvement
-
Resolution: Unresolved
-
Priority:
Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
-
None
-
Replication
-
None
-
None
-
None
-
None
-
None
-
None
-
None
In checkOperationAuthorization, the command is parsed to perform an authorization check, but the parse result is discarded. When the operation is later executed via run(), the command is parsed again.
The TODO at line 126 notes that the parse result should be cached and reused, with the caveat that the dbname may differ between the two phases (e.g., renameCollection uses admin for auth but the original namespace for execution).
- is duplicated by
-
SERVER-123552 Consider reusing command parse result from auth checks
-
- Closed
-
- is related to
-
SERVER-33881 Move old Command members into BasicCommand
-
- Closed
-