Details
diff --git a/src/main/java/azkaban/executor/mail/DefaultMailCreator.java b/src/main/java/azkaban/executor/mail/DefaultMailCreator.java
index a4e5038..cf9a6fe 100644
--- a/src/main/java/azkaban/executor/mail/DefaultMailCreator.java
+++ b/src/main/java/azkaban/executor/mail/DefaultMailCreator.java
@@ -51,8 +51,8 @@ public class DefaultMailCreator implements MailCreator {
@Override
public boolean createFirstErrorMessage(ExecutableFlow flow,
- EmailMessage message, String azkabanName, String clientHostname,
- String clientPortNumber, String... vars) {
+ EmailMessage message, String azkabanName, String scheme,
+ String clientHostname, String clientPortNumber, String... vars) {
ExecutionOptions option = flow.getExecutionOptions();
List<String> emailList = option.getFailureEmails();
@@ -90,7 +90,7 @@ public class DefaultMailCreator implements MailCreator {
message.println("</table>");
message.println("");
String executionUrl =
- "https://" + clientHostname + ":" + clientPortNumber + "/"
+ scheme + "://" + clientHostname + ":" + clientPortNumber + "/"
+ "executor?" + "execid=" + execId;
message.println("<a href='\"" + executionUrl + "\">" + flow.getFlowId()
+ " Execution Link</a>");
@@ -166,8 +166,8 @@ public class DefaultMailCreator implements MailCreator {
@Override
public boolean createSuccessEmail(ExecutableFlow flow, EmailMessage message,
- String azkabanName, String clientHostname, String clientPortNumber,
- String... vars) {
+ String azkabanName, String scheme, String clientHostname,
+ String clientPortNumber, String... vars) {
ExecutionOptions option = flow.getExecutionOptions();
List<String> emailList = option.getSuccessEmails();
@@ -194,7 +194,7 @@ public class DefaultMailCreator implements MailCreator {
message.println("</table>");
message.println("");
String executionUrl =
- "https://" + clientHostname + ":" + clientPortNumber + "/"
+ scheme + "://" + clientHostname + ":" + clientPortNumber + "/"
+ "executor?" + "execid=" + execId;
message.println("<a href=\"" + executionUrl + "\">" + flow.getFlowId()
+ " Execution Link</a>");
diff --git a/src/main/java/azkaban/executor/mail/MailCreator.java b/src/main/java/azkaban/executor/mail/MailCreator.java
index 4b66d82..13f0185 100644
--- a/src/main/java/azkaban/executor/mail/MailCreator.java
+++ b/src/main/java/azkaban/executor/mail/MailCreator.java
@@ -21,14 +21,14 @@ import azkaban.utils.EmailMessage;
public interface MailCreator {
public boolean createFirstErrorMessage(ExecutableFlow flow,
- EmailMessage message, String azkabanName, String clientHostname,
- String clientPortNumber, String... vars);
+ EmailMessage message, String azkabanName, String scheme,
+ String clientHostname, String clientPortNumber, String... vars);
public boolean createErrorEmail(ExecutableFlow flow, EmailMessage message,
String azkabanName, String scheme, String clientHostname,
String clientPortNumber, String... vars);
public boolean createSuccessEmail(ExecutableFlow flow, EmailMessage message,
- String azkabanName, String clientHostname, String clientPortNumber,
- String... vars);
+ String azkabanName, String scheme, String clientHostname,
+ String clientPortNumber, String... vars);
}
diff --git a/src/main/java/azkaban/utils/Emailer.java b/src/main/java/azkaban/utils/Emailer.java
index b032bae..c7dc165 100644
--- a/src/main/java/azkaban/utils/Emailer.java
+++ b/src/main/java/azkaban/utils/Emailer.java
@@ -112,7 +112,7 @@ public class Emailer extends AbstractMailer implements Alerter {
+ mailCreator.getClass().getCanonicalName());
boolean mailCreated =
- mailCreator.createFirstErrorMessage(flow, message, azkabanName,
+ mailCreator.createFirstErrorMessage(flow, message, azkabanName, scheme,
clientHostname, clientPortNumber);
if (mailCreated && !testMode) {
@@ -160,7 +160,7 @@ public class Emailer extends AbstractMailer implements Alerter {
+ mailCreator.getClass().getCanonicalName());
boolean mailCreated =
- mailCreator.createSuccessEmail(flow, message, azkabanName,
+ mailCreator.createSuccessEmail(flow, message, azkabanName, scheme,
clientHostname, clientPortNumber);
if (mailCreated && !testMode) {