fix(event-handler): handle aliased parameters e.g., Query(alias="categoryType")

[rubenfonseca]

Issue number: #3732

Summary

Changes

Please provide a summary of what's being changed

This PR fixes a bug when handling event handler parameters that used aliases.

def test():
    app = APIGatewayRestResolver(enable_validation=True)

    class FunkyTown(BaseModel):
        parameter: str

    @app.get("/my/path")
    def my_path(
        parameter: Annotated[str | None, Query(alias="parameter1")] = None,
    ) -> Response[FunkyTown]:
        assert isinstance(app.current_event, APIGatewayProxyEvent)

        assert parameter == "value1"
        return Response(200, content_types.APPLICATION_JSON, FunkyTown(parameter=parameter))

    result = app(load_event("apiGatewayProxyEvent.json"), {})

    print(result)
    assert result["statusCode"] == 200

User experience

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

After this PR, aliases should be correctly supported.

Checklist

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

• Meet tenets criteria
• I have performed a self-review of this change
• Changes have been tested
• Changes are documented
• PR title follows conventional commit semantics

Is this a breaking change?

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 your choice.

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

[heitorlessa]

Unsure why this test is failing; works locally. Pushing a no-op code change w/ a rebase, before we investigate further.

Edit: global dict mutation race condition w/ running multiple tests.

[heitorlessa]

Indeed racing condition. Reverting no-op change, and merging after CI completes all checks.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[heitorlessa]

thanks a lot for the fix @rubenfonseca 🙌