Closed francoa closed 8 years ago
Travis está fallando, algo en setup parece.
listo
Oka, esta va entonces. Qué es lo finalizado? Para el mensaje del merge. jaja
Si entiendo el script neoPasswordChange permite configurar la contraseña de la base de datos a la que se está conectando node. Lo que no entiendo, es por qué pide LAN, LOC, NET de la misma manera que el script de config (líneas 59 a 67). La idea es que uno pueda conectarse a una base de datos remota? No veo que esos datos se usen después en el script.
Ah si. Fer tiene razón. Y otra cosa. Si uno ya puso la IP LAN, sed no va a encontrar nada porque esta localhost harcoded en el script no?
Por ahi estaria bueno tmb leer OLDPASSWORD desde uno de los archivos porque ya está ahí, para que sea más straightforward.
Mmm, si es una db local deberías poder forzar el cambio de password por las malas. Tal vez estoy diciendo una tontera. Sino, debería pedírselo al usuario en prompt.
En el curl de cambio hay que poner el user y pass actual, no se puede forzar creo. Por eso digo de leerla de cualquira de los archivos prefijados con guión bajo y pedir la nueva en prompt
Y si se setea OLDPASSWORD cuando se ejecuta config
y este script se edita a si mismo cuando se hace el cambio? Se puede?
Che, igual creo que esto reviene a lo que hemos estado hablando. Deberíamos tener un archivo de configuración, en donde se definan este tipo de variables (dirección de la bd, pass) y todos los archivos deberían leer de ahí el valor de esa variable. Para no andar haciendo sed a 10 archivos.
A modo de ejemplo, el archivo de configuración de wordpress. Después todos los otros archivos ejecutables leen desde ahí el valor de esas variables útiles, y las pueden usar para lo que quieran. Algo similar estaría bueno!
// ** MySQL settings - You can get this info from your web host ** //
22 /** The name of the database for WordPress */
23 define('DB_NAME', 'database_name_here');
24
25 /** MySQL database username */
26 define('DB_USER', 'username_here');
27
28 /** MySQL database password */
29 define('DB_PASSWORD', 'password_here');
30
31 /** MySQL hostname */
32 define('DB_HOST', 'localhost');
33
34 /** Database Charset to use in creating database tables. */
35 define('DB_CHARSET', 'utf8');
36
37 /** The Database Collate type. Don't change this if in doubt. */
38 define('DB_COLLATE', '');
Si, se podría hacer un archivito que exponga "Constantes" y que se lea con require('rsa-config.js')
o algo así. Pero bueno esto para mi se tiene que mergear igual así partimos de la base estable no?
On Monday, March 21, 2016 10:26:24 AM ART, Andrés wrote:
Si, se podría hacer un archivito que exponga "Constantes" y que se lea con
require('rsa-config.js')
o algo así. Pero bueno esto para mi se tiene que mergear igual así partimos de la base estable no?
Si vos abrís un Issue para hacer un archivo de constantes, y además borrás las cosas que no sirven de neoPasswdChange, ahí empezamos a hablar en otros términos :D
Para mi el OLDPASSWORD se debería pedir siempre, como una medida de seguridad. Pero bueno, es mi opinión.
Y lo que decías del sed rulo por ahora no habría problema porque la BDD siempre está en local. Otra cosa es si alguna vez es necesario conectarse a una BDD remota como comentaba Fer.
Y lo que se hizo acá son modificaciones menores a setup
y config
para asegurar el correcto funcionamiento de estos scripts (había bug) y hacer el script neoPasswordChange
Mala mía lo del sed, se me mezcló lo que se hace para REMOTE (que va a desaparecer pronto jaja). Ok con lo de OLDPASSWORD entonces! Todo listo?
config
se evita la contraseñaneo4j
setup
se para el server de neo antes de borrar la carpeta bin