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

TemporalTimeZoneIdentifier and TimeZoneIANAName exclude some valid IANA time zone names #1993

Closed
gibson042 opened this issue Dec 28, 2021 · 0 comments · Fixed by #2050
Closed
Assignees
Labels
needs plenary input Needs to be presented to the committee and feedback incorporated spec-text Specification text involved

Comments

@gibson042
Copy link
Collaborator

gibson042 commented Dec 28, 2021

The IANA time zone database includes zones "Etc/GMT+0" through "Etc/GMT+12", but these are not matched by the TimeZoneIANAName production (and therefore not by the TemporalTimeZoneIdentifier production either) because "+" is not matched by TZChar (while the sibling zones "Etc/GMT-0" through "Etc/GMT-14" are matched because "-" is matched by TZChar). TimeZoneBracketedName specifically expands to Etc/GMT ASCIISign Hour, but even that doesn't really work because Hour only matches two-digit sequences 00 through 23 (resulting in incorrect acceptance of "Etc/GMT+00" through "Etc/GMT+09" and incorrect rejection of "Etc/GMT+0" through "Etc/GMT+9").

I think the special TimeZoneBracketedName expansion should be moved to TimeZoneIANAName and fixed to something like Etc/GMT ASCIISign Hour[+Unpadded] or Etc/GMT ASCIISign UnpaddedHour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs plenary input Needs to be presented to the committee and feedback incorporated spec-text Specification text involved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants