diff options
| author | A. Jesse Jiryu Davis <jesse@mongodb.com> | 2021-05-04 19:29:17 -0400 |
|---|---|---|
| committer | Evergreen Agent <no-reply@evergreen.mongodb.com> | 2021-05-13 17:48:10 +0000 |
| commit | 500a405a5ce235507f56fb47e8d5d4b368d3458d (patch) | |
| tree | eecb1020c3049a816d28ffd5461b8d3ecab8686a /src/mongo/db/initialize_api_parameters.cpp | |
| parent | 2070fc76b3604f8c04997862159b0fc721eeb465 (diff) | |
| download | mongo-500a405a5ce235507f56fb47e8d5d4b368d3458d.tar.gz | |
SERVER-56550 Require consistent API params in getMore and txns
Transaction-continuing commands must use the same API parameters as the
transaction's first command (it is no longer optional), and similarly
getMore must use the same as the cursor-creating command.
Diffstat (limited to 'src/mongo/db/initialize_api_parameters.cpp')
| -rw-r--r-- | src/mongo/db/initialize_api_parameters.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/mongo/db/initialize_api_parameters.cpp b/src/mongo/db/initialize_api_parameters.cpp index 3962494fae6..812a452ea31 100644 --- a/src/mongo/db/initialize_api_parameters.cpp +++ b/src/mongo/db/initialize_api_parameters.cpp @@ -107,9 +107,7 @@ void enforceRequireAPIVersion(OperationContext* opCtx, Command* command) { auto isInternalClient = !client->session() || (client->session()->getTags() & transport::Session::kInternalClient); - // TODO (SERVER-56550): Don't excuse getMore or transaction-continuing commands. - if (gRequireApiVersion.load() && !opCtx->getClient()->isInDirectClient() && !isInternalClient && - command->getName() != "getMore" && !opCtx->isContinuingMultiDocumentTransaction()) { + if (gRequireApiVersion.load() && !opCtx->getClient()->isInDirectClient() && !isInternalClient) { uassert( 498870, "The apiVersion parameter is required, please configure your MongoClient's API version", |