-
Notifications
You must be signed in to change notification settings - Fork 246
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
Fails on chinese passport #202
Comments
Not all passports follow the ICAO standard regarding listing Datagroups in the COM's 5C tag. They actually list some elements in the COM 5C tag that is not in the ICAO-standard. This should perhaps not give an error, as reading the passport itself should be fine. Perhaps a solution here could be to return the PassportModel in the callback instead of an error, and add a "warning/errors" parameter to the Model so consumers can be notified of the deviations that has occured. So what actually happens here is that a DG-group is listed in the COM, but it does not actually exist. So it should be skipped instead of throwing |
I have noticed this too. There can be 19 DataGroups but the app only supports to 16. Try and add DG17-19 with no parsing. |
Theres no official documentation on those datagroups. i will try to create a pr for this when i have the time |
@iAbdul could you try again with the branch on the PR above and see if this fixes it for you? Please comment in the PR so that if/when @AndyQ eventually gets around to it he can see that the fix is verified. |
|
Hi,
Code works good with Canadian and usa passport however it fails with Chinese passport and shows alert "unsupported data group". Can you guide me how to make it work on Chinese passport.
Thanks!
The text was updated successfully, but these errors were encountered: