-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Allow the Configuration of Additional Headers on the Jaeger Query HTTP API #2056
Allow the Configuration of Additional Headers on the Jaeger Query HTTP API #2056
Conversation
Functionality confirmed with local testing. Dropping Draft status. |
Codecov Report
@@ Coverage Diff @@
## master #2056 +/- ##
=========================================
+ Coverage 97.39% 97.4% +<.01%
=========================================
Files 207 209 +2
Lines 10307 10340 +33
=========================================
+ Hits 10039 10072 +33
Misses 224 224
Partials 44 44
Continue to review full report at Codecov.
|
85f13cb
to
4169a86
Compare
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
Co-Authored-By: Yuri Shkuro <[email protected]> Signed-off-by: Joe Elliott <[email protected]>
Co-Authored-By: Yuri Shkuro <[email protected]> Signed-off-by: Joe Elliott <[email protected]>
Co-Authored-By: Yuri Shkuro <[email protected]> Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
aa68742
to
5e269c1
Compare
Signed-off-by: Joe Elliott <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
restarting unit tests to get the coverage numbers (call to codecov timed out) |
Which problem is this PR solving?
Short description of the changes
query.additional-headers
that allows for multiple headers to be added to jaeger query's api--query.additional-headers "Access-Control-Allow-Origin: https://mozilla.org" --query-additional-headers "Access-Control-Expose-Headers: X-My-Custom-Header"