-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TASK-464 - Login - Forgot Password option #2439
Conversation
opencga-core/src/main/java/org/opencb/opencga/core/common/MailUtils.java
Show resolved
Hide resolved
Task linked: TASK-464 Login - Forgot Password option |
…verified during SSL/TLS connections #TASK-464
...c/main/java/org/opencb/opencga/catalog/auth/authentication/CatalogAuthenticationManager.java
Outdated
Show resolved
Hide resolved
opencga-core/src/main/java/org/opencb/opencga/core/common/MailUtils.java
Outdated
Show resolved
Hide resolved
opencga-core/src/main/java/org/opencb/opencga/core/common/MailUtils.java
Show resolved
Hide resolved
@@ -133,7 +133,8 @@ public OpenCGAResult resetPassword(String userId) throws CatalogException { | |||
String mailHost = this.emailConfig.getHost(); | |||
String mailPort = this.emailConfig.getPort(); | |||
try { | |||
MailUtils.sendResetPasswordMail(email, newPassword, mailUser, mailPassword, mailHost, mailPort); | |||
MailUtils.sendResetPasswordMail(email, newPassword, mailUser, mailPassword, mailHost, mailPort, | |||
"true", ((User) user.getResults().get(0)).getId()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This hardcoded ssl = "true"
should be in the configuration file. And should be a boolean instead of String.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the variables that are set in the code and are not in the configuration are necessary for the current version to work..... The objective of this task was to change the body of the email message. In the tests I discovered that it did not work with the current configuration, I have added those properties to make it work, if you want to bring that to the configuration it must be done in another requirement, which must be planned for the corresponding version because this must be released now.
https://app.clickup.com/t/24qrh5u