|
||||||||||||||||
This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira |
----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com>
----------------------------------------------------------------
While changing the default recovery strategy is definitely a good move, I would suggest we keep MailCurrentPasswordRetrievalStrategy - or at least mark it as deprecated for now ? I don't think any relevant password mechanism should be able to send clear text password, but perhaps it can serve as an example, showing other behaviours could be adapted.
Or, on second thought, perhaps move the configuration stuff into an abstract class ("MailBasedAbstractStrategy or something), so that MailChangePasswordLinkStrategy is more "focused" ?
Now, we ask users to configure a targetPage - I'm assuming that means very specific components will need to be present in that page. Is this documented ?