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

Log carbonapi uuid and request headers #175

Merged
merged 7 commits into from
Nov 15, 2021

Conversation

msaf1980
Copy link
Collaborator

@msaf1980 msaf1980 commented Nov 4, 2021

Some simplify find query info:

  1. Log carbonapi_uuid in complete query chain
  2. Allow to log additional response headers (for example, may be in future grafana request id for completely trace queries)

@msaf1980 msaf1980 marked this pull request as draft November 4, 2021 09:48
@msaf1980 msaf1980 marked this pull request as ready for review November 4, 2021 09:50
graphite-clickhouse.go Outdated Show resolved Hide resolved
@Felixoid Felixoid merged commit 950f05d into go-graphite:master Nov 15, 2021
msaf1980 added a commit to msaf1980/graphite-clickhouse that referenced this pull request Jan 12, 2022
* log carbonapi uuid (for simplify log find)

* allow to log additional request headers

* request handler: allow to log additional request headers

* add forgotten helper lib

* fix documentation

* incapsulate config into app handler

* one method for logging config in handlers
msaf1980 added a commit to msaf1980/graphite-clickhouse that referenced this pull request Jan 12, 2022
* log carbonapi uuid (for simplify log find)

* allow to log additional request headers

* request handler: allow to log additional request headers

* add forgotten helper lib

* fix documentation

* incapsulate config into app handler

* one method for logging config in handlers
msaf1980 added a commit to msaf1980/graphite-clickhouse that referenced this pull request Feb 10, 2022
* log carbonapi uuid (for simplify log find)

* allow to log additional request headers

* request handler: allow to log additional request headers

* add forgotten helper lib

* fix documentation

* incapsulate config into app handler

* one method for logging config in handlers
msaf1980 added a commit to msaf1980/graphite-clickhouse that referenced this pull request Feb 10, 2022
* log carbonapi uuid (for simplify log find)

* allow to log additional request headers

* request handler: allow to log additional request headers

* add forgotten helper lib

* fix documentation

* incapsulate config into app handler

* one method for logging config in handlers
msaf1980 added a commit to msaf1980/graphite-clickhouse that referenced this pull request Feb 11, 2022
* log carbonapi uuid (for simplify log find)

* allow to log additional request headers

* request handler: allow to log additional request headers

* add forgotten helper lib

* fix documentation

* incapsulate config into app handler

* one method for logging config in handlers
msaf1980 added a commit to msaf1980/graphite-clickhouse that referenced this pull request Feb 20, 2022
* log carbonapi uuid (for simplify log find)

* allow to log additional request headers

* request handler: allow to log additional request headers

* add forgotten helper lib

* fix documentation

* incapsulate config into app handler

* one method for logging config in handlers
@msaf1980 msaf1980 deleted the uuids_and_headers branch February 7, 2024 04:45
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