Skip to content

chore: sync telemetry reporter allowlists - #3482

Closed
ghcp4a-bot[bot] wants to merge 1 commit into
mainfrom
sync/telemetry-reporter-allowlists
Closed

ghcp4a-bot[bot] wants to merge 1 commit into
mainfrom
sync/telemetry-reporter-allowlists

Conversation

@ghcp4a-bot

@ghcp4a-bot ghcp4a-bot Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Skill and reference-file allowlists can drift as plugin sources change. This automated PR refreshes the standalone telemetry reporter's telemetry-reporter/resources/allowed-skill-names.json and telemetry-reporter/resources/allowed-plugin-file-references.json with the lists generated from GitHub-Copilot-for-Azure at commit 0a44db435bce8a28c45c88b4fc4e8b8ff9364bb2. Reporter builds and releases remain unchanged.

@ghcp4a-bot
ghcp4a-bot Bot requested a review from a team as a code owner November 4, 2026 11:06
Copilot AI balanced review requested due to automatic review settings November 4, 2026 11:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues are recorded for the telemetry reporter’s allowlist update.

No open issues 🎉

What changed in this PR

This PR refreshes the standalone telemetry reporter’s file-reference allowlist to recognize newly added Azure AI speech diagnostic resources.

Changes:

  • Adds eight speech reference documents and three log-analysis scripts to the allowlist.
  • Leaves reporter builds and releases unchanged.
File Description
telemetry-reporter/​resources/​allowed-plugin-file-references.json Adds 11 Azure AI speech document and script paths to the telemetry allowlist.

Review effort: Balanced

@ghcp4a-bot
ghcp4a-bot Bot force-pushed the sync/telemetry-reporter-allowlists branch from cf82f13 to 2be2568 Compare November 5, 2026 11:05
@ghcp4a-bot
ghcp4a-bot Bot force-pushed the sync/telemetry-reporter-allowlists branch 3 times, most recently from 3657e0a to 764a5d0 Compare November 10, 2026 11:31
@ghcp4a-bot
ghcp4a-bot Bot force-pushed the sync/telemetry-reporter-allowlists branch from 764a5d0 to c4036bc Compare November 11, 2026 11:04
@tmeschter

Copy link
Copy Markdown
Member

This is no longer needed, as the telemetry reporter requires the allow lists to be passed in as arguments rather than using embedded versions.

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