Pin EventGrid sample files URL to specific commit SHA#4770
Merged
Conversation
Add ?ref=752dade436619ed28bd03ca2e77cfa5acf6222dd query parameter to EventGrid sample files URL to ensure the extension continues to work after upstream files were removed from main branch. Fixes issue caused by Azure/azure-rest-api-specs#38235 and Azure/azure-rest-api-specs#38236 Co-authored-by: alexweininger <12476526+alexweininger@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update EventGrid sample file URL to specific commit SHA
Pin EventGrid sample files URL to specific commit SHA
Oct 29, 2025
Contributor
|
Converting old PR to draft. |
Member
|
I think this PR is fine and should resolve the issue which is the current GitHub repo removed all of the EventGrid sample files. |
Contributor
There was a problem hiding this comment.
Pull request overview
Pins the GitHub Contents API URL used to fetch EventGrid sample events to a known-good commit SHA so the extension can continue retrieving sample payloads after upstream files were removed from main.
Changes:
- Update
sampleFilesUrlto include?ref=<commitSha>so samples resolve consistently. - Add inline documentation pointing to the upstream issue/PR that removed the files.
| 'https://api.github.com/repos/Azure/azure-rest-api-specs/contents/specification/eventgrid/data-plane/' + | ||
| '{eventSource}' + | ||
| '/stable/2018-01-01/examples/cloud-events-schema/'; | ||
| '/stable/2018-01-01/examples/cloud-events-schema?ref=752dade436619ed28bd03ca2e77cfa5acf6222dd'; |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
bwateratmsft
approved these changes
Mar 17, 2026
bwateratmsft
approved these changes
Mar 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Azure/azure-rest-api-specs#38236 removed EventGrid sample files from main branch, breaking the extension's ability to fetch sample events for EventGrid function triggers.
Changes
752dade436619ed28bd03ca2e77cfa5acf6222ddwhere sample files still existRestores functionality for all supported EventGrid sources (Microsoft.Storage, Microsoft.EventHub, Microsoft.ServiceBus, etc.).
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
https://api.github.com/repos/Azure/azure-rest-api-specs/contents/specification/eventgrid/data-plane/Microsoft.Storage/stable/2018-01-01/examples/cloud-events-schemacurl -s -I REDACTED(http block)curl -s REDACTED(http block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.