Skip to content
This repository was archived by the owner on Jul 11, 2025. It is now read-only.

Conversation

ktoso
Copy link
Member

@ktoso ktoso commented Jul 25, 2022

No description provided.

@ktoso ktoso requested a review from slashmo July 25, 2022 04:21
@ktoso ktoso force-pushed the wip-readme branch 2 times, most recently from 1267e0a to 9bff871 Compare July 25, 2022 07:02
Copy link
Contributor

@slashmo slashmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for adding 🙏

README.md Outdated
span.attributes.http.method = exampleRequest.method
span.attributes.http.url = exampleRequest.url
// ...
span.attributes.http.request.headers = exampleRequest.headers
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This wouldn't work the way I implemented headers. You can either set a specific header, e.g. span.attributes.http.request.headers.contentType = "application/json", or call a method on the headers attribute if you don't know the header names upfront.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right gottcha, I'll remove from the example snippet

@ktoso ktoso merged commit 6a8b708 into apple:main Jul 25, 2022
@ktoso ktoso deleted the wip-readme branch July 25, 2022 08:28
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