Project

General

Profile

Feature #24727

Created an encypted pool should prompt user to backup key immediately

Added by Dave F over 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Expected
Assignee:
Daisuke Imai
Category:
GUI (new)
Estimated time:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

We had a user in the IRC chatroom today who has lost 8TB of data on an encrypted pool because the keys were not backed up.

It's suggested that when creating an encrypted pool, the end user should be prompted to backup their key and make note of their passphrase immediately with a very scary looking warning pop-up when the 'Add volume' button is pushed.

The current pop-up can be too easily dismissed. By checking the 'Encryption' box and quickly hovering over the pop-up and out again causes it to disappear.

When the warning message is displayed, the user cannot continue until the key has been downloaded.

Example message to be displayed:

WARNING!

Backup your recovery key now! If the key is lost, the data on the disks will also be lost with no hope of recovery. 

>>> Click the 'Download Recovery Key' button to download your key <<<

This type of encryption is primarily targeted at users who are storing sensitive data and want the ability to remove disks from the pool and dispose of/re-use them without concern for erasure.

iXsystems, Inc. can not be held responsible for any lost or unrecoverable data as a consequence of using this feature.

+-----------------------+
| Download Recovery Key | 
+-----------------------+

Associated revisions

Revision f3f37034 (diff)
Added by Xin Tan about 4 years ago

Subject: prompt user to backup key if create an encrypted pool Done: ADD Encryption property checkbox with material desgin. UPDATED: Pool Name input with material design. Ticket: #24727

Revision 7787c71f (diff)
Added by Xin Tan about 4 years ago

Subject: prompt user to backup key if create an encrypt pool Done: ADDED a snackbar component and check if encrypted Ticket: #24727

History

#1 Updated by William Grzybowski over 4 years ago

  • Category changed from 578 to 569
  • Assignee changed from William Grzybowski to Xin Tan

#2 Updated by William Grzybowski over 4 years ago

  • Target version set to 11.1

#3 Updated by Xin Tan over 4 years ago

  • Status changed from Unscreened to Screened

#4 Updated by Xin Tan about 4 years ago

  • Status changed from Screened to Fix In Progress
  • % Done changed from 0 to 100

Need some final layout fix,then ready to ship

#5 Updated by Xin Tan about 4 years ago

  • Status changed from Fix In Progress to Needs Developer Review

#6 Updated by Dru Lavigne about 4 years ago

  • Status changed from Needs Developer Review to 46

Xin: did this code get reviewed? If not, who should review it?

#7 Updated by Xin Tan about 4 years ago

Hi Dru,
the UI part of the code has done but API doesn't support this feature yet.
Erin could review my code

#8 Updated by Dru Lavigne about 4 years ago

  • Status changed from 46 to Needs Developer Review
  • Assignee changed from Xin Tan to Anonymous

Erin: please review and make a separate ticket for the API piece.

#9 Updated by Anonymous about 4 years ago

  • Status changed from Needs Developer Review to Reviewed by Developer

It definitely warns the user, although this might be a good place to put a checkbox to make sure the user read it before clicking ok

#10 Updated by Dru Lavigne almost 4 years ago

  • Status changed from Reviewed by Developer to 46

Erin: should we mark the ticket as resolved then or do you want to leave it open for additional work?

#11 Avatar?id=14398&size=24x24 Updated by Kris Moore almost 4 years ago

  • Status changed from 46 to Unscreened
  • Assignee changed from Anonymous to Daisuke Imai

Daisuke, this is good place to add that double confirmation dialog

#12 Updated by Daisuke Imai almost 4 years ago

  • Status changed from Unscreened to Screened

#13 Updated by Dru Lavigne almost 4 years ago

  • Target version changed from 11.1 to 11.2-BETA1

#14 Updated by Daisuke Imai almost 4 years ago

  • Status changed from Screened to Fix In Progress

#15 Updated by Daisuke Imai almost 4 years ago

  • Status changed from Fix In Progress to Needs Developer Review

#16 Updated by Dru Lavigne almost 4 years ago

  • Assignee changed from Daisuke Imai to Lola Yang

#17 Updated by Lola Yang almost 4 years ago

  • Status changed from Needs Developer Review to 15
  • Assignee changed from Lola Yang to Daisuke Imai

I cannot see the download key dialog when creating an encrypted pool. Could you give us a screenshot if you have any? thanks

#18 Updated by Vaibhav Chauhan almost 4 years ago

  • Status changed from 15 to Ready For Release

LGTM,

#19 Updated by Vaibhav Chauhan almost 4 years ago

  • Status changed from Ready For Release to Reviewed by Developer

merged to master as well.

#20 Updated by Dru Lavigne almost 4 years ago

  • Status changed from Reviewed by Developer to Resolved
  • Target version changed from 11.2-BETA1 to Master - FreeNAS Nightlies

Also available in: Atom PDF