-
-
Notifications
You must be signed in to change notification settings - Fork 592
feat(cassandra): add ssl option cassandra #3151
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 21 commits
ba9ac99
a87e16b
d7c7be5
22c2887
dfe89e6
59a1733
4e7b372
875e63f
8e1b349
26a38e1
7d76d50
cc1dd06
71100a9
f07aea2
ac19eb4
5b636f7
2ad9f7b
7409635
4c2e3dc
6da1e1e
9a4f4f2
5242c32
18214a0
8ba3319
60622d4
38320ac
d1724a5
b05ef84
86f661b
879e285
fd5772d
545a00e
19f4879
50579f9
42f32ec
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,10 +2,12 @@ package cassandra | |
|
|
||
| import ( | ||
| "context" | ||
| "crypto/tls" | ||
| "fmt" | ||
| "io" | ||
| "path/filepath" | ||
| "strings" | ||
| "time" | ||
|
|
||
| "github.com/docker/go-connections/nat" | ||
|
|
||
|
|
@@ -14,28 +16,35 @@ import ( | |
| ) | ||
|
|
||
| const ( | ||
| port = nat.Port("9042/tcp") | ||
| port = nat.Port("9042/tcp") | ||
| securePort = nat.Port("9142/tcp") // Common port for SSL/TLS connections | ||
| ) | ||
|
|
||
| // CassandraContainer represents the Cassandra container type used in the module | ||
| type CassandraContainer struct { | ||
| testcontainers.Container | ||
| settings Options | ||
| } | ||
|
|
||
| // ConnectionHost returns the host and port of the cassandra container, using the default, native 9000 port, and | ||
| // ConnectionHost returns the host and port of the cassandra container, using the default, native port, | ||
| // obtaining the host and exposed port from the container | ||
| func (c *CassandraContainer) ConnectionHost(ctx context.Context) (string, error) { | ||
| host, err := c.Host(ctx) | ||
| if err != nil { | ||
| return "", err | ||
| } | ||
|
|
||
| port, err := c.MappedPort(ctx, port) | ||
| // Use the secure port if TLS is enabled | ||
| portToUse := port | ||
| if c.settings.tlsConfig != nil { | ||
| portToUse = securePort | ||
| } | ||
|
|
||
| mappedPort, err := c.MappedPort(ctx, portToUse) | ||
| if err != nil { | ||
| return "", err | ||
| } | ||
|
|
||
| return host + ":" + port.Port(), nil | ||
| return host + ":" + mappedPort.Port(), nil | ||
| } | ||
|
|
||
| // WithConfigFile sets the YAML config file to be used for the cassandra container | ||
|
|
@@ -49,7 +58,6 @@ func WithConfigFile(configFile string) testcontainers.CustomizeRequestOption { | |
| FileMode: 0o755, | ||
| } | ||
| req.Files = append(req.Files, cf) | ||
|
|
||
| return nil | ||
| } | ||
| } | ||
|
|
@@ -66,10 +74,8 @@ func WithInitScripts(scripts ...string) testcontainers.CustomizeRequestOption { | |
| FileMode: 0o755, | ||
| } | ||
| initScripts = append(initScripts, cf) | ||
|
|
||
| execs = append(execs, initScript{File: cf.ContainerFilePath}) | ||
| } | ||
|
|
||
| req.Files = append(req.Files, initScripts...) | ||
| return testcontainers.WithAfterReadyCommand(execs...)(req) | ||
| } | ||
|
|
@@ -84,8 +90,7 @@ func RunContainer(ctx context.Context, opts ...testcontainers.ContainerCustomize | |
| // Run creates an instance of the Cassandra container type | ||
| func Run(ctx context.Context, img string, opts ...testcontainers.ContainerCustomizer) (*CassandraContainer, error) { | ||
| req := testcontainers.ContainerRequest{ | ||
| Image: img, | ||
| ExposedPorts: []string{string(port)}, | ||
| Image: img, | ||
| Env: map[string]string{ | ||
| "CASSANDRA_SNITCH": "GossipingPropertyFileSnitch", | ||
| "JVM_OPTS": "-Dcassandra.skip_wait_for_gossip_to_settle=0 -Dcassandra.initial_token=0", | ||
|
|
@@ -94,30 +99,44 @@ func Run(ctx context.Context, img string, opts ...testcontainers.ContainerCustom | |
| "CASSANDRA_ENDPOINT_SNITCH": "GossipingPropertyFileSnitch", | ||
| "CASSANDRA_DC": "datacenter1", | ||
| }, | ||
| WaitingFor: wait.ForAll( | ||
| wait.ForListeningPort(port), | ||
| wait.ForExec([]string{"cqlsh", "-e", "SELECT bootstrapped FROM system.local"}).WithResponseMatcher(func(body io.Reader) bool { | ||
| data, _ := io.ReadAll(body) | ||
| return strings.Contains(string(data), "COMPLETED") | ||
| }), | ||
| ), | ||
| ExposedPorts: []string{string(port)}, | ||
| } | ||
|
|
||
| genericContainerReq := testcontainers.GenericContainerRequest{ | ||
| ContainerRequest: req, | ||
| Started: true, | ||
| } | ||
|
|
||
| var settings Options | ||
|
||
| for _, opt := range opts { | ||
| if err := opt.Customize(&genericContainerReq); err != nil { | ||
| return nil, err | ||
| } | ||
| } | ||
|
|
||
| // Set up wait strategies | ||
| waitStrategies := []wait.Strategy{ | ||
| wait.ForListeningPort(port), | ||
| wait.ForExec([]string{"cqlsh", "-e", "SELECT bootstrapped FROM system.local"}).WithResponseMatcher(func(body io.Reader) bool { | ||
| data, _ := io.ReadAll(body) | ||
| return strings.Contains(string(data), "COMPLETED") | ||
| }).WithStartupTimeout(1 * time.Minute), | ||
| } | ||
mdelapenya marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| // Add TLS wait strategy if TLS config exists | ||
| if settings.tlsConfig != nil { | ||
| waitStrategies = append(waitStrategies, wait.ForListeningPort(securePort).WithStartupTimeout(1*time.Minute)) | ||
| } | ||
|
|
||
| // Apply wait strategy using the correct method | ||
| if err := testcontainers.WithWaitStrategy(wait.ForAll(waitStrategies...)).Customize(&genericContainerReq); err != nil { | ||
| return nil, err | ||
mdelapenya marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| } | ||
|
|
||
| container, err := testcontainers.GenericContainer(ctx, genericContainerReq) | ||
| var c *CassandraContainer | ||
| if container != nil { | ||
| c = &CassandraContainer{Container: container} | ||
| c = &CassandraContainer{Container: container, settings: settings} | ||
| } | ||
|
|
||
| if err != nil { | ||
|
|
@@ -126,3 +145,11 @@ func Run(ctx context.Context, img string, opts ...testcontainers.ContainerCustom | |
|
|
||
| return c, nil | ||
| } | ||
|
|
||
| // TLSConfig returns the TLS configuration for the Cassandra container, nil if TLS is not enabled. | ||
| func (c *CassandraContainer) TLSConfig() *tls.Config { | ||
|
||
| if c.settings.tlsConfig == nil { | ||
| return nil | ||
| } | ||
| return c.settings.tlsConfig.Config | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: do we need to two ports or can it just be one which is either SSL or not?