Skip to content

Add logging for the data submitted#16

Open
desrosj wants to merge 4 commits intotrunkfrom
create-local-log
Open

Add logging for the data submitted#16
desrosj wants to merge 4 commits intotrunkfrom
create-local-log

Conversation

@desrosj
Copy link
Copy Markdown
Member

@desrosj desrosj commented May 1, 2026

To help hosts understand what data was submitted, the values will now be saved into a log file.

@desrosj desrosj self-assigned this May 1, 2026
Copy link
Copy Markdown
Contributor

@peterwilsoncc peterwilsoncc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few notes:

  • It would be good to generate the log independently of whether the reports are submitted to wp.org or not. That will allow contributors to retain log files. This will require moving the log generation in the plugin as it's currently in the submission function.
  • Blocker: Log files are best stored in the bash scripts directory rather than in the mu-plugins directory.
  • With above, the .gitignore file ought to be updated to exclude the log files.

Copy link
Copy Markdown
Contributor

@peterwilsoncc peterwilsoncc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now recording the log file for me, thanks. For testing without submitting, I'm setting the reporter url to https://invalid.invalid which creates the log but doesn't submit, of course.

If moving the log file is too fiddly then I think it's fine to keep it in the mu-directory as this isn't intended to be run on a production machine.

desrosj added 3 commits May 4, 2026 21:34
- Place file in the `ABSPATH` directory.
- Log the submission response and the test results separately.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants