Skip to content

feat: Add AWS WAF MCP server #224

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: Add AWS WAF MCP server #224

wants to merge 4 commits into from

Conversation

sfloresk
Copy link

Fixes

Summary

Add an implementation of AWS WAF MCP server

Changes

Include tools for creating AWS WAF ACL and associate it with an application load balancer.

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? (N)

RFC issue number: N/A

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.

@sfloresk sfloresk requested a review from a team as a code owner April 28, 2025 17:03
@sfloresk sfloresk changed the title Add AWS WAF MCP server feat: Add AWS WAF MCP server Apr 28, 2025
@@ -0,0 +1 @@
3.12
Copy link
Contributor

Choose a reason for hiding this comment

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

I see a difference in 3.12 here and 3.11 in the instructions. Is there anything restricting moving both to 3.13? I also see the project requiring >=3.12

Copy link
Author

Choose a reason for hiding this comment

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

I updated it to 3.13

@scottschreckengaust
Copy link
Contributor

Thank you for your submission. Please fill out an RFC issue. We now have code coverage requirements - would you be willing to add those please? A successful PR will maintain or increase the overall project coverage which is continually rising -> https://app.codecov.io/gh/awslabs/mcp

@scottschreckengaust scottschreckengaust added hold-merging Signals to hold the PR from merging new mcp server labels Apr 28, 2025
@sfloresk
Copy link
Author

@scottschreckengaust Added unit tests and moved to python3.13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold-merging Signals to hold the PR from merging new mcp server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants