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

feat(glean):adding proper glean formatted JSON logs… #50

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

toufali
Copy link
Collaborator

@toufali toufali commented Oct 31, 2023

Opening a new PR for this, since the original PR #36 was reverted due to a bug found on stage server: the "about page" /about returns 502. Relevant error message:

2023/10/31 19:32:07 [error] 30#30: *883 upstream sent too big header while reading response header from upstream, client: 24.148.42.231, server: , request: "GET /about HTTP/1.1", upstream: "http://127.0.0.1:3000/about", host: "[stage.moztodon.nonprod.webservices.mozgcp.net](http://stage.moztodon.nonprod.webservices.mozgcp.net/)"

It's still unclear whether the bug was specifically caused by this Glean PR. The only other code in the buggy release was a copy-change PR. It's also possible the bug could be a stage-only related setup issue.

@toufali
Copy link
Collaborator Author

toufali commented Nov 1, 2023

The opinion is that the error above is probably unrelated to this PR – merging it back in.

@toufali toufali merged commit 9255beb into main Nov 1, 2023
@toufali toufali deleted the feature/glean-server-side-2 branch February 26, 2024 23:50
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.

1 participant