Error al cargar clave de motor del archivo parameters.yml

[tomasdelvechio]

Estaba armando un deploy usando la instalación silenciosa. En un momento armo el archivo parameters.yml con la siguiente sección

# Ruta absoluta a los archivos con las claves en texto plano
    toba-archivo-clave-bd: /tmp/db1_pass
    toba-archivo-clave-admin: /tmp/db2_pass

La creación de los archivos la hago con

echo "postgres" > /tmp/db1_pass
echo "postgres" > /tmp/db2_pass

Sin embargo, me daba de forma persistente error en la contraseña de la base postgres

Creando base 'toba_3_3'...                                                                                                                                              
 ERROR ejecutando SQL:
  [CODIGO]: 7
  [SQLSTATE]: db_08006                                                                                                                                                   
  [MENSAJE]: connection to server at "pg" (172.19.0.2), port 5432 failed: FATAL:  password authentication failed for user "postgres"
  [SQL EJECUTADA]:         

Del lado del server:

2023-05-10 14:39:50.713 UTC [571] DETAIL:  Password does not match for user "postgres".

Mi sorpresa fue grande cuando sin la instalación silenciosa y los mismos datos, pude crear la base sin problemas.

Para probar, tiro un var_dump luego de está linea:

var_dump($clave);

Y la salida es

array(8) {                                                                                                                                                               
  ["motor"]=>                                                                                                                                                            
  string(9) "postgres7"                                                                                                                                                  
  ["profile"]=>                                                                                                                                                          
  string(2) "pg"                                                                                                                                                         
  ["usuario"]=>                                                                                                                                                          
  string(8) "postgres"                                                                                                                                                   
  ["clave"]=>                                                                                                                                                            
  string(9) "postgres                                                                                                                                                    
"                                                                                                                                                                        
  ["base"]=>                                                                                                                                                             
  string(9) "*********"                                                                                                                                                  
  ["puerto"]=>                                                                                                                                                           
  int(5432)                                                                                                                                                              
  ["encoding"]=>                                                                                                                                                         
  string(6) "LATIN1"                                                                                                                                                     
  ["schema"]=>                                                                                                                                                           
  string(10) "desarrollo"                                                                                                                                                
}

La clave venia con un enter.

Me puse a mirar, y encontre la siguiente linea.

Encontré que file_get_contents suele hacer esto. Me parece raro porque hubiera pensado que esto se está usando en un montón de lados, pero resulta que agregue rtrim a la siguiente linea y salio andando.

                        if (! $es_invalido) {
				$clave = rtrim($this->recuperar_contenido_archivo($param[$ind]));
			}

No lo cargué como bug porque no se si yo estoy haciendo algo mal.

[enfoqueNativo]

Hola @tomasdelvechio

La clave venia con un enter.

Me puse a mirar, y encontre la siguiente linea.

Encontré que file_get_contents suele hacer esto. Me parece raro porque hubiera pensado que esto se está usando en un montón de lados, pero resulta que agregue rtrim a la siguiente linea y salio andando.

Me llama la atencion que no haya fallado antes, ya que el metodo para recuperar desde el archivo se usa tambien para definir la clave del usuario administrador, digamos por cada instalacion hay doble chance de fallo en teoria.

Y el uso del paquete siu/instalador pego el cambio hacia este mecanismo de instalacion de movida, digamos cualquier instalacion en los ultimos 4 años debio hacerlo saltar.

                        if (! $es_invalido) {
				$clave = rtrim($this->recuperar_contenido_archivo($param[$ind]));
			}

No lo cargué como bug porque no se si yo estoy haciendo algo mal.

Lo unico que me genera cierta duda es este mensaje, si quizas sea un tema de encoding que no levanta bien la funcion. Si te animas a probar para verificar que no viene por ahi mejor, es mi unica duda... si es eso creo que la salida por mb_string seria mejor.

Si te parece, encaralo con un PR pero directamente modificando dentro del metodo recuperar_contenido_archivo porque en definitiva afectaria ambas claves si se produce.

[tomasdelvechio]

@enfoqueNativo #156

[tomasdelvechio]

Lo armo y lo paso