diff --git a/CHANGELOG.md b/CHANGELOG.md index 3241fa99e9..07397536e7 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,10 @@ Canonical reference for changes, improvements, and bugfixes for Boundary. This new helper command allows users to authorize sessions against MySQL targets and automatically invoke a MySQL client with the appropriate connection parameters and credentials. +* cli: Added `boundary connect mongo` command for connecting to MongoDB targets. + This new helper command allows users to authorize sessions against MongoDB + targets and automatically invoke a MongoDB client with the appropriate + connection parameters and credentials. * Adds support to parse User-Agent headers and emit them in telemetry events ([PR](https://github.com/hashicorp/boundary/pull/5645)). * cli: Added `boundary connect cassandra` command for connecting to Cassandra targets. diff --git a/enos/modules/test_e2e_docker/test.sh b/enos/modules/test_e2e_docker/test.sh index 56e2e6b0ac..e05c8d0a48 100755 --- a/enos/modules/test_e2e_docker/test.sh +++ b/enos/modules/test_e2e_docker/test.sh @@ -17,7 +17,8 @@ apt update # default-mysql-client is used for mysql tests # wget is used for downloading external dependencies and repository keys # apt-transport-https enables HTTPS transport for APT repositories -apt install unzip pass lsb-release postgresql-client default-mysql-client wget apt-transport-https -y +# curl and ca-certificates are required for some repository setups (e.g., MongoDB). +apt install unzip pass lsb-release postgresql-client default-mysql-client wget apt-transport-https curl ca-certificates -y # Function to install Cassandra install_cassandra() { @@ -116,6 +117,13 @@ echo \ apt update apt install docker-ce-cli -y +# Install MongoDB client (mongosh) +# Reference: https://www.mongodb.com/docs/mongodb-shell/install/#debian +curl -fsSL https://pgp.mongodb.com/server-7.0.asc | gpg --dearmor -o /usr/share/keyrings/mongodb-server-7.0.gpg +echo "deb [arch=$(dpkg --print-architecture) signed-by=/usr/share/keyrings/mongodb-server-7.0.gpg] https://repo.mongodb.org/apt/debian bookworm/mongodb-org/7.0 main" | tee /etc/apt/sources.list.d/mongodb-org-7.0.list +apt update +apt install -y mongodb-mongosh + # Run Tests unzip /boundary.zip -d /usr/local/bin/ cd /src/boundary diff --git a/internal/cmd/commands.go b/internal/cmd/commands.go index 8bbecdddc6..5e164240c0 100644 --- a/internal/cmd/commands.go +++ b/internal/cmd/commands.go @@ -417,6 +417,12 @@ func initCommands(ui, serverCmdUi cli.Ui, runOpts *RunOptions) { Func: "mysql", } }), + "connect mongo": wrapper.Wrap(func() wrapper.WrappableCommand { + return &connect.Command{ + Command: base.NewCommand(ui, opts...), + Func: "mongo", + } + }), "connect cassandra": wrapper.Wrap(func() wrapper.WrappableCommand { return &connect.Command{ Command: base.NewCommand(ui, opts...), diff --git a/internal/cmd/commands/connect/connect.go b/internal/cmd/commands/connect/connect.go index b48c345f83..24dea2a37f 100644 --- a/internal/cmd/commands/connect/connect.go +++ b/internal/cmd/commands/connect/connect.go @@ -67,6 +67,7 @@ type Command struct { flagExec string flagUsername string flagDbname string + flagAuthSource string // HTTP httpFlags @@ -80,6 +81,9 @@ type Command struct { // MySQL mysqlFlags + // MongoDB + mongoFlags + // Cassandra cassandraFlags @@ -111,6 +115,8 @@ func (c *Command) Synopsis() string { return postgresSynopsis case "mysql": return mysqlSynopsis + case "mongo": + return mongoSynopsis case "cassandra": return cassandraSynopsis case "rdp": @@ -235,6 +241,9 @@ func (c *Command) Flags() *base.FlagSets { case "mysql": mysqlOptions(c, set) + case "mongo": + mongoOptions(c, set) + case "cassandra": cassandraOptions(c, set) @@ -327,6 +336,8 @@ func (c *Command) Run(args []string) (retCode int) { c.flagExec = c.postgresFlags.defaultExec() case "mysql": c.flagExec = c.mysqlFlags.defaultExec() + case "mongo": + c.flagExec = c.mongoFlags.defaultExec() case "cassandra": c.flagExec = c.cassandraFlags.defaultExec() case "rdp": @@ -683,6 +694,16 @@ func (c *Command) handleExec(clientProxy *apiproxy.ClientProxy, passthroughArgs envs = append(envs, mysqlEnvs...) creds = mysqlCreds + case "mongo": + mongoArgs, mongoEnvs, mongoCreds, mongoErr := c.mongoFlags.buildArgs(c, port, host, addr, creds) + if mongoErr != nil { + argsErr = mongoErr + break + } + args = append(args, mongoArgs...) + envs = append(envs, mongoEnvs...) + creds = mongoCreds + case "cassandra": cassandraArgs, cassandraEnvs, cassandraCreds, cassandraErr := c.cassandraFlags.buildArgs(c, port, host, addr, creds) if cassandraErr != nil { diff --git a/internal/cmd/commands/connect/mongo.go b/internal/cmd/commands/connect/mongo.go new file mode 100644 index 0000000000..f1db661ffe --- /dev/null +++ b/internal/cmd/commands/connect/mongo.go @@ -0,0 +1,120 @@ +package connect + +import ( + "fmt" + "strings" + + "github.com/hashicorp/boundary/api/proxy" + "github.com/hashicorp/boundary/internal/cmd/base" + "github.com/posener/complete" +) + +const ( + mongoSynopsis = "Authorize a session against a target and invoke a MongoDB client to connect" +) + +func mongoOptions(c *Command, set *base.FlagSets) { + f := set.NewFlagSet("MongoDB Options") + + f.StringVar(&base.StringVar{ + Name: "style", + Target: &c.flagMongoStyle, + EnvVar: "BOUNDARY_CONNECT_MONGO_STYLE", + Completion: complete.PredictSet("mongosh"), + Default: "mongosh", + Usage: `Specifies how the CLI will attempt to invoke a MongoDB client. Currently only "mongosh" is supported.`, + }) + + f.StringVar(&base.StringVar{ + Name: "username", + Target: &c.flagUsername, + EnvVar: "BOUNDARY_CONNECT_USERNAME", + Completion: complete.PredictNothing, + Usage: `Specifies the username to pass through to the client. May be overridden by credentials sourced from a credential store.`, + }) + + f.StringVar(&base.StringVar{ + Name: "dbname", + Target: &c.flagDbname, + EnvVar: "BOUNDARY_CONNECT_DBNAME", + Completion: complete.PredictNothing, + Usage: `Specifies the database name to pass through to the client.`, + }) + + f.StringVar(&base.StringVar{ + Name: "auth-source", + Target: &c.flagAuthSource, + EnvVar: "BOUNDARY_CONNECT_MONGO_AUTH_SOURCE", + Completion: complete.PredictNothing, + Default: "admin", + Usage: `Specifies the authentication database for MongoDB. Defaults to "admin" for root-style users.`, + }) +} + +type mongoFlags struct { + flagMongoStyle string +} + +func (m *mongoFlags) defaultExec() string { + return strings.ToLower(m.flagMongoStyle) +} + +func (m *mongoFlags) buildArgs(c *Command, port, ip, _ string, creds proxy.Credentials) (args, envs []string, retCreds proxy.Credentials, retErr error) { + var username, password string + + retCreds = creds + if len(retCreds.UsernamePassword) > 0 { + // Mark credential as consumed so it is not printed to user + retCreds.UsernamePassword[0].Consumed = true + + // For now just grab the first username password credential brokered + username = retCreds.UsernamePassword[0].Username + password = retCreds.UsernamePassword[0].Password + } + + switch m.flagMongoStyle { + case "mongosh": + // Build MongoDB connection string for mongosh + connectionString := "mongodb://" + + // Add username and password to connection string + if username != "" { + connectionString += username + if password != "" { + connectionString += ":" + password + } + connectionString += "@" + } else if c.flagUsername != "" { + connectionString += c.flagUsername + if password != "" { + connectionString += ":" + password + } + connectionString += "@" + } + + // Add host and port + connectionString += ip + if port != "" { + connectionString += ":" + port + } + + // Add database name + if c.flagDbname != "" { + connectionString += "/" + c.flagDbname + } + + // Add authSource parameter if not already present + authSource := c.flagAuthSource + + if !strings.Contains(connectionString, "?") { + connectionString += "?authSource=" + authSource + } else if !strings.Contains(strings.ToLower(connectionString), "authsource=") { + connectionString += "&authSource=" + authSource + } + + args = append(args, connectionString) + default: + return nil, nil, proxy.Credentials{}, fmt.Errorf("unsupported MongoDB style: %s", m.flagMongoStyle) + } + return +} diff --git a/testing/internal/e2e/infra/docker.go b/testing/internal/e2e/infra/docker.go index f84efb8f19..7250980a7f 100644 --- a/testing/internal/e2e/infra/docker.go +++ b/testing/internal/e2e/infra/docker.go @@ -354,6 +354,57 @@ func StartMysql(t testing.TB, pool *dockertest.Pool, network *dockertest.Network } } +// StartMongo starts a MongoDB database in a docker container. +// Returns information about the container +func StartMongo(t testing.TB, pool *dockertest.Pool, network *dockertest.Network, repository, tag string) *Container { + t.Log("Starting MongoDB database...") + c, err := LoadConfig() + require.NoError(t, err) + + err = pool.Client.PullImage(docker.PullImageOptions{ + Repository: fmt.Sprintf("%s/%s", c.DockerMirror, repository), + Tag: tag, + }, docker.AuthConfiguration{}) + require.NoError(t, err) + + networkAlias := "e2emongo" + mongoDb := "e2eboundarydb" + mongoUser := "e2eboundary" + mongoPassword := "e2eboundary" + + resource, err := pool.RunWithOptions(&dockertest.RunOptions{ + Repository: fmt.Sprintf("%s/%s", c.DockerMirror, repository), + Tag: tag, + Env: []string{ + "MONGO_INITDB_ROOT_USERNAME=" + mongoUser, + "MONGO_INITDB_ROOT_PASSWORD=" + mongoPassword, + "MONGO_INITDB_DATABASE=" + mongoDb, + }, + ExposedPorts: []string{"27017/tcp"}, + Name: networkAlias, + Networks: []*dockertest.Network{network}, + }) + require.NoError(t, err) + + return &Container{ + Resource: resource, + UriLocalhost: fmt.Sprintf( + "mongodb://%s:%s@%s/%s?authSource=admin", + mongoUser, + mongoPassword, + resource.GetHostPort("27017/tcp"), + mongoDb, + ), + UriNetwork: fmt.Sprintf( + "mongodb://%s:%s@%s:27017/%s?authSource=admin", + mongoUser, + mongoPassword, + networkAlias, + mongoDb, + ), + } +} + // StartCassandra starts a Cassandra database in a docker container. // Returns information about the container func StartCassandra(t testing.TB, pool *dockertest.Pool, network *dockertest.Network, repository, tag string) *Container { diff --git a/testing/internal/e2e/tests/base/target_tcp_connect_mongo_test.go b/testing/internal/e2e/tests/base/target_tcp_connect_mongo_test.go new file mode 100644 index 0000000000..de11c5cd40 --- /dev/null +++ b/testing/internal/e2e/tests/base/target_tcp_connect_mongo_test.go @@ -0,0 +1,138 @@ +// Copyright (c) HashiCorp, Inc. +// SPDX-License-Identifier: BUSL-1.1 + +package base_test + +import ( + "bytes" + "context" + "io" + "net/url" + "os/exec" + "testing" + + "github.com/creack/pty" + "github.com/hashicorp/boundary/internal/target" + "github.com/hashicorp/boundary/testing/internal/e2e" + "github.com/hashicorp/boundary/testing/internal/e2e/boundary" + "github.com/hashicorp/boundary/testing/internal/e2e/infra" + "github.com/ory/dockertest/v3" + "github.com/stretchr/testify/require" +) + +// TestCliTcpTargetConnectMongo uses the boundary cli to connect to a +// target using `connect mongo` +func TestCliTcpTargetConnectMongo(t *testing.T) { + e2e.MaybeSkipTest(t) + ctx := context.Background() + + pool, err := dockertest.NewPool("") + require.NoError(t, err) + + // e2e_cluster network is created by the e2e infra setup + network, err := pool.NetworksByName("e2e_cluster") + require.NoError(t, err, "Failed to get e2e_cluster network") + + c := infra.StartMongo(t, pool, &network[0], "mongo", "7.0") + require.NotNil(t, c, "MongoDB container should not be nil") + t.Cleanup(func() { + if err := pool.Purge(c.Resource); err != nil { + t.Logf("Failed to purge MongoDB container: %v", err) + } + }) + + u, err := url.Parse(c.UriLocalhost) + require.NoError(t, err, "Failed to parse MongoDB URL") + + user, hostname, port, db := u.User.Username(), u.Hostname(), u.Port(), u.Path[1:] + pw, pwSet := u.User.Password() + t.Logf("MongoDB info: user=%s, db=%s, host=%s, port=%s, password-set:%t", + user, db, hostname, port, pwSet) + + networkUrl, err := url.Parse(c.UriNetwork) + require.NoError(t, err) + containerName := networkUrl.Hostname() + + err = pool.Retry(func() error { + cmd := exec.CommandContext(ctx, "docker", "exec", containerName, + "mongosh", "-u", user, "-p", pw, "--authenticationDatabase", "admin", + "--eval", "db.runCommand('ping')") + return cmd.Run() + }) + require.NoError(t, err, "MongoDB container failed to start") + + boundary.AuthenticateAdminCli(t, ctx) + + orgId, err := boundary.CreateOrgCli(t, ctx) + require.NoError(t, err) + t.Cleanup(func() { + ctx := context.Background() + boundary.AuthenticateAdminCli(t, ctx) + output := e2e.RunCommand(ctx, "boundary", e2e.WithArgs("scopes", "delete", "-id", orgId)) + require.NoError(t, output.Err, string(output.Stderr)) + }) + + projectId, err := boundary.CreateProjectCli(t, ctx, orgId) + require.NoError(t, err) + + targetId, err := boundary.CreateTargetCli( + t, + ctx, + projectId, + port, + target.WithAddress(hostname), + ) + require.NoError(t, err) + + storeId, err := boundary.CreateCredentialStoreStaticCli(t, ctx, projectId) + require.NoError(t, err) + + credentialId, err := boundary.CreateStaticCredentialPasswordCli( + t, + ctx, + storeId, + user, + pw, + ) + require.NoError(t, err) + + err = boundary.AddBrokeredCredentialSourceToTargetCli(t, ctx, targetId, credentialId) + require.NoError(t, err) + + cmd := exec.CommandContext(ctx, + "boundary", + "connect", "mongo", + "-target-id", targetId, + "-dbname", db, + "-auth-source", "admin", + ) + f, err := pty.Start(cmd) + require.NoError(t, err) + t.Cleanup(func() { + err := f.Close() + require.NoError(t, err) + }) + + _, err = f.Write([]byte("db.runCommand('ping')\n")) + require.NoError(t, err) + + _, err = f.Write([]byte("db.getName()\n")) + require.NoError(t, err) + + _, err = f.Write([]byte("exit\n")) + require.NoError(t, err) + _, err = f.Write([]byte{4}) + require.NoError(t, err) + + var buf bytes.Buffer + _, _ = io.Copy(&buf, f) + + output := buf.String() + t.Logf("MongoDB session output: %s", output) + + require.Contains(t, output, `ok:`, "MongoDB ping command should succeed") + require.Contains(t, output, `1`, "MongoDB ping should return ok: 1") + require.Contains(t, output, db, "Database name should appear in the session") + require.NotContains(t, output, "MongoServerSelectionError", "Should not have connection errors") + t.Log("Successfully connected to MongoDB target") +} diff --git a/website/content/docs/commands/connect/index.mdx b/website/content/docs/commands/connect/index.mdx index 42ed579f9f..8a469f164c 100644 --- a/website/content/docs/commands/connect/index.mdx +++ b/website/content/docs/commands/connect/index.mdx @@ -43,6 +43,7 @@ Subcommands: kube Authorize a session against a target and invoke a Kubernetes client to connect cassandra Authorize a session against a target and invoke a Cassandra client to connect mysql Authorize a session against a target and invoke a MySQL client to connect + mongo Authorize a session against a target and invoke a MongoDB client to connect postgres Authorize a session against a target and invoke a Postgres client to connect rdp Authorize a session against a target and invoke an RDP client to connect ssh Authorize a session against a target and invoke an SSH client to connect @@ -57,6 +58,7 @@ of the subcommand in the sidebar or one of the links below: - [kube](/boundary/docs/commands/connect/kube) - [cassandra](/boundary/docs/commands/connect/cassandra) - [mysql](/boundary/docs/commands/connect/mysql) +- [mongo](/boundary/docs/commands/connect/mongo) - [postgres](/boundary/docs/commands/connect/postgres) - [rdp](/boundary/docs/commands/connect/rdp) - [ssh](/boundary/docs/commands/connect/ssh) diff --git a/website/content/docs/commands/connect/mongo.mdx b/website/content/docs/commands/connect/mongo.mdx new file mode 100644 index 0000000000..78b83440ae --- /dev/null +++ b/website/content/docs/commands/connect/mongo.mdx @@ -0,0 +1,84 @@ +--- +layout: docs +page_title: connect mongo - Command +description: >- + The "connect mongo" command performs a target authorization or consumes an existing authorization token, and then launches a proxied MongoDB connection. +--- + +# connect mongo + +Command: `boundary connect mongo` + +The `connect mongo` command authorizes a session against a target and invokes a MongoDB client for the connection. +The command fills in the local address and port. + +@include 'cmd-connect-env-vars.mdx' + + +## Examples + +The following example shows how to connect to a target with the ID `ttcp_eTcMueUYv` using a MongoDB helper: + +```shell-session +$ boundary connect mongo -target-id=ttcp_eTcZMueUYv \ + -dbname=northwind \ + -username=superuser \ + -auth-source=admin \ + -format=table +``` + +When prompted, you must enter the password for the user, "superuser": + + + +```plaintext +Enter password: +Current Mongosh Log ID: 64a1b2c3d4e5f6789012345 +Connecting to: mongodb://127.0.0.1:12345/northwind?authSource=admin +Using MongoDB: 7.0.0 +Using Mongosh: 2.0.0 + +northwind> show collections +orders +products +customers + +northwind> db.products.count() +77 + +northwind> +``` + + + +## Usage + + + +```shell-session +$ boundary connect mongo [options] [args] +``` + + + +@include 'cmd-connect-command-options.mdx' + +### MongoDB options: + +- `-dbname` `(string: "")` - The database name you want to pass through to the client. +You can also specify the database name using the **BOUNDARY_CONNECT_DBNAME** environment variable. + +- `-style` `(string: "mongosh")` - How the CLI attempts to invoke a MongoDB client. +This value also sets a suitable default for `-exec`, if you did not specify a value. +The default and currently-understood value is `mongosh`. +You can also specify how the CLI attempts to invoke a MongoDB client using the **BOUNDARY_CONNECT_MONGO_STYLE** environment variable. + +- `-username` `(string: "")` - The username you want to pass through to the client. +This value may be overridden by credentials sourced from a credential store. +You can also specify a username using the **BOUNDARY_CONNECT_USERNAME** environment variable. + +- `-auth-source` `(string: "admin")` - The authentication database for MongoDB. +Defaults to "admin" for root-style users initialized via MONGO_INITDB_ROOT_USERNAME. +You can also specify the authentication database using the **BOUNDARY_CONNECT_MONGO_AUTH_SOURCE** environment variable. + +@include 'cmd-option-note.mdx' \ No newline at end of file diff --git a/website/data/docs-nav-data.json b/website/data/docs-nav-data.json index 535d032ac2..48aee01dfc 100644 --- a/website/data/docs-nav-data.json +++ b/website/data/docs-nav-data.json @@ -1233,6 +1233,10 @@ "title": "mysql", "path": "commands/connect/mysql" }, + { + "title": "mongo", + "path": "commands/connect/mongo" + }, { "title": "postgres", "path": "commands/connect/postgres"