Bug: v2.40.0 undocumented breaking change in Router

[sevkar]

Expected Behaviour

In out code we accessed current_event directly from Router class befor 2.40.0

Current Behaviour

In 2.40.0 version current_event available in AppSyncResolver only

Code snippet

from aws_lambda_powertools.event_handler import AppSyncResolver
from aws_lambda_powertools.event_handler.appsync import Router

user_router = Router()

@user_router.resolver(field_name="user")
def get_user() -> dict:
    return service.get_user_by_id(user_id=user_router.current_event.identity.sub)

app = AppSyncResolver()
app.include_router(user_router)

Possible Solution

No response

Steps to Reproduce

call current_event attribute from Router class

Powertools for AWS Lambda (Python) version

2.40.0

AWS Lambda function runtime

3.12

Packaging format used

Lambda Layers

Debugging logs

No response

[boring-cyborg]

Thanks for opening your first issue here! We'll come back to you as soon as we can.
In the meantime, check out the #python channel on our Powertools for AWS Lambda Discord: Invite link

[leandrodamascena]

Checking now.

[leandrodamascena]

Hi @karvetskiy, thanks for opening this issue and reporting this bug. Our tests didn't catch accessing current_event using the Router object directly and because of this we have this breaking change in this release, sorry about that. I'm working to fix and will release a new version in a few minutes.

Thanks

[leandrodamascena]

Hey @karvetskiy! If I publish a pre-release on Pypi, can you test if the issue is resolved in your environment before we release a new patch release? Thanks

[sevkar]

Hey @leandrodamascena! Just checked alpha version, still can't access current_event.
Looks like new Router class in event_handler.graphql_appsync.router inhered from wrong implementation of BaseRouter class.
It has different implementation in v2.39.1 (see code)

[heitorlessa]

he hasn't published yet, will ping you shortly @karvetskiy