Log response string to assist user (#486)

jenkinsci · Jan 23, 2019 · 6be0492 · 6be0492
1 parent 928b23e
commit 6be0492
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/main/java/jenkins/plugins/slack/workflow/SlackSendStep.java b/src/main/java/jenkins/plugins/slack/workflow/SlackSendStep.java
@@ -280,7 +280,7 @@ protected SlackResponse run() throws Exception {
                         org.json.JSONObject result = new org.json.JSONObject(responseString);
                         response = new SlackResponse(result);
                     } catch (org.json.JSONException ex) {
-                        listener.error(Messages.FailedToParseSlackResponse());
+                        listener.error(Messages.FailedToParseSlackResponse(responseString));
                         if (step.failOnError) {
                             throw ex;
                         }

diff --git a/src/main/resources/jenkins/plugins/slack/Messages.properties b/src/main/resources/jenkins/plugins/slack/Messages.properties
@@ -6,4 +6,4 @@ NotificationFailed=Slack notification failed. See Jenkins logs for details.
 NotificationFailedWithException=Slack notification failed with exception: {0}
 SlackSendStepValues=Slack Send Pipeline step running, values are - baseUrl: {0}, teamDomain: {1}, channel: {2}, color: {3}, botUser: {4}, tokenCredentialId: {5}
 SlackSendStepValuesEmptyMessage=<empty>
-FailedToParseSlackResponse=Could not parse response from slack, potentially because of invalid configuration (botUser: true and baseUrl set)
+FailedToParseSlackResponse=Could not parse response from slack, potentially because of invalid configuration (botUser: true and baseUrl set), response: {0}