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

Error log is polluted with interface messages #7693

Open
yobottehg opened this issue Aug 15, 2024 · 2 comments
Open

Error log is polluted with interface messages #7693

yobottehg opened this issue Aug 15, 2024 · 2 comments
Assignees

Comments

@yobottehg
Copy link

yobottehg commented Aug 15, 2024

Link to reproduction

No response

Payload Version

3.0.0-beta.68

Node Version

20 LTS latest

Next.js Version

15.0.0-canary.57

Describe the Bug

After running 3.0 on prod for some days I checked the container logs to reproduce some issues authors reported. This is very hard as the logs are full of interface messages which are not application errors like:

  1. [13:28:24] �[31mERROR�[39m: �[36mu: The email or password provided is incorrect.
  2. [13:16:15] �[31mERROR�[39m: �[36ml: The following field is invalid: alt
  3. [14:08:19] �[31mERROR�[39m: �[36ml: The following field is invalid: content.4.video

IMHO this should only be notices / info in the logs but not application errors.

Reproduction Steps

Check error logs in the container after trying to save an invalid field collection entry.

Adapters and Plugins

db-postgres

@yobottehg yobottehg added status: needs-triage Possible bug which hasn't been reproduced yet v3 labels Aug 15, 2024
@yobottehg yobottehg changed the title Error logs is polluted with interface messages Error log is polluted with interface messages Aug 15, 2024
@github-actions github-actions bot removed the status: needs-triage Possible bug which hasn't been reproduced yet label Aug 15, 2024
@yobottehg
Copy link
Author

Updated to Beta 78 and Next.js 104 today and this is still the same.

@denolfe
Copy link
Member

denolfe commented Aug 23, 2024

This is because the way errors are passed to the front end are via error messages. I wouldn't necessarily consider this an issue. We could look into possibly adding a flag that would make it so public errors are not logged.

@denolfe denolfe assigned DanRibbens and unassigned denolfe Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants