[GODRIVER-645] lookupInterfaceDecoder does not ensure type is a pointer Created: 21/Nov/18 Updated: 28/Oct/23 Resolved: 04/Dec/18 |
|
| Status: | Closed |
| Project: | Go Driver |
| Component/s: | BSON |
| Affects Version/s: | None |
| Fix Version/s: | 0.1.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Eric Daniels (Inactive) | Assignee: | Kristofer Brandow (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
A type that has a property of another struct that proxies a primitive like a string cannot be unmarshaled after being marshaled with the following error: "positioned on string, but attempted to read embedded document". Example code:
|
| Comments |
| Comment by Githook User [ 04/Dec/18 ] |
|
Author: {'name': 'Kris Brandow', 'email': 'kris@mongodb.com', 'username': 'skriptble'}Message: Update codec system to use reflect.Value Add support for pointers to the BSON codec system. When doing an This change changes the ValueEncoder and ValueDecoder interfaces to use Move the bson.D family of types into the primitive package.
Change-Id: Idec68f74e354973160bd59f9f8fef009f026f943 |
| Comment by Kristofer Brandow (Inactive) [ 28/Nov/18 ] |
|
Code Review: https://review.gerrithub.io/c/mongodb/mongo-go-driver/+/435374. |
| Comment by Ian Whalen (Inactive) [ 26/Nov/18 ] |
|
Marking as 0.1.0 so we don't lose this, but hoping this will just be fixed user-side. |
| Comment by Ian Whalen (Inactive) [ 26/Nov/18 ] |
|
eric.daniels can you change the EncValue property to a pointer? |