Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update tzdata to 2022b #44333

Closed
albertyw opened this issue Aug 22, 2022 · 1 comment
Closed

Update tzdata to 2022b #44333

albertyw opened this issue Aug 22, 2022 · 1 comment
Labels
dependencies Pull requests that update a dependency file.

Comments

@albertyw
Copy link
Contributor

albertyw commented Aug 22, 2022

Version

v12 and up

Platform

All platforms

Subsystem

icu-small

What steps will reproduce the bug?

$ node
Welcome to Node.js v18.7.0.
Type ".help" for more information.
> const data = new Date(Date.UTC(2022, 8, 6, 12));
undefined
> new Intl.DateTimeFormat('en-US', {timeZone: 'Europe/Kiev'}).format(data)
'10/5/2022'

> new Intl.DateTimeFormat('en-US', {timeZone: 'Europe/Kyiv'}).format(data)
Uncaught RangeError: Invalid time zone specified: Europe/Kyiv
    at new DateTimeFormat (<anonymous>)
# Should Resolve the same as Europe/Kiev

> new Intl.DateTimeFormat('en-US', {timeZone: 'America/Santiago', timeZoneName: ['shortOffset']}).format(data)
'9/6/2022, GMT-3'
# Should be GMT-4

How often does it reproduce? Is there a required condition?

This is reproducible on all version of node with a dependency on icu-small.

What is the expected behavior?

See changes from tzdata 2022b in http://mm.icann.org/pipermail/tz-announce/2022-August/000071.html

Note that tzdata 2022c has also been released (http://mm.icann.org/pipermail/tz-announce/2022-August/000072.html) but contains less substantive changes.

What do you see instead?

Incorrect future timezone conversions for Chile, Ukraine, and Iran.

Additional information

PR in progress.

albertyw added a commit to albertyw/node-1 that referenced this issue Aug 22, 2022
@daeyeon daeyeon added the dependencies Pull requests that update a dependency file. label Aug 22, 2022
@VoltrexKeyva
Copy link
Member

Updated by f59d8d3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file.
Projects
None yet
Development

No branches or pull requests

3 participants