mail: declare DIRIGIBLE_MAIL_SENDER and make the sender / reply-to tenant-overridable
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- java
- Área
- authentication, backend
Línea de trabajo
Start by reading DirigibleConfig and TenantConfigurationKeyPolicy, then trace how generated notify: code resolves DIRIGIBLE_MAIL_SENDER before calling Mail.send(...). Define the sender and reply-to tenant behavior and check existing tenant-configuration tests; done means the effective tenant values are used, with documentation identifying which mail keys are per tenant. The optional per-tenant SMTP credentials need separate scope.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
Mail is configured once per instance. Generated notify: code calls Configurations.get("DIRIGIBLE_MAIL_SENDER", "[email protected]") and then Mail.send(...), and SMTP comes from the deployment-wide DIRIGIBLE_MAIL_* keys.
None of these keys is tenant-overridable: TenantConfigurationKeyPolicy allows only its listed keys plus the DIRIGIBLE_PRINT_TEMPLATE_ prefix. DIRIGIBLE_MAIL_SENDER is not a declared DirigibleConfig key at all.
In a multitenant application every tenant's invoices, dunning letters and payslips are therefore sent from the same address, and replies go to the operator rather than to the tenant's company.
Proposal
- Declare
DIRIGIBLE_MAIL_SENDERinDirigibleConfig, and make it tenant-overridable together with a newDIRIGIBLE_MAIL_REPLY_TO(the cheap, safe part: one SMTP account, a per-tenant From display name or Reply-To). - Optionally: per-tenant SMTP credentials, stored as tenant configuration secrets, falling back to the instance's account.
- Generated
notify:code reads the effective, tenant-resolved value. - Docs: which mail keys are per tenant.
- Lenguaje dominante
- Java
- Estrellas
- 278
- Forks
- 97
- Merge medio
- 16 h 46 min
- PR fusionados (30 d)
- 265
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 eclipse-dirigible/dirigible
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
eclipse-dirigible/dirigible#7958 ·
Los mantenedores suelen responder en 1 día
-
intent: a create-from without fromStatus answers 200 on a missing or unknown source id (no 400/404)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
eclipse-dirigible/dirigible#7945 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
eclipse-dirigible/dirigible#7891 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
eclipse-dirigible/dirigible#6538 ·
Los mantenedores suelen responder en 1 día
-
Create shortcut for Camel filesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
eclipse-dirigible/dirigible#4812 ·
Los mantenedores suelen responder en 1 día
Todos los issues de eclipse-dirigible/dirigible
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
[Bug]: REST API URL starting with "http" but no scheme (e.g. httpbin.org/get) is not given an http:// prefixPosiblemente ocupada @ItsDeadlyProgrammer la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
appsmithorg/appsmith#42471 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
code-maintenance
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
bug priority: medium Review effort [1-5]: 3 status: needs-triage type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
carlos-emr/carlos#5258 ·
Los mantenedores suelen responder en 1 día