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

Add more features for adding HTTP request / response headers to spans. #1184

Closed
rogersd opened this issue Jul 6, 2022 · 0 comments · Fixed by #1253, #1333, #1402, #1403 or #1404
Closed

Add more features for adding HTTP request / response headers to spans. #1184

rogersd opened this issue Jul 6, 2022 · 0 comments · Fixed by #1253, #1333, #1402, #1403 or #1404

Comments

@rogersd
Copy link
Contributor

rogersd commented Jul 6, 2022

I already have #1172 open for this, and I'll be breaking it in to smaller pieces at @lzchen 's request.

Is your feature request related to a problem?
Currently, you can only provide a list of full HTTP request / response header names to be added to the span.

There is also no capacity for header value redaction.

Describe the solution you'd like
It would be nice to be able to specify a regex or "all" to get all headers.

Header value redaction is also a must-have for us.

Describe alternatives you've considered
I considered doing this in my application, but it makes more sense to add it here.

rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Aug 30, 2022
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Aug 30, 2022
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Aug 30, 2022
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Sep 12, 2022
srikanthccv pushed a commit that referenced this issue Sep 13, 2022
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Sep 14, 2022
… in ASGI.

Next part of open-telemetry#1184

Details:

Add test cases for regular expression matching.
Add test cases for "all" keyword.
Add test cases for header sanitizing.
Add documentation for regular expression matching and header sanitation.
Various documentation cleanups and standardization.
Fix keys() in class ASGIGetter so it returns the HTTP header keys instead of a list of available request data. This makes it consistent with the WSGIGetter keys() method.
Make ASGIGetter.get() compare all keys in a case insensitive manner.
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Sep 16, 2022
… in ASGI.

Next part of open-telemetry#1184

Details:

Add test cases for regular expression matching.
Add test cases for "all" keyword.
Add test cases for header sanitizing.
Add documentation for regular expression matching and header sanitation.
Various documentation cleanups and standardization.
Fix keys() in class ASGIGetter so it returns the HTTP header keys instead of a list of available request data. This makes it consistent with the WSGIGetter keys() method.
Make ASGIGetter.get() compare all keys in a case insensitive manner.
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Oct 17, 2022
… in ASGI.

Next part of open-telemetry#1184

Details:

Add test cases for regular expression matching.
Add test cases for header sanitizing.
Add documentation for regular expression matching and header sanitation.
Various documentation cleanups and standardization.
Fix keys() in class ASGIGetter so it returns the HTTP header keys instead of a list of available request data. This makes it consistent with the WSGIGetter keys() method.
Make ASGIGetter.get() compare all keys in a case insensitive manner.
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Oct 17, 2022
… in ASGI.

Next part of open-telemetry#1184

Details:

Add test cases for regular expression matching.
Add test cases for header sanitizing.
Add documentation for regular expression matching and header sanitation.
Various documentation cleanups and standardization.
Fix keys() in class ASGIGetter so it returns the HTTP header keys instead of a list of available request data. This makes it consistent with the WSGIGetter keys() method.
Make ASGIGetter.get() compare all keys in a case insensitive manner.
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Oct 17, 2022
… in ASGI.

Next part of open-telemetry#1184

Details:

Add test cases for regular expression matching.
Add test cases for header sanitizing.
Add documentation for regular expression matching and header sanitation.
Various documentation cleanups and standardization.
Fix keys() in class ASGIGetter so it returns the HTTP header keys instead of a list of available request data. This makes it consistent with the WSGIGetter keys() method.
Make ASGIGetter.get() compare all keys in a case insensitive manner.
Move TestCustomHeaders to a separate file to avoid module length error.
rogersd pushed a commit to rogersd/opentelemetry-python-contrib that referenced this issue Oct 17, 2022
… in ASGI.

Next part of open-telemetry#1184

Details:

Add test cases for regular expression matching.
Add test cases for header sanitizing.
Add documentation for regular expression matching and header sanitation.
Various documentation cleanups and standardization.
Fix keys() in class ASGIGetter so it returns the HTTP header keys instead of a list of available request data. This makes it consistent with the WSGIGetter keys() method.
Make ASGIGetter.get() compare all keys in a case insensitive manner.
Move TestCustomHeaders to a separate file to avoid module length error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment