a8cteam51 / safety-net

Scrub options, deactivate denylisted plugins, and delete user data on development sites.
20 stars 1 forks source link

Add WP Remote User Sync plugin and options #94

Closed stronenv closed 1 year ago

stronenv commented 1 year ago

Added the WP Remote User Sync to deny list and the plugin options to scrub list.

Hey @NickGreen! It looks like the plugin stores all options in a wprus array, is there a way to only scrub the encryption keys?

NickGreen commented 1 year ago

Added the WP Remote User Sync to deny list

Could you please add it in alphabetical order? :)

It looks like the plugin stores all options in a wprus array, is there a way to only scrub the encryption keys?

There is definitely a way, but we usually only do that if we need to. For example, if clearing out the whole option causes fatals, or like in the case of Stripe, we want to keep the test API keys in place.

If there's a good reason to only clear the keys, I'm happy to help you add that (an example of how we do that for another plugin is here), just let me know.

stronenv commented 1 year ago

Thanks for your feedback, @NickGreen!

I've added two commits. Let me know what you think.

NickGreen commented 1 year ago

@stronenv This looks good and clean. Did you test it?

Disregard! I see in Slack you said:

I’ve tested on local

So, I think we're good to go. I'll merge it.