[yagruda]

Issue number: closes #8581

Summary

Changes

• In OpenAPIRequestValidationMiddleware:
  • Added sequence parameter handling in _normalize_multi_params and _process_model_param to convert single string values into single-item lists (and split comma strings if unparsed), ensuring sequence headers like list[str] validate consistently across HTTP API, Function URL, ALB, and VPC Lattice.
  • Allowed scalar header parameters to be joined via _restore_scalar_parameter by removing the multi-value exclusion for headers, aligning with RFC 9110 Section 5.3 where multiple header lines are equivalent to a comma-separated list.
  • Updated _get_body to extract the media type without parameters, compare case-insensitively, and accept +json media types (such as application/problem+json and application/vnd.api+json).
• Updated test_native_repeated_parameters_keep_existing_behavior in test_openapi_comma_parameters.py and added regression test cases in test_openapi_validation_middleware.py.

User experience

• Declaring a route header like x_country: Annotated[list[str], Header(alias="x-country")] accepts single values on all resolvers (e.g. x-country: US) instead of failing with 422 list_type on HTTP API / Function URL / ALB.
• Repeated header lines for a scalar string parameter (e.g. in multiValueHeaders) are joined into a comma-delimited string consistently across REST and HTTP API resolvers instead of raising 422 string_type.
• Valid requests with Content-Type: Application/JSON or application/problem+json are parsed as JSON rather than rejected with 415.

---

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

[boring-cyborg]

Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need.
In the meantime, check out the #python channel on our Powertools for AWS Lambda Discord: Invite link

[leandrodamascena]

Thanks @yagruda, confirmed on all three points. I ran it behind a real REST API and a real HTTP API with the same function: a single x-country value is accepted as a one-item list on every resolver, repeated x-req lines on REST come back as "a,b" like HTTP API already did, and Application/JSON, application/problem+json and application/vnd.api+json are parsed as JSON while text/plain still gets a 415. Comparing the media type exactly also closed a small hole, application/jsonx used to pass the startswith check.

Approving. This goes out in 3.39.0, so it will sit here for a while before I merge it. No rebase needed in the meantime, and you'll be credited in the release notes. I'll mention the repeated-header change on REST in the notes since it turns a 422 into a 200.

[leandrodamascena]

One more push from my side: the two [value] branches were unreachable, headers always arrive as a string and query strings as a list, so I removed them and added tests for the paths that were still uncovered: an optional sequence header that is not sent, and a header model whose list[str] field gets a single value or a comma-joined one. Patch coverage is 100% now. Re-approving, nothing changes for you.

[yagruda]

One more push from my side: the two [value] branches were unreachable, headers always arrive as a string and query strings as a list, so I removed them and added tests for the paths that were still uncovered: an optional sequence header that is not sent, and a header model whose list[str] field gets a single value or a comma-joined one. Patch coverage is 100% now. Re-approving, nothing changes for you.

Nice job, thank you!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.97%. Comparing base (8089f8a) to head (37959b1).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #8602      +/-   ##
===========================================
- Coverage    96.97%   96.97%   -0.01%     
===========================================
  Files          321      321              
  Lines        16277    16288      +11     
  Branches      1459     1462       +3     
===========================================
+ Hits         15785    15795      +10     
  Misses         352      352              
- Partials       140      141       +1     

☔ View full report in Codecov by Harness.
📢 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.

[boring-cyborg]

Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience!

[powertools-for-aws-oss-automation]

Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience!