fix: version 2.10.1 -- "Probar correo" tiraba Error interno si la clave SMTP no descifraba
El decrypt() de la clave SMTP guardada estaba fuera del try/catch tanto en testSmtpConfig
(el boton "Probar") como en getSmtpConfigInternal (usado por el mailer real para forgot-
password, registro, invitaciones). Si ENCRYPTION_KEY no coincide con la que se uso para
guardar -- el caso documentado de exilaser, que recibio un dump de LinkedCorp con su propia
clave -- el fallo de decrypt() se propagaba sin control: "Error interno" generico en el boton
de prueba, y un crash silencioso en el envio real en vez de la degradacion a modo dev que el
propio comentario del codigo promete. Ambos casos ahora devuelven un mensaje claro / degradan
correctamente.
De paso se agrega `app.set('trust proxy', 1)`: la app corre detras de Nginx Proxy Manager pero
Express no lo sabia, asi que express-rate-limit usaba la IP del proxy para todos los clientes
(un cupo compartido en vez de uno por usuario) y tiraba un aviso de validacion en cada request.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
@@ -26,12 +26,22 @@ export async function getSmtpConfig() {
|
||||
export async function getSmtpConfigInternal() {
|
||||
const values = await repo.getAll(SMTP_KEYS);
|
||||
if (!values.smtp_host || !values.smtp_user || !values.smtp_pass_enc) return null;
|
||||
let pass;
|
||||
try {
|
||||
pass = decrypt(values.smtp_pass_enc);
|
||||
} catch {
|
||||
// ENCRYPTION_KEY distinta a la que se uso para guardar (ej. un dump restaurado en otro
|
||||
// servidor, caso exilaser) -- se trata como "no configurado" para no tumbar al que llama
|
||||
// (mailer.js degrada solo a modo dev, nunca debe romper forgot-password/registro/etc.).
|
||||
console.error('[smtp] no se pudo descifrar smtp_pass_enc -- revisa ENCRYPTION_KEY o vuelve a guardar la clave SMTP desde Configuracion');
|
||||
return null;
|
||||
}
|
||||
return {
|
||||
host: values.smtp_host,
|
||||
port: values.smtp_port ? Number(values.smtp_port) : 587,
|
||||
user: values.smtp_user,
|
||||
from: values.smtp_from || values.smtp_user,
|
||||
pass: decrypt(values.smtp_pass_enc),
|
||||
pass,
|
||||
};
|
||||
}
|
||||
|
||||
@@ -69,7 +79,14 @@ export async function testSmtpConfig(payload, actorId) {
|
||||
if (!pass) {
|
||||
const stored = await repo.getAll(['smtp_pass_enc']);
|
||||
if (!stored.smtp_pass_enc) throw new HttpError(400, 'No hay una clave guardada ni escrita para probar');
|
||||
pass = decrypt(stored.smtp_pass_enc);
|
||||
try {
|
||||
pass = decrypt(stored.smtp_pass_enc);
|
||||
} catch {
|
||||
// Pasa si ENCRYPTION_KEY cambio desde que se guardo (ej. un dump restaurado en otro
|
||||
// servidor con su propia clave, como exilaser) -- sin este catch tiraba un error generico
|
||||
// sin pista de como resolverlo.
|
||||
throw new HttpError(400, 'No se pudo descifrar la clave SMTP guardada (puede que se haya guardado con otra clave de cifrado). Vuelve a escribirla y guarda de nuevo.');
|
||||
}
|
||||
}
|
||||
|
||||
const transporter = nodemailer.createTransport({
|
||||
|
||||
Reference in New Issue
Block a user