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

X9CM OIDs belong ISO (1) not JOINT_ISO_ITU_T (2) #55

Open
naveedpash opened this issue Dec 14, 2021 · 4 comments
Open

X9CM OIDs belong ISO (1) not JOINT_ISO_ITU_T (2) #55

naveedpash opened this issue Dec 14, 2021 · 4 comments
Labels
area/standard Related to a ASN.1 standard implemented in rasn. area/types Related to rasn’s types for ASN.1 help wanted Extra attention is needed kind/bug Something isn't working

Comments

@naveedpash
Copy link

https://github.com/XAMPPRocky/rasn/blob/8f3a6731283bf4102ad9bd774b97bd69ec241c5e/src/types/oid.rs#L352

Hey, I am trying to implement ITU-T X.227 (ISO Association Control Service Element). Part of it requires me to write down the following OID:
joint-iso-itu-t(2) association-control(2) as-id(3) acse-ase(1) version(1)

So I decided to add it to your src/types/oid.rs file in then import it into my own crate (that I'm putting in the standards folder for a pull request later). That's when I noticed that a block of OIDs for X9-CM (or X9.57) is incorrectly written under root OID JOINT_ISO_ITU_T(2). I think it should be under ISO(1).

Here is the reference for my belief: Reference record for OID 1.2.840.10040.2.

Let me know if this is correct, I'll be happy to draw up a pull request.

@XAMPPRocky
Copy link
Collaborator

Thank you for your issue! It might be that it’s registered under both, because looking at RFC 5280 (the RFC where these OIDs came from), it’s under 2, 2. I would accept a PR adding the ISO versions of these OIDs.

https://datatracker.ietf.org/doc/html/rfc5280

@naveedpash
Copy link
Author

Ok, I'll submit one but just to be sure:
I can also add the OIDs of my interest from this arc:
joint-iso-itu-t(2) association-control(2) as-id(3) acse-ase(1) version(1)
without conflicting with the existing OIDs?

Meaning

oids! {
  JOINT_ISO_ITU_T_MEMBER_BODY => 2, 2;
}

would not conflict with

oids! {
  JOINT_ISO_ITU_T_ASSOCIATION_CONTROL => 2, 2;
}

right?

I think it should be fine because who ever uses your library just needs to import the correct arcs and use them...

@XAMPPRocky
Copy link
Collaborator

XAMPPRocky commented Dec 15, 2021

Yeah it should be fine, we don't actually ensure the tree is unique IIRC

@naveedpash
Copy link
Author

I've run into a bunch of these OIDs during my project,
I'll give a pull request once I've done gathered as many as necessary.

@XAMPPRocky XAMPPRocky added the area/standard Related to a ASN.1 standard implemented in rasn. label Apr 16, 2022
@XAMPPRocky XAMPPRocky added kind/bug Something isn't working area/types Related to rasn’s types for ASN.1 help wanted Extra attention is needed labels May 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/standard Related to a ASN.1 standard implemented in rasn. area/types Related to rasn’s types for ASN.1 help wanted Extra attention is needed kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants