Merge branch 'use-content-type-application-proto-for-pyroscope-querier-responses'

awesense-paul · awesense-paul · commit 5184d5eb6b39 · 2025-03-05T14:10:10.000-08:00
diff --git a/pyroscope/shared.js b/pyroscope/shared.js
@@ -66,7 +66,7 @@ const wrapResponse = (hndl) => {
       const strRes = JSON.stringify(normalizeProtoResponse(_res.toObject()))
       return res.code(200).send(strRes)
     }
-    return res.code(200).send(Buffer.from(_res.serializeBinary()))
+    return res.code(200).type('application/proto').send(Buffer.from(_res.serializeBinary()))
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ const wrapResponse = (hndl) => {`
`66`	`66`	`const strRes = JSON.stringify(normalizeProtoResponse(_res.toObject()))`
`67`	`67`	`return res.code(200).send(strRes)`
`68`	`68`	`}`
`69`		`- return res.code(200).send(Buffer.from(_res.serializeBinary()))`
	`69`	`+ return res.code(200).type('application/proto').send(Buffer.from(_res.serializeBinary()))`
`70`	`70`	`}`
`71`	`71`	`}`
`72`	`72`