feat(stepfunctions-tasks): add httpinvoke step functions task

[MathieuGilbert]

This adds an HttpInvoke Step Functions task construct, which allows calling public APIs as described here.

• Accepts an Event Bridge API Destination IConnection, setting required permissions to use its credentials.
• Grants permission to invoke the root path for the endpoint (apiRoot prop). This allows passing the relative endpoint path at execution time (apiEndpoint prop).
• Defines an enum for the allowed options for the URL encoding style of arrays, used when encoding the request body.

Closes #28278

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[aws-cdk-automation]

The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.

A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.

[aws-cdk-automation]

This PR has been in the CHANGES REQUESTED state for 3 weeks, and looks abandoned. To keep this PR from being closed, please continue work on it. If not, it will automatically be closed in a week.

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

[msambol]

Great stuff. Mostly nits on formatting.

[TheRealAmazonKendra]

Thank you for your contribution! This looks great overall but I just have one suggested change (in addition to the feedback already given).

[TheRealAmazonKendra]

Given that this field is reliant on urlEncodeBody being set, these two fields should probably be one field. Maybe the only field here should still be called urlEncodeBody but have it take an Encoding (not sure about this name) class with a static function that has an optional input of ArrayEncodingFormat. Something along the lines of

urlEncodeBody: Encoding.set(ArrayEncodingFormat.INDICES);

to include encoding format, or

urlEncodeBody: Encoding.set();

to just include encoding with the default format.

This way, if they leave off this prop altogether, it's defaulted to false.

[TheRealAmazonKendra]

I'll also add the note here that set probably isn't a great function name here.

[MathieuGilbert]

Thanks for pointing this out, I agree it's a bit awkward with the 2 params. I played around a bit, and I think a single prop can work with a couple extra enum options and renaming it to urlEncodingFormat. You can not pass it or specify URLEncodingFormat.NONE to get no encoding, pass it as URLEncodingFormat.DEFAULT to get encoding with the default array encoding (the underlying service defaults to INDICES), or specify the existing values to use those.

Pull request has been modified.

[MathieuGilbert]

@msambol @TheRealAmazonKendra Thanks for reviewing, I think I've addressed all of your comments. Please let me know if there's anything else!

[TheRealAmazonKendra]

Thank you for all your work on this!

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[TheRealAmazonKendra]

@Mergifyio update

[mergify]

update

❌ Mergify doesn't have permission to update

For security reasons, Mergify can't update this pull request. Try updating locally.
GitHub response: refusing to allow a GitHub App to create or update workflow .github/workflows/request-cli-integ-test.yml without workflows permission

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
• Commit ID: f784b79
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).