Skip to content

Conversation

MaxOpperman
Copy link
Contributor

@MaxOpperman MaxOpperman commented Aug 13, 2025

Summary

Changes

Please provide a summary of what's being changed

  • Add an empty default logger region for local environments.

Edit: I tested this locally with my project that raised issue #4318 by building the powertools-lambda package and adding it to my node_modules. This fix resolved the issue.

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed.

Issue number: closes #4318


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.

Copy link

boring-cyborg bot commented Aug 13, 2025

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 #typescript channel on our Powertools for AWS Lambda Discord: Invite link

@boring-cyborg boring-cyborg bot added the logger This item relates to the Logger Utility label Aug 13, 2025
@pull-request-size pull-request-size bot added the size/XS PR between 0-9 LOC label Aug 13, 2025
@dreamorosi dreamorosi changed the title fix(logger): resolve issue when no logger region is set improv(logger): resolve issue when no logger region is set Aug 13, 2025
@boring-cyborg boring-cyborg bot added the tests PRs that add or change tests label Aug 13, 2025
@pull-request-size pull-request-size bot added size/M PR between 30-99 LOC and removed size/XS PR between 0-9 LOC labels Aug 13, 2025
@svozza svozza requested a review from dreamorosi August 13, 2025 11:07
@svozza
Copy link
Contributor

svozza commented Aug 13, 2025

Thank you so much for such a quick turnaround on identifying and fixing this issue @MaxOpperman!

Copy link

@svozza svozza merged commit b74b3b4 into aws-powertools:main Aug 13, 2025
34 checks passed
Copy link

boring-cyborg bot commented Aug 13, 2025

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

@ncino-esselman
Copy link

I was just opening an issue! When will this be packaged?

@dreamorosi
Copy link
Contributor

I was just opening an issue! When will this be packaged?

Most likely in the next 12-18hrs, unless issues arise.

@svozza
Copy link
Contributor

svozza commented Aug 14, 2025

Just a note that this fix has now been released as v2.25.1 on NPM. Once again, a big thank you to @MaxOpperman!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
logger This item relates to the Logger Utility size/M PR between 30-99 LOC tests PRs that add or change tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maintenance: AWS_REGION environment variable required after upgrading to @aws-lambda-powertools/logger v2.25.0
4 participants