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

Add Support for Parsing All Compliant ISO-8601 Date Formats #570

Closed
pfconrey opened this issue Oct 2, 2014 · 2 comments
Closed

Add Support for Parsing All Compliant ISO-8601 Date Formats #570

pfconrey opened this issue Oct 2, 2014 · 2 comments
Milestone

Comments

@pfconrey
Copy link

pfconrey commented Oct 2, 2014

Some providers create JSON date stamps in ISO-8601 formats that cannot be parsed by the jackson-databind library. Here is a sampling of some valid formats that do not parse correctly:

2014-10-03T18:00:00.6-05:00
2014-10-03T18:00:00.61-05:00
1997-07-16T19:20+01:00
1997-07-16T19:20:30.45+01:00

The last two actually come from the ISO-8601 notes on http://www.w3.org/TR/NOTE-datetime.

@cowtowncoder
Copy link
Member

Thank you for pointing out this gap!

ISO-8601 definition is big enough that I can't say this would get us to "all compliant variants", but definitely sounds like a reasonable improvement.
One thing I need to verify is just whether some of these might already be covered by one recent patch.

@cowtowncoder cowtowncoder added this to the 1.9.13 milestone Oct 3, 2014
cowtowncoder added a commit that referenced this issue Oct 3, 2014
@cowtowncoder cowtowncoder modified the milestones: 2.4.3, 1.9.13 Oct 3, 2014
@cowtowncoder cowtowncoder modified the milestones: 2.4.3, 2.5.0 Oct 3, 2014
cowtowncoder added a commit that referenced this issue Oct 3, 2014
cowtowncoder added a commit that referenced this issue Oct 3, 2014
@cowtowncoder
Copy link
Member

Ok. So, after verifying, this fix IS included in 2.4.3, as originally planned.

@cowtowncoder cowtowncoder modified the milestones: 2.5.0, 2.4.3 Oct 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants