Closed (won't fix)
Project:
Purge
Version:
8.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
14 Feb 2017 at 06:10 UTC
Updated:
26 Sep 2023 at 20:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
joshi.rohit100Comment #3
joshi.rohit100Forgot to upload patch :)
Comment #4
joshi.rohit100Bootstraping drupal
Comment #5
joshi.rohit100Comment #6
nielsvm commentedI like this thought, although it should definitely be part of
purge_drushand not live in downstream modules.Thank you!
Comment #7
joshi.rohit100Comment #8
nielsvm commentedI looked into this and unfortunately decided against accepting this actual patch.
As I said, I love the idea, but the problem I have with this approach is that it makes
drush sql-sanitizedo something the user totally doesn't expect. Let's see how the command is described:Its very clearly aimed at user data, and no matter how I look at it, the purge queue contains state data and not barely ever privacy sensitive data. I do agree that we need something clever that clears the queue automatically just after a deployment, and I'll think further about how to do this.
Comment #9
moshe weitzman commentedDrush team plans to clear state data in sql-sanitize. So this patch is reasonable IMO. See https://github.com/drush-ops/drush/issues/2798
Comment #10
japerry