fix copilot.session.id classification#7253
Conversation
There was a problem hiding this comment.
Pull request overview
This PR aligns telemetry classification for copilot.session.id with the intended GDPR classification pipeline output by reclassifying it as SystemMetadata and emitting the raw session ID (no hashing), enabling correlation across Copilot agent events.
Changes:
- Reclassify
copilot.session.idfromEndUserPseudonymizedInformationtoSystemMetadata. - Stop hashing
copilot.session.idwhen writing usage and span attributes; emit the session ID directly.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
cli/azd/internal/tracing/fields/fields.go |
Updates CopilotSessionId classification to SystemMetadata and adjusts the field comment to reflect that it’s no longer hashed. |
cli/azd/internal/agent/copilot_agent.go |
Replaces hashed session-id telemetry emission with direct string attributes for both usage attributes and session spans. |
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash: pwsh: WindowsPowerShell install MSI install Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
Update `copilot.session.id` to the appropriate classification and avoid hashing. Contributes to Azure#7163
Update `copilot.session.id` to the appropriate classification and avoid hashing. Contributes to Azure#7163
Update `copilot.session.id` to the appropriate classification and avoid hashing. Contributes to Azure#7163
Update `copilot.session.id` to the appropriate classification and avoid hashing. Contributes to Azure#7163
Update `copilot.session.id` to the appropriate classification and avoid hashing. Contributes to Azure#7163
Update `copilot.session.id` to the appropriate classification and avoid hashing. Contributes to Azure#7163
Update
copilot.session.idto the appropriate classification and avoid hashing.Contributes to #7163