Skip to content

fix(data_class): merge querystring parameters in ALB/APIGW classes#8154

Merged
leandrodamascena merged 2 commits intodevelopfrom
fix/7993-merge-query-string-parameters
Apr 14, 2026
Merged

fix(data_class): merge querystring parameters in ALB/APIGW classes#8154
leandrodamascena merged 2 commits intodevelopfrom
fix/7993-merge-query-string-parameters

Conversation

@leandrodamascena
Copy link
Copy Markdown
Contributor

Issue number: closes #7993

Summary

This PR fixes a bug when using both queryStringParameters and multiValueQueryStringParameters in ALB and APIGW resolver.

Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change


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

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@leandrodamascena leandrodamascena requested a review from a team as a code owner April 14, 2026 10:06
@boring-cyborg boring-cyborg bot added the tests label Apr 14, 2026
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 14, 2026
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 14, 2026
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.69%. Comparing base (cd8829b) to head (18bec35).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #8154   +/-   ##
========================================
  Coverage    96.68%   96.69%           
========================================
  Files          285      285           
  Lines        14197    14209   +12     
  Branches      1175     1179    +4     
========================================
+ Hits         13727    13739   +12     
  Misses         341      341           
  Partials       129      129           

☔ 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.

@leandrodamascena leandrodamascena merged commit 7fc9efb into develop Apr 14, 2026
17 of 18 checks passed
@leandrodamascena leandrodamascena deleted the fix/7993-merge-query-string-parameters branch April 14, 2026 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files. tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: multiValueQueryStringParameters interferes with queryStringParameters

2 participants