Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Pin EventGrid sample files URL to specific commit SHA#4770

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
Copilot wants to merge2 commits intomain
base:main
Choose a base branch
Loading
fromcopilot/update-eventgrid-sample-url

Conversation

Copy link
Contributor

CopilotAI commentedOct 29, 2025
edited
Loading

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

  • Pin GitHub API URL to commit752dade436619ed28bd03ca2e77cfa5acf6222dd where sample files still exist
  • Add documentation referencing upstream issues
// BeforeconstsampleFilesUrl='https://api.github.com/repos/Azure/azure-rest-api-specs/contents/specification/eventgrid/data-plane/'+'{eventSource}'+'/stable/2018-01-01/examples/cloud-events-schema/';// AfterconstsampleFilesUrl='https://api.github.com/repos/Azure/azure-rest-api-specs/contents/specification/eventgrid/data-plane/'+'{eventSource}'+'/stable/2018-01-01/examples/cloud-events-schema?ref=752dade436619ed28bd03ca2e77cfa5acf6222dd';

Restores 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-schema
    • Triggering command:curl -s -I REDACTED (http block)
    • Triggering command: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

This section details on the original issue you should resolve

<issue_title>Update EventGrid sample file URL to use specific commit SHA afterAzure/azure-rest-api-specs#38235</issue_title>
<issue_description>## Background
Recent changes in the Azure/azure-rest-api-specs repository (see issue#38235 and PR#38236) removed almost all EventGrid sample directories and example files from main. As a result, the following URL used inEventGridTypeStep.ts now fails:

const sampleFilesUrl =    'https://api.github.com/repos/Azure/azure-rest-api-specs/contents/specification/eventgrid/data-plane/' +    '{eventSource}' +    '/stable/2018-01-01/examples/cloud-events-schema/';

Proposal

Use a specific SHA reference when fetching EventGrid sample files from the GitHub API. This will ensure the extension continues to provide sample events for all supported sources, even though the files are no longer present in the main branch.

Suggested approach:

References

Impact

  • This workaround will allow the extension to continue functioning for EventGrid event sources despite the upstream file deletion.
  • Future changes in azure-rest-api-specs may require further updates.

Labels: bug, eventgrid
</issue_description>

Comments on the Issue (you are@copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn moreCopilot coding agent tips in the docs.

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 byAzure/azure-rest-api-specs#38235 andAzure/azure-rest-api-specs#38236Co-authored-by: alexweininger <12476526+alexweininger@users.noreply.github.com>
CopilotAI changed the title[WIP] Update EventGrid sample file URL to specific commit SHAPin EventGrid sample files URL to specific commit SHAOct 29, 2025
Copilot finished work on behalf ofalexweiningerOctober 29, 2025 18:25
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@alexweiningeralexweiningerAwaiting requested review from alexweininger

At least 1 approving review is required to merge this pull request.

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update EventGrid sample file URL to use specific commit SHA after Azure/azure-rest-api-specs#38235

2 participants

@alexweininger

[8]ページ先頭

©2009-2025 Movatter.jp