Skip to content
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

[Bug] cron:archive CLI changed response format in Matomo 5 - instead of DateTime before every row, only time is displayed #21924

Closed
4 tasks done
peterbo opened this issue Feb 13, 2024 · 2 comments · Fixed by #21940
Assignees
Labels
Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone

Comments

@peterbo
Copy link
Contributor

peterbo commented Feb 13, 2024

What happened?

cron:archive CLI changed response format - instead of DateTime before every row, only time is displayed:

  • before Matomo 5: INFO [2024-02-01 22:27:07] 8632 INIT
  • Matomo 5: INFO [08:03:33] 11096 INIT

What should happen?

The log/response format should still include date and time. Logging/analyzing/debugging the archiver output is very hard, if you just have the time and not the date. Also, monitoring scripts that search for errors in the output files do not work anymore and would have to be changed.

How can this be reproduced?

Setup cron archiving and output into a logfile.

Matomo version

5

PHP version

all

Server operating system

all

What browsers are you seeing the problem on?

No response

Computer operating system

all

Relevant log output

- before Matomo 5: `INFO [2024-02-01 22:27:07] 8632  INIT`
- Matomo 5: `INFO      [08:03:33] 11096  INIT`

Validations

@peterbo peterbo added Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels Feb 13, 2024
@michalkleiner
Copy link
Contributor

Likely caused by the update of symfony/console and how it configures its formatters.

@sgiehl
Copy link
Member

sgiehl commented Feb 14, 2024

Yes. I guess Symfony changed the default. I'll quickly check if that's easy to change.

@sgiehl sgiehl added Regression Indicates a feature used to work in a certain way but it no longer does even though it should. and removed Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels Feb 14, 2024
@sgiehl sgiehl self-assigned this Feb 14, 2024
@sgiehl sgiehl added this to the 5.0.3 milestone Feb 14, 2024
@mneudert mneudert linked a pull request Feb 14, 2024 that will close this issue
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants