feat(samples/cloudwatch-applicationsignals-mcp): add docker .net aspnetcore app #1746

ezhang6811 · 2025-11-13T21:30:19Z

Fixes

Summary

Adding Docker .NET Asp.NET Core app to our enablement tool testing infra.

Changes

This PR adds a new .NET ASP.NET Core baseline application for testing AWS Application Signals enablement via the get_enablement_guide MCP tool. The application is designed to run as a Docker container on EC2.

New files added

docker-apps/dotnet/aspnetcore/ - Complete ASP.NET Core application with:
- Dockerfile using .NET SDK 9.0 with health checks
- REST API with three endpoints: /, /health, and /api/buckets
- AWS SDK S3 integration to list S3 buckets
- Traffic generation script for continuous testing
- .csproj file with .NET 9.0, AWS SDK S3, and AWS Extensions dependencies
infrastructure/ec2/cdk/config/dotnet-aspnetcore.json - CDK configuration for deployment

Documentation updates:

Updated README.md with .NET ASP.NET Core app build/deploy instructions
Added dotnet-aspnetcore to ECR repository table
Added DotnetAspnetcoreCdkStack to CDK stack deployment table

User experience

N/A

Checklist

If your change doesn't seem to apply, please leave them unchecked.

I have reviewed the contributing guidelines
I have performed a self-review of this change
Changes have been tested
Changes are documented

Is this a breaking change? (Y/N) N

RFC issue number:

Checklist:

Migration process documented
Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

codecov · 2025-11-13T21:31:45Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.27%. Comparing base (bb62109) to head (d96005e).
⚠️ Report is 70 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1746   +/-   ##
=======================================
  Coverage   90.27%   90.27%           
=======================================
  Files         784      784           
  Lines       60059    60059           
  Branches     9733     9733           
=======================================
  Hits        54218    54218           
  Misses       3649     3649           
  Partials     2192     2192

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

github-advanced-security · 2025-11-13T21:31:55Z

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copilot

Pull Request Overview

This PR adds a new .NET ASP.NET Core Docker application to test AWS Application Signals enablement via the get_enablement_guide MCP tool. The application runs as a containerized service on EC2 with REST API endpoints for health checks and S3 bucket listing.

Key changes:

Added complete .NET ASP.NET Core application with Dockerfile, REST API endpoints, AWS S3 integration, and traffic generation script
Added CDK infrastructure configuration for EC2 deployment
Updated documentation with build/deploy instructions

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
dotnet-aspnetcore.json	CDK configuration defining app name, health check path, Docker image, language, and port settings
generate-traffic.sh	Bash script for continuous traffic generation to health and API endpoints
Program.cs	ASP.NET Core application with endpoints for health checks and S3 bucket listing
Dockerfile	Multi-stage Docker build using .NET SDK 9.0 with health checks and security hardening
AspNetCoreApp.csproj	.NET 9.0 project file with AWS S3 SDK dependencies
README.md	Updated documentation with dotnet-aspnetcore build/deploy instructions

Comments suppressed due to low confidence (1)

samples/cloudwatch-applicationsignals-mcp/get-enablement-guide-samples/docker-apps/dotnet/aspnetcore/Program.cs:25

This assignment to awsRegion is useless, since its value is never read.

var awsRegion = Environment.GetEnvironmentVariable("AWS_REGION") ?? "us-east-1";

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-11-13T21:33:25Z

...applicationsignals-mcp/get-enablement-guide-samples/docker-apps/dotnet/aspnetcore/Program.cs

+var awsRegion = Environment.GetEnvironmentVariable("AWS_REGION") ?? "us-east-1";
+


The awsRegion variable is declared but never used. It should either be removed or utilized in the S3 client configuration. Other applications in the repository (python-flask, nodejs-express) use the AWS_REGION environment variable to configure their S3 clients, but the .NET application relies on the default AWS SDK configuration via AddAWSService<IAmazonS3>() which doesn't accept explicit region configuration in this form.

Suggested change

var awsRegion = Environment.GetEnvironmentVariable("AWS_REGION") ?? "us-east-1";

...ignals-mcp/get-enablement-guide-samples/infrastructure/ec2/cdk/config/dotnet-aspnetcore.json

Copilot · 2025-11-13T21:33:26Z

...applicationsignals-mcp/get-enablement-guide-samples/docker-apps/dotnet/aspnetcore/Program.cs

