Skip to content
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

cabfOrganizationIdentifier extension for VAT and PSD based organizationIdentifiers cannot have referenceStateOrProvince #848

Merged
merged 11 commits into from
Jun 17, 2024

Conversation

XolphinMartijn
Copy link
Contributor

No description provided.

LintMetadata: lint.LintMetadata{
Name: "e_cabf_org_identifier_psd_vat_has_state",
Description: "The cabfOrganizationIdentifier field for PSD org VAT Registration Schemes cannot include the referenceStateOrProvince field.",
Citation: "Current PSD and VAT based identifiers are issued from Country level only. No state or province information is included.",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The citation is usually the section/paragraph/clause number of the given requirement (E.G 7.1.1.2a).

Would you mind finding that and attaching it here? Its purpose is because the Description field is usually a paraphrasing and it is important for others to be able to quickly lookup the requirement in case a dispute arises.

Indeed, I tried to grep around in CABF EV 1.7.0 in order to independently verify this lint but I couldn't find it's source (I assume that it is there, but it's important that these are fast and easy to look up).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've instead added the section, which is 7.1.4.2.8.

The problem with this one is that it's not very clearly outlined. Section 7.1.2.2 is what this lint is based on, yet the definition of what goes where, is indirectly in 7.1.4.2.8, which shows that only NTR based orgIdentifiers have the state value.

Let me know if you see a better way of showing this within the code

Copy link
Member

@christopher-henderson christopher-henderson Jun 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I apologize deeply, but I'm a bit confused about the citation numbers.

I certainly see this requirement in Ballot SC17.

However, the entirety of section 7 in the EV guidelines is just two pages of document preamble.
image

I am quite sure that I'm staring something blank in the face, but I'm not seeing it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@christopher-henderson I just now see you're on V1.7.0. The EVGs were recently converted to the RFC3647 format, hence, I've used the current version's section numbers as reference:

https://github.com/cabforum/servercert/blob/main/docs/EVG.md#71428-subject-organization-identifier-field

For version 1.7.0, the relevant section number is 9.2.8.

(This has me wondering, which numbering should be used, the section-at-time-of-ballot, or section-at-time-of-lint-creation?)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for pointing me in the right direction @XolphinMartijn!

The EVGs were recently converted to the RFC3647 format

This may end up being problematic should CABF halt publishing they way that they have been all these years. However, ZLint generally makes references to the older format, so if you would not mind then I would appreciate that we cite 9.2.8.


Since this lint is paraphrasing, I presume that this is the requirement that you are citing:

For the NTR Registration Scheme identifier, if required under Section 7.1.4.2.4, a 2 character ISO 3166-2 identifier for the subdivision (state or province) of the nation in which the Registration Scheme is operated, preceded by plus "+" (0x2B (ASCII), U+002B (UTF-8));

That is, NTR is the only scheme that explicitly allows the State filed. Thus, this lint derives the inverse that a State is not legal for either VAT nor PSD schemes, correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may end up being problematic should CABF halt publishing they way that they have been all these years.

Should the question arrise, I will advocate the need to keep old versions around ;)

However, ZLint generally makes references to the older format, so if you would not mind then I would appreciate that we cite 9.2.8.

Sure. Done!

That is, NTR is the only scheme that explicitly allows the State filed. Thus, this lint derives the inverse that a State is not legal for either VAT nor PSD schemes, correct?

That is correct indeed.

@christopher-henderson christopher-henderson merged commit 04d863f into zmap:master Jun 17, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants