468020 - Regex used to parse RFC3339 dates is incorrect

Status: RESOLVED FIXED

(Calendar :: Provider: GData, defect)

Description

[Patrick Cloke [:clokep]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1b2) Gecko/20081201 Firefox/3.1b2
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1b3pre) Gecko/20081201 Lightning/1.0pre Thunderbird/3.0b1

The regex given in the function fromRFC3339 to parse the RFC 3339 string is incorrect (in calGoogleUtils.js). It uses is trying to escape a period (.) for use with fraction of a second, but since it is contained within a string TWO back slashes are needed, not one.  This causes the wildcard (.) to match the first three characters of the timezone offset (i.e. -05) thinking its a fraction of a second.

Reproducible: Always

Steps to Reproduce:
Call the fromRFC3339 string with a RFC 3339 string formatted with a timezone (2008-12-04T21:10:00-05:00)
Actual Results:  
The regex matches the following:
year=2008
month=12
day=04
hour=21
minute=10
second=00
fraction-of-a-second=-05
With the timezone offset being unmatched

Expected Results:  
It matches the following:
year=2008
month=12
day=04
hour=21
minute=10
second=00
fraction-of-a-second unmatched
timezone-offset-hours=-05
timezone-offset-minutes=00

Comment 1

[Patrick Cloke [:clokep]]

Attachment: Regex Fix

Comment 2

[Philipp Kewisch [:Fallen][☁️📆]]

Comment on attachment 351506 [details] [diff] [review]
Regex Fix

Thanks for the patch. Fortunately, I haven't seen Google ever spit out split seconds in their dates. The fix is nevertheless needed though.

Comment 3

[Philipp Kewisch [:Fallen][☁️📆]]

A testsuite for these dates might be nice, requesting in-testsuite

Comment 4

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/031732472726

Please change status/fields as appropriate.

Comment 5

[Philipp Kewisch [:Fallen][☁️📆]]

These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".