Skip to content
This repository was archived by the owner on Aug 23, 2023. It is now read-only.

in cluster calls, set user agent #1469

Merged
merged 1 commit into from
Sep 23, 2019
Merged

in cluster calls, set user agent #1469

merged 1 commit into from
Sep 23, 2019

Conversation

Dieterbe
Copy link
Contributor

@Dieterbe Dieterbe commented Sep 19, 2019

before

POST /index/find HTTP/1.1.
Host: 10.36.10.50:6060.
User-Agent: Go-http-client/1.1.
Content-Length: 58.
Content-Type: application/json.
Uber-Trace-Id: 1d0fc4fe63bc4740:711fa951aa00a28b:188525e7ddc4c59b:0.
Accept-Encoding: gzip.
.
{"patterns":["some.id.of.a.metric.1*"],"orgId":1,"from":0}

after

POST /index/find HTTP/1.1.
Host: 172.23.0.15:6060.
User-Agent: metrictank/v0.13.0-7-g267c5bb (mode Shard; state NodeReady) Go/go1.13.
Content-Length: 58.
Content-Type: application/json.
Uber-Trace-Id: 5ef53711c1016414:3c63b8cea9962f65:1353d9db56dae1b7:1.
Accept-Encoding: gzip.
.
{"patterns":["some.id.of.a.metric.1*"],"orgId":1,"from":0}

@Dieterbe Dieterbe requested review from woodsaj and fkaleo September 19, 2019 13:34
@woodsaj
Copy link
Member

woodsaj commented Sep 19, 2019

why?

@Dieterbe
Copy link
Contributor Author

it's helpful when troubleshooting by ngrepping or tcpdumping

@Dieterbe Dieterbe merged commit f403caf into master Sep 23, 2019
@Dieterbe Dieterbe deleted the user-agent branch September 23, 2019 09:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants