Skip to content

Commit 7a6ee14

Browse files
committed
revert: drop all MongoDB connection string changes
Reverts MongoDB Atlas connection string support due to validation issues in the Zod schemas. VoyageAI integration remains intact.
1 parent 40153de commit 7a6ee14

File tree

16 files changed

+28
-643
lines changed

16 files changed

+28
-643
lines changed

apps/sim/app/api/tools/mongodb/delete/route.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,11 @@ import { createMongoDBConnection, sanitizeCollectionName, validateFilter } from
88
const logger = createLogger('MongoDBDeleteAPI')
99

1010
const DeleteSchema = z.object({
11-
connectionString: z.string().optional(),
12-
host: z.string().default(''),
13-
port: z.coerce.number().int().nonnegative().default(27017),
11+
host: z.string().min(1, 'Host is required'),
12+
port: z.coerce.number().int().positive('Port must be a positive integer'),
1413
database: z.string().min(1, 'Database name is required'),
15-
username: z.string().default(''),
16-
password: z.string().default(''),
14+
username: z.string().min(1, 'Username is required'),
15+
password: z.string().min(1, 'Password is required'),
1716
authSource: z.string().optional(),
1817
ssl: z.enum(['disabled', 'required', 'preferred']).default('preferred'),
1918
collection: z.string().min(1, 'Collection name is required'),
@@ -76,7 +75,6 @@ export async function POST(request: NextRequest) {
7675
}
7776

7877
client = await createMongoDBConnection({
79-
connectionString: params.connectionString,
8078
host: params.host,
8179
port: params.port,
8280
database: params.database,

apps/sim/app/api/tools/mongodb/execute/route.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,11 @@ import { createMongoDBConnection, sanitizeCollectionName, validatePipeline } fro
88
const logger = createLogger('MongoDBExecuteAPI')
99

1010
const ExecuteSchema = z.object({
11-
connectionString: z.string().optional(),
12-
host: z.string().default(''),
13-
port: z.coerce.number().int().nonnegative().default(27017),
11+
host: z.string().min(1, 'Host is required'),
12+
port: z.coerce.number().int().positive('Port must be a positive integer'),
1413
database: z.string().min(1, 'Database name is required'),
15-
username: z.string().default(''),
16-
password: z.string().default(''),
14+
username: z.string().min(1, 'Username is required'),
15+
password: z.string().min(1, 'Password is required'),
1716
authSource: z.string().optional(),
1817
ssl: z.enum(['disabled', 'required', 'preferred']).default('preferred'),
1918
collection: z.string().min(1, 'Collection name is required'),
@@ -62,7 +61,6 @@ export async function POST(request: NextRequest) {
6261
const pipelineDoc = JSON.parse(params.pipeline)
6362

6463
client = await createMongoDBConnection({
65-
connectionString: params.connectionString,
6664
host: params.host,
6765
port: params.port,
6866
database: params.database,

apps/sim/app/api/tools/mongodb/insert/route.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,11 @@ import { createMongoDBConnection, sanitizeCollectionName } from '../utils'
88
const logger = createLogger('MongoDBInsertAPI')
99

1010
const InsertSchema = z.object({
11-
connectionString: z.string().optional(),
12-
host: z.string().default(''),
13-
port: z.coerce.number().int().nonnegative().default(27017),
11+
host: z.string().min(1, 'Host is required'),
12+
port: z.coerce.number().int().positive('Port must be a positive integer'),
1413
database: z.string().min(1, 'Database name is required'),
15-
username: z.string().default(''),
16-
password: z.string().default(''),
14+
username: z.string().min(1, 'Username is required'),
15+
password: z.string().min(1, 'Password is required'),
1716
authSource: z.string().optional(),
1817
ssl: z.enum(['disabled', 'required', 'preferred']).default('preferred'),
1918
collection: z.string().min(1, 'Collection name is required'),
@@ -55,7 +54,6 @@ export async function POST(request: NextRequest) {
5554

5655
const sanitizedCollection = sanitizeCollectionName(params.collection)
5756
client = await createMongoDBConnection({
58-
connectionString: params.connectionString,
5957
host: params.host,
6058
port: params.port,
6159
database: params.database,

apps/sim/app/api/tools/mongodb/introspect/route.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,8 @@ import { createMongoDBConnection, executeIntrospect } from '../utils'
88
const logger = createLogger('MongoDBIntrospectAPI')
99

1010
const IntrospectSchema = z.object({
11-
connectionString: z.string().optional(),
12-
host: z.string().default(''),
13-
port: z.coerce.number().int().nonnegative().default(27017),
11+
host: z.string().min(1, 'Host is required'),
12+
port: z.coerce.number().int().positive('Port must be a positive integer'),
1413
database: z.string().optional(),
1514
username: z.string().optional(),
1615
password: z.string().optional(),
@@ -37,7 +36,6 @@ export async function POST(request: NextRequest) {
3736
)
3837

3938
client = await createMongoDBConnection({
40-
connectionString: params.connectionString,
4139
host: params.host,
4240
port: params.port,
4341
database: params.database || 'admin',

apps/sim/app/api/tools/mongodb/query/route.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,11 @@ import { createMongoDBConnection, sanitizeCollectionName, validateFilter } from
88
const logger = createLogger('MongoDBQueryAPI')
99

1010
const QuerySchema = z.object({
11-
connectionString: z.string().optional(),
12-
host: z.string().default(''),
13-
port: z.coerce.number().int().nonnegative().default(27017),
11+
host: z.string().min(1, 'Host is required'),
12+
port: z.coerce.number().int().positive('Port must be a positive integer'),
1413
database: z.string().min(1, 'Database name is required'),
15-
username: z.string().default(''),
16-
password: z.string().default(''),
14+
username: z.string().min(1, 'Username is required'),
15+
password: z.string().min(1, 'Password is required'),
1716
authSource: z.string().optional(),
1817
ssl: z.enum(['disabled', 'required', 'preferred']).default('preferred'),
1918
collection: z.string().min(1, 'Collection name is required'),
@@ -91,7 +90,6 @@ export async function POST(request: NextRequest) {
9190
}
9291

9392
client = await createMongoDBConnection({
94-
connectionString: params.connectionString,
9593
host: params.host,
9694
port: params.port,
9795
database: params.database,

apps/sim/app/api/tools/mongodb/update/route.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,11 @@ import { createMongoDBConnection, sanitizeCollectionName, validateFilter } from
88
const logger = createLogger('MongoDBUpdateAPI')
99

1010
const UpdateSchema = z.object({
11-
connectionString: z.string().optional(),
12-
host: z.string().default(''),
13-
port: z.coerce.number().int().nonnegative().default(27017),
11+
host: z.string().min(1, 'Host is required'),
12+
port: z.coerce.number().int().positive('Port must be a positive integer'),
1413
database: z.string().min(1, 'Database name is required'),
15-
username: z.string().default(''),
16-
password: z.string().default(''),
14+
username: z.string().min(1, 'Username is required'),
15+
password: z.string().min(1, 'Password is required'),
1716
authSource: z.string().optional(),
1817
ssl: z.enum(['disabled', 'required', 'preferred']).default('preferred'),
1918
collection: z.string().min(1, 'Collection name is required'),
@@ -100,7 +99,6 @@ export async function POST(request: NextRequest) {
10099
}
101100

102101
client = await createMongoDBConnection({
103-
connectionString: params.connectionString,
104102
host: params.host,
105103
port: params.port,
106104
database: params.database,

0 commit comments

Comments
 (0)