[CSHARP-4530] TryParseGenericTypeName error in non generic type Created: 16/Feb/23 Updated: 27/Oct/23 Resolved: 06/Mar/23 |
|
| Status: | Closed |
| Project: | C# Driver |
| Component/s: | Serialization |
| Affects Version/s: | 2.19.0 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Unknown |
| Reporter: | Andrea Guerrieri | Assignee: | Boris Dogadov |
| Resolution: | Gone away | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
| Comments |
| Comment by PM Bot [ 06/Mar/23 ] | |||||||
|
There hasn't been any recent activity on this ticket, so we're resolving it. Thanks for reaching out! Please feel free to comment on this if you're able to provide more information. | |||||||
| Comment by Boris Dogadov [ 17/Feb/23 ] | |||||||
|
We only get to that particular recursion call because System.Type.GetType("System.Collections.Generic.List`1[[JobManager.Common.TimeFilter, JobManager.Common]]") call returns null for some reason. Could you please check whether
returns a valid type, and what does
return? If order to investigate further, we would need to reproduce the issue.
that demonstrates the failure? Thanks.
| |||||||
| Comment by Andrea Guerrieri [ 17/Feb/23 ] | |||||||
|
the problem seem in the implementation of TryParseGenericTypeName
is not enough check for [ to consider the typeName as Generic, you should look imho at ` symbol to ensure that there is an actual arguments list. Or at worse, return false if typeArgumentNames array is empty or contains only empty strings. JobManager.Common.TimeFilter exists in the assembly | |||||||
| Comment by Boris Dogadov [ 17/Feb/23 ] | |||||||
|
Hi maillist@eusoft.net, thank you for the all detailed info. I was able to reproduce this behaviour only when type discriminator for the generic type ([JobManager.Common.TimeFilter, JobManager.Common]) was invalid. My suspicion is that JobManager.Common.TimeFilter type does not exist in JobManager.Common assembly. |