diff --git a/iam-login-service/src/main/java/it/infn/mw/iam/api/aup/AupSignatureController.java b/iam-login-service/src/main/java/it/infn/mw/iam/api/aup/AupSignatureController.java index 5780e1a00..170ae7874 100644 --- a/iam-login-service/src/main/java/it/infn/mw/iam/api/aup/AupSignatureController.java +++ b/iam-login-service/src/main/java/it/infn/mw/iam/api/aup/AupSignatureController.java @@ -48,6 +48,7 @@ import it.infn.mw.iam.audit.events.aup.AupSignedEvent; import it.infn.mw.iam.audit.events.aup.AupSignedOnBehalfEvent; import it.infn.mw.iam.core.time.TimeProvider; +import it.infn.mw.iam.notification.NotificationFactory; import it.infn.mw.iam.persistence.model.IamAccount; import it.infn.mw.iam.persistence.model.IamAup; import it.infn.mw.iam.persistence.model.IamAupSignature; @@ -69,15 +70,17 @@ public class AupSignatureController { private final IamAupRepository aupRepo; private final TimeProvider timeProvider; private final ApplicationEventPublisher eventPublisher; + private final NotificationFactory notificationFactory; public AupSignatureController(AupSignatureConverter conv, AccountUtils utils, IamAupSignatureRepository signatureRepo, IamAupRepository aupRepo, TimeProvider timeProvider, - ApplicationEventPublisher publisher) { + ApplicationEventPublisher publisher, NotificationFactory notificationFactory) { this.signatureConverter = conv; this.accountUtils = utils; this.signatureRepo = signatureRepo; this.aupRepo = aupRepo; this.timeProvider = timeProvider; this.eventPublisher = publisher; + this.notificationFactory = notificationFactory; } private Supplier aupNotFoundException() { @@ -194,6 +197,7 @@ public void deleteSignatureForAccount(@PathVariable String accountId, eventPublisher .publishEvent(AupSignatureDeletedEvent.deletedByClient(this, principal, signature.get())); } + notificationFactory.createAupSignatureRequestMessage(signatureAccount); } } diff --git a/iam-login-service/src/main/java/it/infn/mw/iam/notification/TransientNotificationFactory.java b/iam-login-service/src/main/java/it/infn/mw/iam/notification/TransientNotificationFactory.java index 1cef9174f..07679a4bf 100644 --- a/iam-login-service/src/main/java/it/infn/mw/iam/notification/TransientNotificationFactory.java +++ b/iam-login-service/src/main/java/it/infn/mw/iam/notification/TransientNotificationFactory.java @@ -395,7 +395,7 @@ public IamEmailNotification createAccountRestoredMessage(IamAccount account) { model.put(RECIPIENT_FIELD, recipient); model.put(ORGANISATION_NAME, organisationName); - String subject = "Account restored"; + String subject = String.format("[%s IAM] Account restored", organisationName); IamEmailNotification notification = createMessage("accountRestored.ftl", model, IamNotificationType.ACCOUNT_RESTORED, subject, asList(account.getUserInfo().getEmail())); @@ -410,6 +410,7 @@ protected IamEmailNotification createMessage(String templateName, Map receiverAddress) { try { + String formattedSubject = String.format("[%s IAM] %s", organisationName, subject); Template template = freeMarkerConfiguration.getTemplate(templateName); String body = FreeMarkerTemplateUtils.processTemplateIntoString(template, model); @@ -417,7 +418,7 @@ protected IamEmailNotification createMessage(String templateName, Map