[GitHub] [fineract] BLasan commented on a change in pull request #1597: Add condition to start TLS (FINERACT-1070)

[GitBox <gi...@apache.org>]

BLasan commented on a change in pull request #1597:
URL: https://github.com/apache/fineract/pull/1597#discussion_r570796803



##########
File path: fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/service/GmailBackedPlatformEmailService.java
##########
@@ -61,6 +64,18 @@ public void sendDefinedEmail(EmailDetail emailDetails) {
         final String authuser = smtpCredentialsData.getUsername();
         final String authpwd = smtpCredentialsData.getPassword();
 
+        LOG.info("Use name: ", smtpCredentialsData.getUsername());
+        LOG.info("Use Password: ", smtpCredentialsData.getPassword());

Review comment:
       I haven't configured smtp on my pc. @francisguchie can use these logs to test the scenario in his own machine. This needs to be a draft PR. I'll change this into a draft one :)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org