Closed Bug 1551893 Opened 6 months ago Closed 6 months ago

Disallow non-zero-padded time element in Date.parse if time part marker T exists

Categories

(Core :: JavaScript Engine, defect, P3)

68 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: saschanaz, Assigned: saschanaz)

Details

(Keywords: parity-chrome, parity-safari, site-compat)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 Firefox/68.0

Steps to reproduce:

Date.parse("2019-05-15T1:1:1")

Actual results:

1557849661000

Expected results:

NaN

Bug 1500748 conditionally requires zero pads for date element but it still always permits non-padded time element of ISO8601.

Component: Untriaged → JavaScript Engine
Product: Firefox → Core
Keywords: parity-edge
Keywords: parity-safari

Make T as a strictness marker for ISO8601 time element, as it already does for the date element.

Arai, is this ready to land?

Flags: needinfo?(arai.unmht)
Priority: -- → P3

yes

Assignee: nobody → saschanaz
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(arai.unmht)
Keywords: checkin-needed

Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/00a761b7fce8
Require strict zero-pad for time element if T exists r=arai

Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.