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

fix: request logs #1964

Merged
merged 2 commits into from
Jan 11, 2024
Merged

fix: request logs #1964

merged 2 commits into from
Jan 11, 2024

Conversation

gjedlicska
Copy link
Contributor

Description & motivation

Changes:

To-do before merge:

Screenshots:

Validation of changes:

Checklist:

  • My pull request follows the guidelines in the Contributing guide?
  • My pull request does not duplicate any other open Pull Requests for the same update/change?
  • My commits are related to the pull request and do not amend unrelated code or documentation.
  • My code follows a similar style to existing code.
  • I have added appropriate tests.
  • I have updated or added relevant documentation.

References

@gjedlicska gjedlicska requested a review from iainsproat January 11, 2024 15:15
@gjedlicska gjedlicska merged commit ee84900 into main Jan 11, 2024
2 of 17 checks passed
@gjedlicska gjedlicska deleted the gergo/reqLogImprovementMain branch January 11, 2024 15:17
gjedlicska added a commit that referenced this pull request Jan 11, 2024
* fix: request logs

* chore: remove comments
gjedlicska added a commit that referenced this pull request Jan 17, 2024
* gergo/apolloQueryDuration (#1949)

* add apollo query duration

* feat: add more details to apollo query logging

* fix: pr review

* feat: format log messages as clef (#1950)

* fix(logging): pinoClef log levels must be a string

* chore(fe2): reducing log level for some spammy req logs

* minor adjustment

* more robust path resolution

* better req log text

* feat(fe2): improved and more thorough logging to help with observability (#1948)

* better req log text

* minor improvements to server logging

* WIP FE2 req logging

* FE2 apollo operation logging

* undid apolloPlugin changes due to Gergos PR

* seq message templates introduced

* fix: request logs (#1964)

* fix: request logs

* chore: remove comments

* feat: add graphql subscription metrics (#1970)

* optimized preview msg resultListener

* fix(server): locking to avoid postgres notification listeners processing the same message multiple times (#1972)

* fix(server): locking to avoid postgres notification listeners processing the same message multiple times

* optimized locking

* minor cleanup

* msg update

* log level adjustments

* reduce failsafe expiry

---------

Co-authored-by: Iain Sproat <[email protected]>
Co-authored-by: Kristaps Fabians Geikins <[email protected]>
Co-authored-by: Kristaps Fabians Geikins <[email protected]>
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