View Issue Details

IDProjectCategoryView StatusLast Update
0004205Composrwelcome_emailspublic2020-04-24 06:13
ReporterChris GrahamAssigned To 
SeverityFeature-request 
Status non-assignedResolutionopen 
Product Version 
Fixed in Version 
Summary0004205: Add temporary_plain password compatibility scheme
DescriptionWe support 'temporary' (forced-to-change) passwords and 'plain' (non-hashed) passwords, but we don't support the combination together.
The combination has a use for people setting up new accounts and wanting to reference the password in an immediate welcome e-mail.

This is what we would need to do:
1) Make sure any code referencing 'temporary' also supports 'temporary_plain'.
2) Make sure any code referencing 'plain' also supports 'temporary_plain'.
3) If the hidden 'no_password_hashing' option is set, the add-new-member form should create a 'temporary_plain' member not a 'temporary' member if the 'Force temporary password' checkbox is ticked (checked).
4) Add a new 'added_accounts_password_compat_scheme' hidden option for setting the default for, or hiding, the aforementioned checkbox (so the admin can ensure that all new accounts are temporary [or not]). It should take these possible values:
i) '' for showing the checkbox defaulted to unticked (unchecked)
ii) 'default_temporary' for showing the checkbox defaulted to ticked (checked)
iii) 'default_temporary_plain' for the above, but behind-the-scenes it sets 'temporary_plain' not 'temporary'
iv) 'force_standard' for not showing the checkbox and forcing the standard ('') hashed password scheme
v) 'force_temporary' for not showing the checkbox and forcing the 'temporary' password scheme
vi) 'force_temporary_plain' for not showing the checkbox and forcing the 'temporary_plain' password scheme

If this is all done right, the webmaster will be able to:
a) Set the new 'added_accounts_password_compat_scheme' hidden option to 'force_temporary_plain' to force new manually added accounts to have 'temporary_plain' passwords
b) Set a welcome e-mail to include something like this in the text... \{+START,IF,\{$EQ,{m_password_compat_scheme},temporary_plain\}\}Your temporary password is: \{m_pass_hash_salted\}\{+END\}
(the slashes are needed to defer Tempcode parsing until after the variable substitution happens, necessary for Tempcode checks to work upon that data)

The above would need testing as a part of this issue.
Additional InformationSee https://compo.sr/forum/topicview/browse/designing/help-with-account.htm?post_id=6591&topic_id=1712&timestamp=1587700593&redirected=1#post_6591
TagsNo tags attached.
Time estimation (hours)1.5
Sponsorship openOpen

Relationships

related to 0004206 resolvedChris Graham Support for sending an e-mail when you add a member manually 

Activities

There are no notes attached to this issue.

Issue History

Date Modified Username Field Change
2020-04-24 03:36 Chris Graham New Issue
2020-04-24 03:36 Chris Graham Category content_reviews => welcome_emails
2020-04-24 03:36 Chris Graham Sponsorship open Open => Open
2020-04-24 03:46 Chris Graham Description Updated View Revisions
2020-04-24 03:46 Chris Graham Sponsorship open Open => Open
2020-04-24 04:00 Chris Graham Additional Information Updated View Revisions
2020-04-24 04:00 Chris Graham Sponsorship open Open => Open
2020-04-24 04:02 Chris Graham Relationship added related to 0004206