-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Is this active? #5
Comments
FWIW, I had a small amount of involvement here, and probably still have some interest, but our company shifted our priorities elsewhere.
On this - you do need to watch out to catch anything new with its own tables & fields that could be capturing PII. For example, new webforms or fields. Whilst only developers would create those on many projects, it's very likely that many non-developers set those up on other sites. So a UI still has a lot of value on those; although I recognise that those might not be the primary audience of this tool. |
Yeah, true. I might have made my statement from our point of view where we control large Drupal sites. Similar to CDN module. The actual "engine" is the cdn module itself and then it has submodule "ui" which you can enable/use if needed. |
I need to test this. And if it works I might Fork the repo. |
@back-2-95 You could also add PR's instead of forking? Keep things centralized? |
@robiningelbrecht What do you think about your own interest/time to evolve this? Another question: how I include this in my project via Composer? Do I need to setup additional repository in composer.json? |
@back-2-95 My daytime job does not involve Drupal anymore, so I'm not planning on actively maintaining this. You should be able to install this by executing |
I'm happy to keep an eye out and help review things too, though only with a certain level of capacity to do so! Small changes at a time can work :-) |
I did try that req but got We at https://github.com/druidfi do still quite many Drupal projects so this in an interest for us. We have also done some forks like this before: https://github.com/druidfi/mona-plugin I'll discuss with my colleagues how we would like to procede. |
In any case, it might be a good idea to include LICENSE file on your repo. |
@robiningelbrecht here is a small PR #6 |
@robiningelbrecht followup PR #7 |
@robiningelbrecht as you seem to be familiar how Drush commands work, what do you think about How that could also utilize gdpr dump? |
@back-2-95 |
E.g So I was thinking maybe the custom code can modify things some how. |
If we would want to provide that kind of functionality, we would have to add a new drush command: I think that would be the best approach. |
That approach would rely on this package being available on the source database, which when connecting remotely, I'm not sure you can guarantee? I'd be concerned about implying that the dump is GDPR-safe, if it might not be. Or maybe that's not valid, I'm not totally sure. Just wanted to flag the thought up, in case. |
Yes this package/module needs to exist on both ends. We have Drush itself on projects with Composer so with that we ensure every environment has same Drush version and also other packages too. |
The "challenge" with GDPR here is that altering the dump must happen on source before transfer. As you cannot transfer the user data to your computer and then create compliant dump. Transfer of user data has already happened. So options are:
|
Hello, ended up here from https://www.drupal.org/project/gdpr/issues/2931820
I think the approach here is simple and maintainable. But I must ask if this is in any active development?
There is one PR for the UI where the discussion has also ended.
My 2cents is that creating UI is just nice-2-have and might revert attention from actual stuff. That said I'd say configuration (via $settings.php) is better way to configure your project's data anonymisation. That way we could avoid content producers etc from tinkering this configuration + avoid the need for exporting confs related to this.
Only developers should conf these.
Looking forward for reply.
The text was updated successfully, but these errors were encountered: