-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add READ_NUMERIC_STRING_AS_DATE_TIMESTAMP DeserializationFeature #3830
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 READ_NUMERIC_STRING_AS_DATE_TIMESTAMP DeserializationFeature #3830
Conversation
To read numeric strings as date timestamp[1] a new feature toggle is needed. 1: FasterXML/jackson-modules-java8#263
Hmmh. Would this more generally work with timestamps for I guess I am trying to think of valid reasons to add (more) support for "quoted numbers" -- ideally those would not be used given JSON has native distinction between numbers and Strings. |
I'll answer here: FasterXML/jackson-modules-java8#263. Best to keep it all in one place. |
Currently as imagined, it would only work with the JSR310 classes |
Ok; I added longer description in FasterXML/jackson-modules-java8#263 but basically:
So I'd rather get it added directly in that module as config option there, and avoid adding "too specific" |
That makes sense. I wasn't aware the module also had it's own options. |
Currently it is possible there aren't, but in general modules definite can have configuration options; some do, many don't (many use central configuration options to change their behavior). |
Closing in favor of adding something like |
To read numeric strings as date timestamp[1] a new feature toggle is needed.
1: FasterXML/jackson-modules-java8#263
This PR is paired with FasterXML/jackson-modules-java8#269.