- Issue created by @chrisck
- Status changed to Needs review
almost 2 years ago 12:05pm 20 June 2023 - last update
almost 2 years ago 3 pass - 🇮🇳India nikhil_110
I have add patch for log data export.. please review
- Status changed to Needs work
almost 2 years ago 10:35am 4 July 2023 - 🇮🇳India chandrashekhar_srijan
The patch applied successfully. The message is getting logged at admin/reports/dblog. But i suggest to change the message.
It is preferable to say View data export has completed successfully and the export file is downloaded.
- Status changed to Needs review
almost 2 years ago 11:23am 4 July 2023 - last update
almost 2 years ago 3 pass - 🇮🇳India nikhil_110
I have add patch for log data export and also address Comment #3. please review
- Status changed to RTBC
over 1 year ago 7:32pm 24 July 2023 - 🇨🇴Colombia dvasquez
How can we add to the log the current authenticated user name when downloaded the data?
- Status changed to Needs work
10 months ago 5:03am 25 June 2024 - 🇬🇧United Kingdom steven jones
Can we make this optional, I can imagine that not all sites would want this.
How about adding a way to turn it on using
\Drupal\Core\Site\Settings
, and then for those that need it, they can enable it.
It's not usual that Drupal would log executing a view. - First commit to issue fork.
- Merge request !42Issue #3367294: Log data export event based on settings.php. → (Open) created by diwakar07
- 🇮🇳India PrabuEla chennai
PrabuEla → changed the visibility of the branch 3367294-log-data-export to hidden.
- 🇮🇳India PrabuEla chennai
PrabuEla → changed the visibility of the branch 3367294-log-data-export to hidden.
- 🇮🇳India PrabuEla chennai
PrabuEla → changed the visibility of the branch 3367294-log-data-export to active.
- Status changed to Needs review
10 months ago 7:53am 25 June 2024 - 🇮🇳India diwakar07
Hi,
Created an MR to log the view export events based on the value of a key set in settings.php file.For enabling the logging of the view export event, add
$settings['data_export_logging'] = TRUE;
in your settings.php file.
Please review. - Status changed to RTBC
8 months ago 11:51am 7 August 2024 - 🇮🇳India sourav_paul Kolkata
I've applied the patch & it's working as described...
IMO: the log message could be more precise by showing the actual file name, It would be more useful for site administrators.
Attaching SS with this:
- Status changed to Needs work
about 2 months ago 3:27pm 13 February 2025 - 🇬🇧United Kingdom steven jones
Yeah, if we're going to allow logging then we should add some sense of what was exported, and to where.
I assume people want this sort of thing so that they can audit people exporting data from the site and keep an eye on them etc.
- 🇬🇧United Kingdom steven jones
✨ Create event once a data export finishes Active has a great idea: introduce an event that is emitted on export. We could get that issue in and then re-work this one to be an example implementation in a submodule.