trustedlogin / trustedlogin-vendor

Plugin to interact with TrustedLogin's encrypted storage infrastructure to redirect support staff into an authenticated session on client installations.
GNU General Public License v3.0
0 stars 1 forks source link

Add button to reset access keys #46

Closed zackkatz closed 3 years ago

zackkatz commented 4 years ago

When successful:

inztinkt commented 4 years ago

The double opt-in MUST NOT process if the dialog doesn't load. Too many times there are JS errors on a site.

Can you show me a screenshot of the GH dialog? Want to confirm if we're using an alert/modal or inline notice.

zackkatz commented 4 years ago

@inztinkt

1. Have "Danger Zone"-like section

2. Button shows modal

3. Forces confirm password

Not sure this would work in WP, but it's one more step, since this is a big change. I think this prevents a "person walked away from their computer without locking the screen" attack.

4. Show confirmation that the [keys were cycled]

inztinkt commented 4 years ago

@zackkatz For Step 3, should they be asked to log back into their WP site or TL SaaS?

zackkatz commented 4 years ago

@inztinkt For now, let's skip Step 3 and make sure to check nonces when completing the cycling.

inztinkt commented 4 years ago
Screenshot 2020-08-03 15 15 43

@zackkatz Should 'danger zone' go on the bottom of the settings page or on it's own settings page?

inztinkt commented 4 years ago

@zackkatz I've built this to leverage the browsers' usual confirm dialogue.

I can certainly add the secondary confirmation and warning if you'd like. Just need to confirm what the UX/output/etc should contain and if you want it in a modal/alert/etc.

zackkatz commented 4 years ago

@zackkatz Should 'danger zone' go on the bottom of the settings page or on it's own settings page?

Let's wait for Tracy to weigh in on this.

zackkatz commented 3 years ago

Closing for now—it'll need redesign.