Bug: logger breaks on nodejs Docker lambdas since 2.4.0 about powertools-lambda-typescript HOT 5 CLOSED

I would like to take this issue :)

from powertools-lambda-typescript.

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

from powertools-lambda-typescript.

Hi @ralbertazzi - thank you for opening this issue.

I confirm that I am able to reproduce the issue:

I agree also with the fix proposed, if the value is :/etc/localtime we should default to :UTC. The fix should be applied in this function, which should be modified to do something like this:

public formatTimestamp(now: Date): string {
  const defaultTimezone = 'UTC';

  /**
   * If a specific timezone is configured and it's not the default `UTC`,
   * format the timestamp with the appropriate timezone offset.
   **/
- const configuredTimezone = this.envVarsService?.getTimezone();
+ const envVartimezone = this.envVarsService?.getTimezone();
+ const configuredTimezone =
+   envVartimezone === ':/etc/localtime' ? defaultTimezone : envVartimezone;
  if (configuredTimezone && !configuredTimezone.includes(defaultTimezone))
    return this.#generateISOTimestampWithOffset(now, configuredTimezone);

  return now.toISOString();
}

Additionally, we should also add a test case at the bottom of this block here to test for this case, similar to:

it('uses the default UTC timezone when the TZ env variable is set to `:/etc/localtime`', () => {
  // Prepare
  process.env.TZ = ':/etc/localtime';
  const formatter = new PowertoolsLogFormatter({
    envVarsService: new EnvironmentVariablesService(),
  });

  // Act
  const timestamp = formatter.formatTimestamp(new Date());

  // Assess
  expect(timestamp).toEqual('2016-06-20T12:08:10.000Z');
});

I'll mark the issue as help-wanted and good-first-issue to signify that contributions are very welcome.

If nobody picks it up, we'll work on it and include the fix in the next release (~next Wednesday).

In the meantime, as a stopgap solution you can set the TZ environment variable to :UTC to continue using this version of Logger.

In parallel, I'll also consult the AWS Lambda team to see if the container image should align its behavior with the managed runtime & set the value to :UTC.

from powertools-lambda-typescript.

⚠️ COMMENT VISIBILITY WARNING ⚠️

This issue is now closed. Please be mindful that future comments are hard for our team to see.

If you need more assistance, please either tag a team member or open a new issue that references this one.

If you wish to keep having a conversation with other community members under this issue feel free to do so.

from powertools-lambda-typescript.

This is now released under v2.7.0 version!

from powertools-lambda-typescript.