Start Dumbster only if instance of SMTP provider.

labkey-bpatel · labkey-bpatel · commit 92a3b7a8b6ab · 2026-02-09T22:00:06.000-08:00
diff --git a/modules/dumbster/src/org/labkey/dumbster/DumbsterModule.java b/modules/dumbster/src/org/labkey/dumbster/DumbsterModule.java
@@ -20,6 +20,8 @@
 import org.labkey.api.module.CodeOnlyModule;
 import org.labkey.api.module.ModuleContext;
 import org.labkey.api.settings.AppProps;
+import org.labkey.api.util.MailHelper;
+import org.labkey.api.util.SmtpTransportProvider;
 import org.labkey.api.view.BaseWebPartFactory;
 import org.labkey.api.view.Portal;
 import org.labkey.api.view.ViewContext;
@@ -65,7 +67,7 @@ public WebPartView<?> getWebPartView(@NotNull ViewContext portalCtx, @NotNull Po
     @Override
     public void doStartup(ModuleContext moduleContext)
     {
-        if (AppProps.getInstance().isMailRecorderEnabled())
+        if (MailHelper.getActiveProvider() instanceof SmtpTransportProvider && AppProps.getInstance().isMailRecorderEnabled())
             DumbsterManager.get().start();
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,8 @@`
`20`	`20`	`import org.labkey.api.module.CodeOnlyModule;`
`21`	`21`	`import org.labkey.api.module.ModuleContext;`
`22`	`22`	`import org.labkey.api.settings.AppProps;`
	`23`	`+import org.labkey.api.util.MailHelper;`
	`24`	`+import org.labkey.api.util.SmtpTransportProvider;`
`23`	`25`	`import org.labkey.api.view.BaseWebPartFactory;`
`24`	`26`	`import org.labkey.api.view.Portal;`
`25`	`27`	`import org.labkey.api.view.ViewContext;`
`@@ -65,7 +67,7 @@ public WebPartView<?> getWebPartView(@NotNull ViewContext portalCtx, @NotNull Po`
`65`	`67`	`@Override`
`66`	`68`	`public void doStartup(ModuleContext moduleContext)`
`67`	`69`	`{`
`68`		`- if (AppProps.getInstance().isMailRecorderEnabled())`
	`70`	`+ if (MailHelper.getActiveProvider() instanceof SmtpTransportProvider && AppProps.getInstance().isMailRecorderEnabled())`
`69`	`71`	`DumbsterManager.get().start();`
`70`	`72`	`}`
`71`	`73`	`}`