mail: declare DIRIGIBLE_MAIL_SENDER and make the sender / reply-to tenant-overridable
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- java
- Ambito
- authentication, backend
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Java
- Stelle
- 278
- Fork
- 97
- Merge medio
- 19h 30m
- PR unite (30g)
- 242
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di eclipse-dirigible/dirigible
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
eclipse-dirigible/dirigible#7958 ·
I maintainer di solito rispondono entro 1 giorno
-
intent: a create-from without fromStatus answers 200 on a missing or unknown source id (no 400/404)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
eclipse-dirigible/dirigible#7945 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
eclipse-dirigible/dirigible#7891 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
eclipse-dirigible/dirigible#6538 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
eclipse-dirigible/dirigible#4812 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di eclipse-dirigible/dirigible
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
apache/rocketmq-dashboard#7302 ·
I maintainer di solito rispondono entro 3 giorni
-
area: android good first issue help wanted testing
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Gautam-aman/ResQNet#56 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation hermes-queue
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
chrisalunlloyd2-sudo/mindpalace#294 ·
I maintainer di solito rispondono entro 1 giorno
-
better night vision compatAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
QuicksilverMC/Argentum#31 ·