+    catch (Exception ex)
+    {
+        logger.LogError(ex, "S3 client error: {ErrorMessage}", ex.Message);


Generic catch clause.

Suggested change

catch (Exception ex)

{

logger.LogError(ex, "S3 client error: {ErrorMessage}", ex.Message);

catch (AmazonS3Exception s3Ex)

{

logger.LogError(s3Ex, "AmazonS3Exception: {ErrorMessage}", s3Ex.Message);

return Results.Json(new { error = "AWS S3 error occurred while retrieving buckets" }, statusCode: 500);

}

catch (Exception ex)

{

logger.LogError(ex, "Unexpected error: {ErrorMessage}", ex.Message);

yiyuan-he · 2025-11-13T23:02:51Z

samples/cloudwatch-applicationsignals-mcp/get-enablement-guide-samples/README.md

 | python-flask       | docker-apps/python/flask     | python-flask    |
 | java-springboot    | docker-apps/java/spring-boot | java-springboot |
 | nodejs-express     | docker-apps/nodejs/express   | nodejs-express  |
+| dotnet-aspnetcore  | docker-apps/dotnet/aspnetcore| dotnet-aspnetcore|


yiyuan-he · 2025-11-13T23:03:15Z

samples/cloudwatch-applicationsignals-mcp/get-enablement-guide-samples/README.md

 | python-flask       | PythonFlaskCdkStack    |
 | java-springboot    | JavaSpringBootCdkStack |
 | nodejs-express     | NodejsExpressCdkStack  |
+| dotnet-aspnetcore  | DotnetAspnetcoreCdkStack|


yiyuan-he · 2025-11-13T23:05:02Z

...applicationsignals-mcp/get-enablement-guide-samples/docker-apps/dotnet/aspnetcore/Program.cs

sanity check: did you run deployment with existing ec2 cdk infra and verify things are working?

github-actions · 2025-11-29T01:34:36Z

This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

ezhang6811 added 6 commits November 13, 2025 12:48

create dotnet sample app

b562862

Merge branch 'main' into dotnet-core-app

a67d630

add root endpoint

d4afb65

add cdk config file

51c70d0

update readme

c4f86db

use dotnet 9

10dce23

Copilot AI review requested due to automatic review settings November 13, 2025 21:30

ezhang6811 requested review from a team, blairhyy-amazon, mxiamxia, syed-ahsan-ishtiaque, thpierce, vastin, wangzlei and yiyuan-he as code owners November 13, 2025 21:30

github-project-automation bot added this to awslabs/mcp Project Nov 13, 2025

github-project-automation bot moved this to To triage in awslabs/mcp Project Nov 13, 2025

Copilot started reviewing on behalf of ezhang6811 November 13, 2025 21:30 View session

Copilot finished reviewing on behalf of ezhang6811 November 13, 2025 21:32

Copilot AI reviewed Nov 13, 2025

View reviewed changes

format fix w pre-commit

a8bf9bd

yiyuan-he reviewed Nov 13, 2025

View reviewed changes

ezhang6811 added 2 commits November 14, 2025 10:13

Merge branch 'main' into dotnet-core-app

353bef3

update README

d96005e

scottschreckengaust added the hold-merging Signals to hold the PR from merging label Nov 14, 2025

yiyuan-he mentioned this pull request Nov 18, 2025

test: add .NET sample app and cdk infra aws-observability/aws-application-signals-test-framework#495

Merged

github-actions bot added stale These are items that have been around for a long time without progress and removed stale These are items that have been around for a long time without progress labels Nov 29, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(samples/cloudwatch-applicationsignals-mcp): add docker .net aspnetcore app #1746

feat(samples/cloudwatch-applicationsignals-mcp): add docker .net aspnetcore app #1746

ezhang6811 commented Nov 13, 2025

Uh oh!

codecov bot commented Nov 13, 2025 •

edited

Loading

Uh oh!

github-advanced-security bot commented Nov 13, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Nov 13, 2025

Uh oh!

Uh oh!

Copilot AI Nov 13, 2025

Uh oh!

yiyuan-he Nov 13, 2025 •

edited

Loading

Uh oh!

yiyuan-he Nov 13, 2025

Uh oh!

yiyuan-he Nov 13, 2025

Uh oh!

github-actions bot commented Nov 29, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		var awsRegion = Environment.GetEnvironmentVariable("AWS_REGION") ?? "us-east-1";

-    catch (Exception ex)
-    {
-        logger.LogError(ex, "S3 client error: {ErrorMessage}", ex.Message);
+    catch (AmazonS3Exception s3Ex)
+    {
+        logger.LogError(s3Ex, "AmazonS3Exception: {ErrorMessage}", s3Ex.Message);
+        return Results.Json(new { error = "AWS S3 error occurred while retrieving buckets" }, statusCode: 500);
+    }
+    catch (Exception ex)
+    {
+        logger.LogError(ex, "Unexpected error: {ErrorMessage}", ex.Message);

feat(samples/cloudwatch-applicationsignals-mcp): add docker .net aspnetcore app #1746

Are you sure you want to change the base?

feat(samples/cloudwatch-applicationsignals-mcp): add docker .net aspnetcore app #1746

Conversation

ezhang6811 commented Nov 13, 2025

Summary

Changes

User experience

Checklist

Acknowledgment

Uh oh!

codecov bot commented Nov 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

github-advanced-security bot commented Nov 13, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Nov 13, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Nov 13, 2025

Choose a reason for hiding this comment

Uh oh!

yiyuan-he Nov 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yiyuan-he Nov 13, 2025

Choose a reason for hiding this comment

Uh oh!

yiyuan-he Nov 13, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Nov 29, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

codecov bot commented Nov 13, 2025 •

edited

Loading

yiyuan-he Nov 13, 2025 •

edited

Loading