Skip to content

Optionally include/exclude grpc headers by name#58

Open
jbf154 wants to merge 2 commits intoopentracing-contrib:masterfrom
jbf154:exclude_include_headers
Open

Optionally include/exclude grpc headers by name#58
jbf154 wants to merge 2 commits intoopentracing-contrib:masterfrom
jbf154:exclude_include_headers

Conversation

@jbf154
Copy link

@jbf154 jbf154 commented Aug 23, 2021

Sometimes headers can contain sensitive information (for example, the Authorization header). This update enables the user to optionally specify a list of headers to:

  • specifically exclude or
  • specifically include

@yurishkuro
Copy link

What relationship does tracing interceptor have with authorization headers?

@jbf154
Copy link
Author

jbf154 commented Aug 23, 2021

If you use any header (like the Authorization header) to transmit sensitive data and have opentracing configured to tag the HTTP headers in the span, that tag will have the contents of the sensitive header. Right now you can disable the HEADERS attribute, but that disables tagging all headers. Sometimes it is useful to tag some headers and omit others, hence this change.

In our use case, we are using OAuth bearer tokens passed into our services using the Authorization header. The tokens are sensitive and ideally would not be included in the spans generated.

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