microsoftgraph / microsoftgraph/msgraph-sdk-php
Telemetry handler experience
Open
Nobody has claimed this yet.
request: feature
- Dominant language
- PHP
- Stars
- 669
- Forks
- 150
- Avg merge
- 15h 21m
- Merged PRs (30d)
- 3
Description
2 things to address:
- The telemetry handler would only set the client request Id if not already present, allowing people to set it manually if they need some kind of end-to-end tracing.
- The telemetry handler should start a span and attach the client request id as a tag.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the telemetry handler in the PHP SDK and read its existing client request ID behavior first. Complete the unchecked work by starting a span and attaching the client request ID as a tag; the issue does not name a file or test, so verify the behavior using the repository's relevant telemetry tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- observability
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100