๐Ÿ‡ฎ๐Ÿ‡ณIndia @shivamitakari

Account created on 22 January 2013, almost 12 years ago
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

shivamitakari โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

shivamitakari โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

Created a patch to add support for configuring copy prevention per page basis.

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

I have tested the patch select2-check-indexes-3271205-8.patch and it works fine.

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

Hello Greg,

Apologies for coming late on this thread. May I please know what do you mean by adding that option in drush itself?

with this patch, I'm adding one parameter, and its used in the sql-sanitize command.

./vendor/bin/drush sql:sanitize --sanitize-tfa=no

This will skip the sanitization.

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

Hello Greg,

Thanks for quickly having look.
Yes disabling can be an option but I do still need users with admin roles to have TFA on non-prod env for safety and with the current TFA module there is no option to skip the TFA sanitization

This patch just gives an option for users if they want to skip sanitization.

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

Hello Greg,

Please find the below motivation for the patch.

I usually do production database sanitization before importing it to lower environments (like development and test).

I have TFA setup on a production website and whenever I sanitize the production database, TFA data is getting sanitized as it's not having the option to skip the sanitization. Due to this in lower environments, while testing users need to set up TFA again and they can't use the same TFA which they have set up on production.

๐Ÿ‡ฎ๐Ÿ‡ณIndia shivamitakari

Fixed by setting default value as empty array.

Production build 0.71.5 2024