-
Notifications
You must be signed in to change notification settings - Fork 65
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
Improve XDS Validations #279
Comments
Subtask 1 -- Check completeness of XDS Associations, see above Subtask 2 -- Check uniqueness of uniqueIDs |
Current OIDValidator regexp require two digit at the begining of OID, but for example ISO root start with one digit. See https://wiki.ihe.net/index.php/OID_Registration and https://en.wikipedia.org/wiki/Object_identifier |
Note quite. It allows one or two digits at the beginning. |
I disagree. You can test it here https://regexr.com/38m0v |
@FinOxy: In the wikipedia link you have mentioned (e.g. https://en.wikipedia.org/wiki/Object_identifier) and also the ISO 9834 specification, you can see that the root arc "0" belong to ITU-T, here a official Oid from that tree: http://www.oid-info.com/cgi-bin/display?oid=0.9.2342.19200300.100.4 So the first digit can be only a 0,1 or 2 - so the regex is ok (but sure not perform according to the iso spec). The java JDK also bring a Oid class (org.ietf.jgss.Oid) which contain a strong validation (also following certain other rules of ISO 9834). We could think about replacing the regex with the validation by the Oid class constructor, but since validation is quiet often performed and might be performance critical, the regex might be sufficient for our purpose. |
As reported in https://groups.google.com/forum/#!topic/ipf-user/FTvCuqXSaJ4:
According to ITI TF-3:4.1.4, we have to check that the SubmissionSet contains Associations to all its DocumentEntries, Folders, and Folder-DocumentEntry Associations.
The text was updated successfully, but these errors were encountered: