Add Stripe CLI hosting integration #950
Merged
Conversation
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
Copilot
AI
changed the title
[WIP] Add hosting integration for Stripe CLI
Add Stripe CLI hosting integration
Nov 6, 2025
Co-authored-by: aaronpowell <434140+aaronpowell@users.noreply.github.com>
Co-authored-by: aaronpowell <434140+aaronpowell@users.noreply.github.com>
Co-authored-by: aaronpowell <434140+aaronpowell@users.noreply.github.com>
aaronpowell
force-pushed
the
copilot/add-stripe-cli-integration
branch
from
7823d96 to
d4abb78
Compare
Member
aaronpowell
temporarily deployed
to
azure-artifacts
GitHub Actions
Inactive
|
Is there a specific advantage on targeting the locally installed CLI rather than leveraging the existing container? It feels like it's adding a manual step (installation of the CLI) that can be skipped altogether. |
Why not use the container? As with anything Aspire-related, the whole point is that anyone can clone the repo, hit F5, and it just works. |
Member
|
That's a fair point - I was trying to simplify getting the signing code, but it comes at a cost of simplicity. The branch has a rewrite that uses the container version. |
aaronpowell
temporarily deployed
to
azure-artifacts
GitHub Actions
Inactive
Member
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds a hosting integration for Stripe CLI to enable local webhook forwarding and testing in Aspire applications. The integration wraps the Stripe CLI in a container and provides extension methods for configuring webhook forwarding to application endpoints.
- Adds
StripeResource(ContainerResource) and extension methods for Stripe CLI configuration - Implements webhook signing secret extraction from CLI logs
- Includes comprehensive test coverage (20 unit tests, 2 integration tests, example AppHost)
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 16 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/CommunityToolkit.Aspire.Testing/RequiresAuthenticatedToolAttribute.cs | New testing attribute for marking tests requiring authenticated external tools |
| tests/CommunityToolkit.Aspire.Testing/RequiresAuthenticatedToolDiscoverer.cs | xUnit trait discoverer for the RequiresAuthenticatedToolAttribute |
| tests/CommunityToolkit.Aspire.Hosting.Stripe.Tests/CommunityToolkit.Aspire.Hosting.Stripe.Tests.csproj | Test project configuration for Stripe integration tests |
| tests/CommunityToolkit.Aspire.Hosting.Stripe.Tests/AppHostTests.cs | Integration tests validating resource creation and health |
| tests/CommunityToolkit.Aspire.Hosting.Stripe.Tests/AddStripeTests.cs | Unit tests covering extension methods and argument configuration |
| src/CommunityToolkit.Aspire.Hosting.Stripe/StripeResource.cs | Resource definition for Stripe CLI container |
| src/CommunityToolkit.Aspire.Hosting.Stripe/StripeExtensions.cs | Extension methods for adding and configuring Stripe CLI |
| src/CommunityToolkit.Aspire.Hosting.Stripe/StripeContainerImageTags.cs | Container image configuration constants |
| src/CommunityToolkit.Aspire.Hosting.Stripe/README.md | Integration documentation and usage examples |
| src/CommunityToolkit.Aspire.Hosting.Stripe/CommunityToolkit.Aspire.Hosting.Stripe.csproj | Project configuration for hosting integration |
| examples/stripe/CommunityToolkit.Aspire.Hosting.Stripe.AppHost/Properties/launchSettings.json | Launch settings for example AppHost |
| examples/stripe/CommunityToolkit.Aspire.Hosting.Stripe.AppHost/Program.cs | Example AppHost demonstrating webhook forwarding |
| examples/stripe/CommunityToolkit.Aspire.Hosting.Stripe.AppHost/CommunityToolkit.Aspire.Hosting.Stripe.AppHost.csproj | AppHost project configuration |
| examples/stripe/CommunityToolkit.Aspire.Hosting.Stripe.Api/Properties/launchSettings.json | Launch settings for example API |
| examples/stripe/CommunityToolkit.Aspire.Hosting.Stripe.Api/Program.cs | Example API with webhook endpoint |
| examples/stripe/CommunityToolkit.Aspire.Hosting.Stripe.Api/CommunityToolkit.Aspire.Hosting.Stripe.Api.csproj | API project configuration |
| CommunityToolkit.Aspire.slnx | Solution file updated to include Stripe integration projects |
| .github/workflows/tests.yaml | CI workflow updated to include Stripe integration tests |
tests/CommunityToolkit.Aspire.Hosting.Stripe.Tests/AddStripeTests.cs
Outdated
Show resolved
Hide resolved
Comment on lines
84
to
86
| var stripe = builder.AddStripe("stripe", apiKey) | ||
| .WithListen("http://localhost:5082/webhooks") | ||
| .WithApiKey(apiKey); // optional: forwards the key to the CLI via --api-key |
There was a problem hiding this comment.
The example uses WithListen with a plain string URL, but there is no overload that accepts a string. The available overloads accept either IResourceBuilder<IResourceWithEndpoints> or IResourceBuilder<ExternalServiceResource>.
To fix this example:
var webhookEndpoint = builder.AddExternalService("webhook-endpoint", "http://localhost:5082");
var stripe = builder.AddStripe("stripe", apiKey)
.WithListen(webhookEndpoint, webhookPath: "/webhooks")
.WithApiKey(apiKey);
Comment on lines
206
to
213
| foreach (var line in logEvent) | ||
| { | ||
| if (TryExtractSigningSecret(line.Content, out var signingSecret)) | ||
| { | ||
| resource.WebhookSigningSecret = signingSecret; | ||
| return; | ||
| } | ||
| } |
There was a problem hiding this comment.
This foreach loop implicitly filters its target sequence - consider filtering the sequence explicitly using '.Where(...)'.
Comment on lines
17
to
24
| foreach (object? argument in traitAttribute.GetConstructorArguments()) | ||
| { | ||
| if (argument is string value) | ||
| { | ||
| toolName = value; | ||
| break; | ||
| } | ||
| } |
There was a problem hiding this comment.
This foreach loop implicitly filters its target sequence - consider filtering the sequence explicitly using '.Where(...)'.
tests/CommunityToolkit.Aspire.Hosting.Stripe.Tests/AddStripeTests.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
aaronpowell
reviewed
Dec 8, 2025
Member
aaronpowell
left a comment
aaronpowell
left a comment
There was a problem hiding this comment.
@copilot there's some comments on the README - can you ensure it's updated correctly with the API design
Co-authored-by: aaronpowell <434140+aaronpowell@users.noreply.github.com>
Contributor
Author
Updated the README with the correct API design. Changes in commit 4d55c0e:
|
Contributor
Minimum allowed line rate is |
aaronpowell
temporarily deployed
to
azure-artifacts
GitHub Actions
Inactive
This was referenced Jan 14, 2026
Bump CommunityToolkit.Aspire.Hosting.Flagd from 13.0.0 to 13.1.0
askpt/openfeature-aspire-sample#270
Closed
Closed
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.
Implementation Complete: Stripe CLI Hosting Integration
Successfully implemented a complete hosting integration for the Stripe CLI in the .NET Aspire Community Toolkit.
✅ All Tasks Completed
Integration Package (
src/CommunityToolkit.Aspire.Hosting.Stripe/)StripeResource- ContainerResource for Stripe CLIStripeExtensions- Comprehensive extension methodsTest Package (
tests/CommunityToolkit.Aspire.Hosting.Stripe.Tests/)Example Application (
examples/stripe/)Code Quality Improvements
Recent Fixes (Latest Commit)
AddExternalServiceinstead of plain string URLs["event1", "event2"]Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.