[CDRIVER-2223] JSON reader cannot parse integers for legacy $date syntax Created: 26/Jul/17 Updated: 28/Oct/23 Resolved: 27/Jul/17 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | json, libbson |
| Affects Version/s: | None |
| Fix Version/s: | 1.7.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Jeremy Mikola | Assignee: | Jeremy Mikola |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Description |
|
933fd98 from
We should revert that commit and add some tests for reading legacy values. |
| Comments |
| Comment by Ramon Fernandez Marina [ 12/Sep/17 ] |
|
Author: {'username': u'jmikola', 'name': u'Jeremy Mikola', 'email': u'jmikola@gmail.com'}Message: Since 933fd98107d11e2f9e516c229c7eb23dad941812, the JSON reader emitted an This commit adds previously absent tests for legacy $date syntax and adds |
| Comment by Ramon Fernandez Marina [ 12/Sep/17 ] |
|
Author: {'username': u'jmikola', 'name': u'Jeremy Mikola', 'email': u'jmikola@gmail.com'}Message: Since d7406f5dc010f4eb223c9eef8c144aa6efc33504, all parse error tests were This commit also restructures the corpus test runner so that callbacks may be |
| Comment by Jeremy Mikola [ 27/Jul/17 ] |
|
Cherry-picked to r1.7: https://github.com/mongodb/libbson/commit/176b9c1489cb7452074550ef01ce1696f0e7c6ac |
| Comment by Githook User [ 27/Jul/17 ] |
|
Author: {'email': 'jmikola@gmail.com', 'username': 'jmikola', 'name': 'Jeremy Mikola'}Message: Since 933fd98107d11e2f9e516c229c7eb23dad941812, the JSON reader emitted an This commit adds previously absent tests for legacy $date syntax and adds |
| Comment by Githook User [ 27/Jul/17 ] |
|
Author: {'email': 'jmikola@gmail.com', 'username': 'jmikola', 'name': 'Jeremy Mikola'}Message: Since d7406f5dc010f4eb223c9eef8c144aa6efc33504, all parse error tests were This commit also restructures the corpus test runner so that callbacks may be |