Take named options in `PostgreSqlContainer.withSSL`
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- docker, postgresql, typescript
Línea de trabajo
Empieza con packages/modules/postgresql/src/postgresql-container.ts e inspecciona PostgreSqlContainer.withSSL junto con withSSLCert y las sobrecargas posicionales coordinadas con #1518. Añade la forma con campos con nombre y conserva las formas posicionales como sobrecargas obsoletas. El trabajo estará terminado cuando quienes llamen puedan pasar { cert, key, ca } y las firmas posicionales existentes sigan disponibles como sobrecargas obsoletas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happens
PostgreSqlContainer has two SSL methods that take the same three file paths in different positions:
withSSL(certFile: string, keyFile: string, caCertFile?: string)
withSSLCert(caCertFile: string, certFile: string, keyFile: string)
All three are plain strings, so passing them in the wrong order compiles. The mistake shows up only when the server fails to start or a client fails to verify.
Expected
A form with named fields, for example withSSL({ cert, key, ca }), with the positional forms kept as deprecated overloads.
Where
packages/modules/postgresql/src/postgresql-container.ts.
Notes
withSSLCert(caCertFile, certFile, keyFile)matches Go'spostgres.WithSSLCert(caCertFile, certFile, keyFile).withSSLis the one that differs.- Both were added in #1224.
- The same applies to any module method that takes several paths or credentials positionally. Worth a line in the
add-moduleskill.
Dependencies
- Nothing blocks this. It can start now.
- Coordinate with #1518: for the positional overloads.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.6k
- Forks
- 274
- Merge medio
- 1 d 14 h
- PR fusionados (30 d)
- 44
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de testcontainers/testcontainers-node
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
testcontainers/testcontainers-node#1533 ·
Los mantenedores suelen responder en 1 día
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
testcontainers/testcontainers-node#1531 ·
Los mantenedores suelen responder en 1 día
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
testcontainers/testcontainers-node#1529 ·
Los mantenedores suelen responder en 1 día
-
enhancement minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
testcontainers/testcontainers-node#1528 ·
Los mantenedores suelen responder en 1 día
-
bug patch
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
testcontainers/testcontainers-node#1526 ·
Los mantenedores suelen responder en 1 día
Todos los issues de testcontainers/testcontainers-node
Issues similares
-
afk-ok area:data-quality importer size:S
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
enorm-labs/event-junkie#3301 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
one-zero-eight/apply-bot#39 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
bigbluebutton/bbb-ui-components-react#108 ·
Los mantenedores suelen responder en 2 días
-
area: app type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
ExpidetGames/code-reviewer#308 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día